ashatora/openlayers
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Cache array length in ol.Map

Browse Source
This commit is contained in:
Tom Payne
2013-04-20 11:27:55 +02:00
parent 332fd250b4
commit e9b0e2b0e8
1 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/ol/map.js
View File

@@ -617,8 +617,9 @@ ol.Map.prototype.handleMapBrowserEvent = function(mapBrowserEvent) {
var interactions = this.getInteractions(); var interactions = this.getInteractions();
var interactionsArray = /** @type {Array.<ol.interaction.Interaction>} */ var interactionsArray = /** @type {Array.<ol.interaction.Interaction>} */
(interactions.getArray()); (interactions.getArray());
var i;
if (this.dispatchEvent(mapBrowserEvent) !== false) { if (this.dispatchEvent(mapBrowserEvent) !== false) {
for (var i = interactionsArray.length - 1; i >= 0; i--) { for (i = interactionsArray.length - 1; i >= 0; i--) {
var interaction = interactionsArray[i]; var interaction = interactionsArray[i];
interaction.handleMapBrowserEvent(mapBrowserEvent); interaction.handleMapBrowserEvent(mapBrowserEvent);
if (mapBrowserEvent.defaultPrevented) { if (mapBrowserEvent.defaultPrevented) {
@@ -663,8 +664,8 @@ ol.Map.prototype.handlePostRender = function() {
} }
var postRenderFunctions = this.postRenderFunctions_; var postRenderFunctions = this.postRenderFunctions_;
var i; var i, ii;
for (i = 0; i < postRenderFunctions.length; ++i) { for (i = 0, ii = postRenderFunctions.length; i < ii; ++i) {
postRenderFunctions[i](this, frameState); postRenderFunctions[i](this, frameState);
} }
postRenderFunctions.length = 0; postRenderFunctions.length = 0;
@@ -796,8 +797,8 @@ ol.Map.prototype.renderFrame_ = function(time) {
var backgroundColor = this.getBackgroundColor(); var backgroundColor = this.getBackgroundColor();
var viewHints = view.getHints(); var viewHints = view.getHints();
var layerStates = {}; var layerStates = {};
var layer; var ii, layer;
for (i = 0; i < layersArray.length; ++i) { for (i = 0, ii = layersArray.length; i < ii; ++i) {
layer = layersArray[i]; layer = layersArray[i];
layerStates[goog.getUid(layer)] = layer.getLayerState(); layerStates[goog.getUid(layer)] = layer.getLayerState();
} }
@@ -828,7 +829,7 @@ ol.Map.prototype.renderFrame_ = function(time) {
var preRenderFunctions = this.preRenderFunctions_; var preRenderFunctions = this.preRenderFunctions_;
var n = 0, preRenderFunction; var n = 0, preRenderFunction;
for (i = 0; i < preRenderFunctions.length; ++i) { for (i = 0, ii = preRenderFunctions.length; i < ii; ++i) {
preRenderFunction = preRenderFunctions[i]; preRenderFunction = preRenderFunctions[i];
if (preRenderFunction(this, frameState)) { if (preRenderFunction(this, frameState)) {
preRenderFunctions[n++] = preRenderFunction; preRenderFunctions[n++] = preRenderFunction;
@@ -1003,8 +1004,9 @@ ol.Map.createOptionsInternal = function(options) {
rendererHints = ol.DEFAULT_RENDERER_HINTS; rendererHints = ol.DEFAULT_RENDERER_HINTS;
} }
var n = rendererHints.length;
var i, rendererHint; var i, rendererHint;
for (i = 0; i < rendererHints.length; ++i) { for (i = 0; i < n; ++i) {
rendererHint = rendererHints[i]; rendererHint = rendererHints[i];
if (rendererHint == ol.RendererHint.CANVAS) { if (rendererHint == ol.RendererHint.CANVAS) {
if (ol.ENABLE_CANVAS && ol.renderer.canvas.SUPPORTED) { if (ol.ENABLE_CANVAS && ol.renderer.canvas.SUPPORTED) {