﻿/**
* @name MarkerManager
* @version 1.0
* @copyright (c) 2007 Google Inc.
* @author Doug Ricket, others
*
* @fileoverview Marker manager is an interface between the map and the user,
* designed to manage adding and removing many points when the viewport changes.
* <br /><br />
* <b>How it Works</b>:<br/> 
* The MarkerManager places its markers onto a grid, similar to the map tiles.
* When the user moves the viewport, it computes which grid cells have
* entered or left the viewport, and shows or hides all the markers in those
* cells.
* (If the users scrolls the viewport beyond the markers that are loaded,
* no markers will be visible until the <code>EVENT_moveend</code> 
* triggers an update.)
* In practical consequences, this allows 10,000 markers to be distributed over
* a large area, and as long as only 100-200 are visible in any given viewport,
* the user will see good performance corresponding to the 100 visible markers,
* rather than poor performance corresponding to the total 10,000 markers.
* Note that some code is optimized for speed over space,
* with the goal of accommodating thousands of markers.
*/

/*
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
* You may obtain a copy of the License at
*
*     http://www.apache.org/licenses/LICENSE-2.0
*
* Unless required by applicable law or agreed to in writing, software
* distributed under the License is distributed on an "AS IS" BASIS,
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
* See the License for the specific language governing permissions and
* limitations under the License. 
*/

/**
* @name MarkerManagerOptions
* @class This class represents optional arguments to the {@link MarkerManager}
*     constructor.
* @property {Number} [maxZoom] Sets the maximum zoom level monitored by a
*     marker manager. If not given, the manager assumes the maximum map zoom
*     level. This value is also used when markers are added to the manager
*     without the optional {@link maxZoom} parameter.
* @property {Number} [borderPadding] Specifies, in pixels, the extra padding
*     outside the map's current viewport monitored by a manager. Markers that
*     fall within this padding are added to the map, even if they are not fully
*     visible.
* @property {Boolean} [trackMarkers=false] Indicates whether or not a marker
*     manager should track markers' movements. If you wish to move managed
*     markers using the {@link setPoint}/{@link setLatLng} methods, 
*     this option should be set to {@link true}.
*/

/**
* Creates a new MarkerManager that will show/hide markers on a map.
*
* @constructor
* @param {Map} map The map to manage.
* @param {Object} opt_opts A container for optional arguments:
*   {Number} maxZoom The maximum zoom level for which to create tiles.
*   {Number} borderPadding The width in pixels beyond the map border,
*                   where markers should be display.
*   {Boolean} trackMarkers Whether or not this manager should track marker
*                   movements.
*/
function MarkerManager(map, opt_opts) {
    var me = this;
    me.map_ = map;
    me.mapZoom_ = map.getZoom();
    me.projection_ = map.getCurrentMapType().getProjection();

    opt_opts = opt_opts || {};
    me.tileSize_ = MarkerManager.DEFAULT_TILE_SIZE_;

    var mapTypes = map.getMapTypes();
    var mapMaxZoom = mapTypes[0].getMaximumResolution();
    for (var i = 0; i < mapTypes.length; i++) {
        var mapTypeMaxZoom = mapTypes[i].getMaximumResolution();
        if (mapTypeMaxZoom > mapMaxZoom) {
            mapMaxZoom = mapTypeMaxZoom;
        }
    }
    me.maxZoom_ = opt_opts.maxZoom || mapMaxZoom;

    me.trackMarkers_ = opt_opts.trackMarkers;
    me.show_ = opt_opts.show || true;

    var padding;
    if (typeof opt_opts.borderPadding === "number") {
        padding = opt_opts.borderPadding;
    } else {
        padding = MarkerManager.DEFAULT_BORDER_PADDING_;
    }
    // The padding in pixels beyond the viewport, where we will pre-load markers.
    me.swPadding_ = new GSize(-padding, padding);
    me.nePadding_ = new GSize(padding, -padding);
    me.borderPadding_ = padding;

    me.gridWidth_ = [];

    me.grid_ = [];
    me.grid_[me.maxZoom_] = [];
    me.numMarkers_ = [];
    me.numMarkers_[me.maxZoom_] = 0;

    GEvent.bind(map, "moveend", me, me.onMapMoveEnd_);

    // NOTE: These two closures provide easy access to the map.
    // They are used as callbacks, not as methods.
    me.removeOverlay_ = function(marker) {
        map.removeOverlay(marker);
        me.shownMarkers_--;
    };
    me.addOverlay_ = function(marker) {
        if (me.show_) {
            map.addOverlay(marker);
            me.shownMarkers_++;
        }
    };

    me.resetManager_();
    me.shownMarkers_ = 0;

    me.shownBounds_ = me.getMapGridBounds_();
}

