Make ol.Map.isDef stricter

This commit is contained in:
Tom Payne
2012-07-29 14:17:25 +02:00
parent adfbf2ae7b
commit a8664df76b
4 changed files with 9 additions and 7 deletions

View File

@@ -671,8 +671,9 @@ ol.Map.prototype.handleViewportResize = function() {
* @return {boolean} Is defined.
*/
ol.Map.prototype.isDef = function() {
return goog.isDef(this.getCenter()) && goog.isDef(this.getResolution()) &&
goog.isDef(this.getSize());
return goog.isDefAndNotNull(this.getCenter()) &&
goog.isDef(this.getResolution()) &&
goog.isDefAndNotNull(this.getSize());
};
@@ -845,10 +846,10 @@ ol.Map.prototype.updateMatrices_ = function() {
if (this.matriciesDirty_) {
var center = /** @type {ol.Coordinate} */ this.getCenter();
var center = /** @type {!ol.Coordinate} */ this.getCenter();
var resolution = /** @type {number} */ this.getResolution();
var rotation = /** @type {number} */ this.getRotation();
var size = /** @type {ol.Size} */ this.getSize();
var rotation = this.getRotation();
var size = /** @type {!ol.Size} */ this.getSize();
goog.vec.Mat4.makeIdentity(this.coordinateToPixelMatrix_);
goog.vec.Mat4.translate(this.coordinateToPixelMatrix_,

View File

@@ -108,7 +108,7 @@ ol.control.Drag.prototype.handleMapBrowserEvent = function(mapBrowserEvent) {
this.startY = browserEvent.clientY;
this.deltaX = 0;
this.deltaY = 0;
this.startCenter = /** @type {ol.Coordinate} */ map.getCenter();
this.startCenter = /** @type {!ol.Coordinate} */ map.getCenter();
this.startCoordinate = /** @type {ol.Coordinate} */
mapBrowserEvent.getCoordinate();
if (this.handleDragStart(mapBrowserEvent)) {

View File

@@ -53,7 +53,7 @@ ol.dom.TileLayerRenderer.prototype.render = function() {
if (!map.isDef()) {
return;
}
var mapExtent = /** @type {ol.Extent} */ map.getExtent();
var mapExtent = /** @type {!ol.Extent} */ map.getExtent();
var mapResolution = /** @type {number} */ map.getResolution();
var tileLayer = this.getLayer();

View File

@@ -263,6 +263,7 @@ ol.webgl.TileLayerRenderer.prototype.render = function() {
var gl = this.getGL();
var map = this.getMap();
goog.asserts.assert(map.isDef());
var mapExtent = /** @type {!ol.Extent} */ map.getExtent();
var mapResolution = /** @type {number} */ map.getResolution();
var mapSize = map.getSize();