// Static constants:
MarkerManager.DEFAULT_TILE_SIZE_ = 1024;
MarkerManager.DEFAULT_BORDER_PADDING_ = 100;
MarkerManager.MERCATOR_ZOOM_LEVEL_ZERO_RANGE = 256;


/**
* Initializes MarkerManager arrays for all zoom levels
* Called by constructor and by clearAllMarkers
*/
MarkerManager.prototype.resetManager_ = function() {
    var me = this;
    var mapWidth = MarkerManager.MERCATOR_ZOOM_LEVEL_ZERO_RANGE;
    for (var zoom = 0; zoom <= me.maxZoom_; ++zoom) {
        me.grid_[zoom] = [];
        me.numMarkers_[zoom] = 0;
        me.gridWidth_[zoom] = Math.ceil(mapWidth / me.tileSize_);
        mapWidth <<= 1;
    }
};

/**
* Removes all markers in the manager, and
* removes any visible markers from the map.
*/
MarkerManager.prototype.clearMarkers = function() {
    var me = this;
    me.processAll_(me.shownBounds_, me.removeOverlay_);
    me.resetManager_();
};


/**
* Gets the tile coordinate for a given latlng point.
*
* @param {LatLng} latlng The geographical point.
* @param {Number} zoom The zoom level.
* @param {GSize} padding The padding used to shift the pixel coordinate.
*               Used for expanding a bounds to include an extra padding
*               of pixels surrounding the bounds.
* @return {GPoint} The point in tile coordinates.
*
*/
MarkerManager.prototype.getTilePoint_ = function(latlng, zoom, padding) {
    var pixelPoint = this.projection_.fromLatLngToPixel(latlng, zoom);
    return new GPoint(
      Math.floor((pixelPoint.x + padding.width) / this.tileSize_),
      Math.floor((pixelPoint.y + padding.height) / this.tileSize_));
};


/**
* Finds the appropriate place to add the marker to the grid.
* Optimized for speed; does not actually add the marker to the map.
* Designed for batch-processing thousands of markers.
*
* @param {Marker} marker The marker to add.
* @param {Number} minZoom The minimum zoom for displaying the marker.
* @param {Number} maxZoom The maximum zoom for displaying the marker.
*/
MarkerManager.prototype.addMarkerBatch_ = function(marker, minZoom, maxZoom) {
    var mPoint = marker.getPoint();
    marker.MarkerManager_minZoom = minZoom;
    // Tracking markers is expensive, so we do this only if the
    // user explicitly requested it when creating marker manager.
    if (this.trackMarkers_) {
        GEvent.bind(marker, "changed", this, this.onMarkerMoved_);
    }

    var gridPoint = this.getTilePoint_(mPoint, maxZoom, GSize.ZERO);

    for (var zoom = maxZoom; zoom >= minZoom; zoom--) {
        var cell = this.getGridCellCreate_(gridPoint.x, gridPoint.y, zoom);
        cell.push(marker);

        gridPoint.x = gridPoint.x >> 1;
        gridPoint.y = gridPoint.y >> 1;
    }
};


/**
* Returns whether or not the given point is visible in the shown bounds. This
* is a helper method that takes care of the corner case, when shownBounds have
* negative minX value.
*
* @param {Point} point a point on a grid.
* @return {Boolean} Whether or not the given point is visible in the currently
* shown bounds.
*/
MarkerManager.prototype.isGridPointVisible_ = function(point) {
    var me = this;
    var vertical = me.shownBounds_.minY <= point.y &&
      point.y <= me.shownBounds_.maxY;
    var minX = me.shownBounds_.minX;
    var horizontal = minX <= point.x && point.x <= me.shownBounds_.maxX;
    if (!horizontal && minX < 0) {
        // Shifts the negative part of the rectangle. As point.x is always less
        // than grid width, only test shifted minX .. 0 part of the shown bounds.
        var width = me.gridWidth_[me.shownBounds_.z];
        horizontal = minX + width <= point.x && point.x <= width - 1;
    }
    return vertical && horizontal;
};


/**
* Reacts to a notification from a marker that it has moved to a new location.
* It scans the grid all all zoom levels and moves the marker from the old grid
* location to a new grid location.
*
* @param {Marker} marker The marker that moved.
* @param {LatLng} oldPoint The old position of the marker.
* @param {LatLng} newPoint The new position of the marker.
*/
MarkerManager.prototype.onMarkerMoved_ = function(marker, oldPoint, newPoint) {
    // NOTE: We do not know the minimum or maximum zoom the marker was
    // added at, so we start at the absolute maximum. Whenever we successfully
    // remove a marker at a given zoom, we add it at the new grid coordinates.
    var me = this;
    var zoom = me.maxZoom_;
    var changed = false;
    var oldGrid = me.getTilePoint_(oldPoint, zoom, GSize.ZERO);
    var newGrid = me.getTilePoint_(newPoint, zoom, GSize.ZERO);
    while (zoom >= 0 && (oldGrid.x !== newGrid.x || oldGrid.y !== newGrid.y)) {
        var cell = me.getGridCellNoCreate_(oldGrid.x, oldGrid.y, zoom);
        if (cell) {
            if (me.removeFromArray_(cell, marker)) {
                me.getGridCellCreate_(newGrid.x, newGrid.y, zoom).push(marker);
            }
        }
        // For the current zoom we also need to update the map. Markers that no
        // longer are visible are removed from the map. Markers that moved into
        // the shown bounds are added to the map. This also lets us keep the count
        // of visible markers up to date.
        if (zoom === me.mapZoom_) {
            if (me.isGridPointVisible_(oldGrid)) {
                if (!me.isGridPointVisible_(newGrid)) {
                    me.removeOverlay_(marker);
                    changed = true;
                }
            } else {
                if (me.isGridPointVisible_(newGrid)) {
                    me.addOverlay_(marker);
                    changed = true;
                }
            }
        }
        oldGrid.x = oldGrid.x >> 1;
        oldGrid.y = oldGrid.y >> 1;
        newGrid.x = newGrid.x >> 1;
        newGrid.y = newGrid.y >> 1;
        --zoom;
    }
    if (changed) {
        me.notifyListeners_();
    }
};


/**
* Removes marker from the manager and from the map
* (if it's currently visible).
* @param {GMarker} marker The marker to delete.
*/
MarkerManager.prototype.removeMarker = function(marker) {
    var me = this;
    var zoom = me.maxZoom_;
    var changed = false;
    var point = marker.getPoint();
    var grid = me.getTilePoint_(point, zoom, GSize.ZERO);
    while (zoom >= 0) {
        var cell = me.getGridCellNoCreate_(grid.x, grid.y, zoom);

        if (cell) {
            me.removeFromArray_(cell, marker);
        }
        // For the current zoom we also need to update the map. Markers that no
        // longer are visible are removed from the map. This also lets us keep the count
        // of visible markers up to date.
        if (zoom === me.mapZoom_) {
            if (me.isGridPointVisible_(grid)) {
                me.removeOverlay_(marker);
                changed = true;
            }
        }
        grid.x = grid.x >> 1;
        grid.y = grid.y >> 1;
        --zoom;
    }
    if (changed) {
        me.notifyListeners_();
    }
    me.numMarkers_[marker.MarkerManager_minZoom]--;
};


/**
* Add many markers at once.
* Does not actually update the map, just the internal grid.
*
* @param {Array of Marker} markers The markers to add.
* @param {Number} minZoom The minimum zoom level to display the markers.
* @param {Number} opt_maxZoom The maximum zoom level to display the markers.
*/
MarkerManager.prototype.addMarkers = function(markers, minZoom, opt_maxZoom) {
    var maxZoom = this.getOptMaxZoom_(opt_maxZoom);
    for (var i = markers.length - 1; i >= 0; i--) {
        this.addMarkerBatch_(markers[i], minZoom, maxZoom);
    }

    this.numMarkers_[minZoom] += markers.length;
};


/**
* Returns the value of the optional maximum zoom. This method is defined so
* that we have just one place where optional maximum zoom is calculated.
*
* @param {Number} opt_maxZoom The optinal maximum zoom.
* @return The maximum zoom.
*/
MarkerManager.prototype.getOptMaxZoom_ = function(opt_maxZoom) {
    return opt_maxZoom || this.maxZoom_;
};


/**
* Calculates the total number of markers potentially visible at a given
* zoom level.
*
* @param {Number} zoom The zoom level to check.
*/
MarkerManager.prototype.getMarkerCount = function(zoom) {
    var total = 0;
    for (var z = 0; z <= zoom; z++) {
        total += this.numMarkers_[z];
    }
    return total;
};

/** 
* Returns a marker given latitude, longitude and zoom. If the marker does not 
* exist, the method will return a new marker. If a new marker is created, 
* it will NOT be added to the manager. 
* 
* @param {Number} lat - the latitude of a marker. 
* @param {Number} lng - the longitude of a marker. 
* @param {Number} zoom - the zoom level 
* @return {GMarker} marker - the marker found at lat and lng 
*/
MarkerManager.prototype.getMarker = function(lat, lng, zoom) {
    var me = this;
    var mPoint = new GLatLng(lat, lng);
    var gridPoint = me.getTilePoint_(mPoint, zoom, GSize.ZERO);

    var marker = new GMarker(mPoint);
    var cellArray = me.getGridCellNoCreate_(gridPoint.x, gridPoint.y, zoom);
    if (cellArray != undefined) {
        for (var i = 0; i < cellArray.length; i++) {
            if (lat == cellArray[i].getLatLng().lat() &&
         lng == cellArray[i].getLatLng().lng()) {
                marker = cellArray[i];
            }
        }
    }
    return marker;
};

/**
* Add a single marker to the map.
*
* @param {Marker} marker The marker to add.
* @param {Number} minZoom The minimum zoom level to display the marker.
* @param {Number} opt_maxZoom The maximum zoom level to display the marker.
*/
MarkerManager.prototype.addMarker = function(marker, minZoom, opt_maxZoom) {
    var me = this;
    var maxZoom = this.getOptMaxZoom_(opt_maxZoom);
    me.addMarkerBatch_(marker, minZoom, maxZoom);
    var gridPoint = me.getTilePoint_(marker.getPoint(), me.mapZoom_, GSize.ZERO);
    if (me.isGridPointVisible_(gridPoint) &&
      minZoom <= me.shownBounds_.z &&
      me.shownBounds_.z <= maxZoom) {
        me.addOverlay_(marker);
        me.notifyListeners_();
    }
    this.numMarkers_[minZoom]++;
};

/**
* Returns true if this bounds (inclusively) contains the given point.
* @param {Point} point  The point to test.
* @return {Boolean} This Bounds contains the given Point.
*/
GBounds.prototype.containsPoint = function(point) {
    var outer = this;
    return (outer.minX <= point.x &&
          outer.maxX >= point.x &&
          outer.minY <= point.y &&
          outer.maxY >= point.y);
};

/**
* Get a cell in the grid, creating it first if necessary.
*
* Optimization candidate
*
* @param {Number} x The x coordinate of the cell.
* @param {Number} y The y coordinate of the cell.
* @param {Number} z The z coordinate of the cell.
* @return {Array} The cell in the array.
*/
MarkerManager.prototype.getGridCellCreate_ = function(x, y, z) {
    var grid = this.grid_[z];
    if (x < 0) {
        x += this.gridWidth_[z];
    }
    var gridCol = grid[x];
    if (!gridCol) {
        gridCol = grid[x] = [];
        return (gridCol[y] = []);
    }
    var gridCell = gridCol[y];
    if (!gridCell) {
        return (gridCol[y] = []);
    }
    return gridCell;
};


/**
* Get a cell in the grid, returning undefined if it does not exist.
*
* NOTE: Optimized for speed -- otherwise could combine with getGridCellCreate_.
*
* @param {Number} x The x coordinate of the cell.
* @param {Number} y The y coordinate of the cell.
* @param {Number} z The z coordinate of the cell.
* @return {Array} The cell in the array.
*/
MarkerManager.prototype.getGridCellNoCreate_ = function(x, y, z) {
    var grid = this.grid_[z];
    if (x < 0) {
        x += this.gridWidth_[z];
    }
    var gridCol = grid[x];
    return gridCol ? gridCol[y] : undefined;
};


/**
* Turns at geographical bounds into a grid-space bounds.
*
* @param {LatLngBounds} bounds The geographical bounds.
* @param {Number} zoom The zoom level of the bounds.
* @param {GSize} swPadding The padding in pixels to extend beyond the
* given bounds.
* @param {GSize} nePadding The padding in pixels to extend beyond the
* given bounds.
* @return {GBounds} The bounds in grid space.
*/
MarkerManager.prototype.getGridBounds_ = function(bounds, zoom, swPadding, nePadding) {
    zoom = Math.min(zoom, this.maxZoom_);

    var bl = bounds.getSouthWest();
    var tr = bounds.getNorthEast();
    var sw = this.getTilePoint_(bl, zoom, swPadding);
    var ne = this.getTilePoint_(tr, zoom, nePadding);
    var gw = this.gridWidth_[zoom];

    // Crossing the prime meridian requires correction of bounds.
    if (tr.lng() < bl.lng() || ne.x < sw.x) {
        sw.x -= gw;
    }
    if (ne.x - sw.x + 1 >= gw) {
        // Computed grid bounds are larger than the world; truncate.
        sw.x = 0;
        ne.x = gw - 1;
    }
    var gridBounds = new GBounds([sw, ne]);
    gridBounds.z = zoom;
    return gridBounds;
};


/**
* Gets the grid-space bounds for the current map viewport.
*
* @return {Bounds} The bounds in grid space.
*/
MarkerManager.prototype.getMapGridBounds_ = function() {
    var me = this;
    return me.getGridBounds_(me.map_.getBounds(), me.mapZoom_, me.swPadding_, me.nePadding_);
};


/**
* Event listener for map:movend.
* NOTE: Use a timeout so that the user is not blocked
* from moving the map.
*
*/
MarkerManager.prototype.onMapMoveEnd_ = function() {
    var me = this;
    me.objectSetTimeout_(this, this.updateMarkers_, 0);
};


/**
* Call a function or evaluate an expression after a specified number of
* milliseconds.
*
* Equivalent to the standard window.setTimeout function, but the given
* function executes as a method of this instance. So the function passed to
* objectSetTimeout can contain references to this.
*    objectSetTimeout(this, function () { alert(this.x) }, 1000);
*
* @param {Object} object  The target object.
* @param {Function} command  The command to run.
* @param {Number} milliseconds  The delay.
* @return {Boolean}  Success.
*/
MarkerManager.prototype.objectSetTimeout_ = function(object, command, milliseconds) {
    return window.setTimeout(function() {
        command.call(object);
    }, milliseconds);
};


/**
* Is this layer visible?
*
* Returns visibility setting
*
* @return {Boolean} Visible
*/
MarkerManager.prototype.visible = function() {
    return this.show_ ? true : false;
};


/**
* Returns true if the manager is hidden.
* Otherwise returns false.
* @return {Boolean} Hidden
*/
MarkerManager.prototype.isHidden = function() {
    return !this.show_;
};


/**
* Shows the manager if it's currently hidden.
*/
MarkerManager.prototype.show = function() {
    this.show_ = true;
    this.refresh();
};


/**
* Hides the manager if it's currently visible
*/
MarkerManager.prototype.hide = function() {
    this.show_ = false;
    this.refresh();
};


/**
* Toggles the visibility of the manager.
*/
MarkerManager.prototype.toggle = function() {
    this.show_ = !this.show_;
    this.refresh();
};


/**
* Refresh forces the marker-manager into a good state.
* <ol>
*   <li>If never before initialized, shows all the markers.</li>
*   <li>If previously initialized, removes and re-adds all markers.</li>
* </ol>
*/
MarkerManager.prototype.refresh = function() {
    var me = this;
    if (me.shownMarkers_ > 0) {
        me.processAll_(me.shownBounds_, me.removeOverlay_);
    }
    // An extra check on me.show_ to increase performance (no need to processAll_)
    if (me.show_) {
        me.processAll_(me.shownBounds_, me.addOverlay_);
    }
    me.notifyListeners_();
};


/**
* After the viewport may have changed, add or remove markers as needed.
*/
MarkerManager.prototype.updateMarkers_ = function() {
    var me = this;
    me.mapZoom_ = this.map_.getZoom();
    var newBounds = me.getMapGridBounds_();

    // If the move does not include new grid sections,
    // we have no work to do:
    if (newBounds.equals(me.shownBounds_) && newBounds.z === me.shownBounds_.z) {
        return;
    }

    if (newBounds.z !== me.shownBounds_.z) {
        me.processAll_(me.shownBounds_, me.removeOverlay_);
        if (me.show_) { // performance
            me.processAll_(newBounds, me.addOverlay_);
        }
    } else {
        // Remove markers:
        me.rectangleDiff_(me.shownBounds_, newBounds, me.removeCellMarkers_);

        // Add markers:
        if (me.show_) { // performance
            me.rectangleDiff_(newBounds, me.shownBounds_, me.addCellMarkers_);
        }
    }
    me.shownBounds_ = newBounds;

    me.notifyListeners_();
};


/**
* Notify listeners when the state of what is displayed changes.
*/
MarkerManager.prototype.notifyListeners_ = function() {
    GEvent.trigger(this, "changed", this.shownBounds_, this.shownMarkers_);
};


/**
* Process all markers in the bounds provided, using a callback.
*
* @param {Bounds} bounds The bounds in grid space.
* @param {Function} callback The function to call for each marker.
*/
MarkerManager.prototype.processAll_ = function(bounds, callback) {
    for (var x = bounds.minX; x <= bounds.maxX; x++) {
        for (var y = bounds.minY; y <= bounds.maxY; y++) {
            this.processCellMarkers_(x, y, bounds.z, callback);
        }
    }
};


/**
* Process all markers in the grid cell, using a callback.
*
* @param {Number} x The x coordinate of the cell.
* @param {Number} y The y coordinate of the cell.
* @param {Number} z The z coordinate of the cell.
* @param {Function} callback The function to call for each marker.
*/
MarkerManager.prototype.processCellMarkers_ = function(x, y, z, callback) {
    var cell = this.getGridCellNoCreate_(x, y, z);
    if (cell) {
        for (var i = cell.length - 1; i >= 0; i--) {
            callback(cell[i]);
        }
    }
};


/**
* Remove all markers in a grid cell.
*
* @param {Number} x The x coordinate of the cell.
* @param {Number} y The y coordinate of the cell.
* @param {Number} z The z coordinate of the cell.
*/
MarkerManager.prototype.removeCellMarkers_ = function(x, y, z) {
    this.processCellMarkers_(x, y, z, this.removeOverlay_);
};


/**
* Add all markers in a grid cell.
*
* @param {Number} x The x coordinate of the cell.
* @param {Number} y The y coordinate of the cell.
* @param {Number} z The z coordinate of the cell.
*/
MarkerManager.prototype.addCellMarkers_ = function(x, y, z) {
    this.processCellMarkers_(x, y, z, this.addOverlay_);
};


/**
* Use the rectangleDiffCoords_ function to process all grid cells
* that are in bounds1 but not bounds2, using a callback, and using
* the current MarkerManager object as the instance.
*
* Pass the z parameter to the callback in addition to x and y.
*
* @param {Bounds} bounds1 The bounds of all points we may process.
* @param {Bounds} bounds2 The bounds of points to exclude.
* @param {Function} callback The callback function to call
*                   for each grid coordinate (x, y, z).
*/
MarkerManager.prototype.rectangleDiff_ = function(bounds1, bounds2, callback) {
    var me = this;
    me.rectangleDiffCoords_(bounds1, bounds2, function(x, y) {
        callback.apply(me, [x, y, bounds1.z]);
    });
};


/**
* Calls the function for all points in bounds1, not in bounds2
*
* @param {Bounds} bounds1 The bounds of all points we may process.
* @param {Bounds} bounds2 The bounds of points to exclude.
* @param {Function} callback The callback function to call
*                   for each grid coordinate.
*/
MarkerManager.prototype.rectangleDiffCoords_ = function(bounds1, bounds2, callback) {
    var minX1 = bounds1.minX;
    var minY1 = bounds1.minY;
    var maxX1 = bounds1.maxX;
    var maxY1 = bounds1.maxY;
    var minX2 = bounds2.minX;
    var minY2 = bounds2.minY;
    var maxX2 = bounds2.maxX;
    var maxY2 = bounds2.maxY;

    var x, y;
    for (x = minX1; x <= maxX1; x++) {  // All x in R1
        // All above:
        for (y = minY1; y <= maxY1 && y < minY2; y++) {  // y in R1 above R2
            callback(x, y);
        }
        // All below:
        for (y = Math.max(maxY2 + 1, minY1);  // y in R1 below R2
         y <= maxY1; y++) {
            callback(x, y);
        }
    }

    for (y = Math.max(minY1, minY2);
       y <= Math.min(maxY1, maxY2); y++) {  // All y in R2 and in R1
        // Strictly left:
        for (x = Math.min(maxX1 + 1, minX2) - 1;
         x >= minX1; x--) {  // x in R1 left of R2
            callback(x, y);
        }
        // Strictly right:
        for (x = Math.max(minX1, maxX2 + 1);  // x in R1 right of R2
         x <= maxX1; x++) {
            callback(x, y);
        }
    }
};


/**
* Removes value from array. O(N).
*
* @param {Array} array  The array to modify.
* @param {any} value  The value to remove.
* @param {Boolean} opt_notype  Flag to disable type checking in equality.
* @return {Number}  The number of instances of value that were removed.
*/
MarkerManager.prototype.removeFromArray_ = function(array, value, opt_notype) {
    var shift = 0;
    for (var i = 0; i < array.length; ++i) {
        if (array[i] === value || (opt_notype && array[i] === value)) {
            array.splice(i--, 1);
            shift++;
        }
    }
    return shift;
};
