From f5978c659cae4543889dafb4f186d9547f4c53f8 Mon Sep 17 00:00:00 2001 From: GaborFarkas Date: Mon, 30 May 2016 10:44:25 +0200 Subject: [PATCH] Adding some defaults --- src/ol/render/webgl/imagereplay/index.js | 13 +++++++------ src/ol/render/webgl/immediate.js | 1 + src/ol/render/webgl/webgl.js | 13 +++++++++++++ src/ol/render/webgl/webgl.jsdoc | 3 +++ 4 files changed, 24 insertions(+), 6 deletions(-) create mode 100644 src/ol/render/webgl/webgl.js create mode 100644 src/ol/render/webgl/webgl.jsdoc diff --git a/src/ol/render/webgl/imagereplay/index.js b/src/ol/render/webgl/imagereplay/index.js index b3e3c0133e..c6d13694f6 100644 --- a/src/ol/render/webgl/imagereplay/index.js +++ b/src/ol/render/webgl/imagereplay/index.js @@ -21,6 +21,7 @@ goog.require('ol.vec.Mat4'); goog.require('ol.webgl'); goog.require('ol.webgl.Buffer'); goog.require('ol.webgl.Context'); +goog.require('ol.render.webgl'); /** @@ -926,9 +927,9 @@ ol.render.webgl.LineStringReplay = function(tolerance, maxExtent) { /** * @private - * @type {ol.Color} + * @type {ol.Color|undefined} */ - this.strokeColor_ = null; + this.strokeColor_ = undefined; /** @@ -1161,14 +1162,14 @@ ol.render.webgl.LineStringReplay.prototype.drawReplay_ = */ ol.render.webgl.LineStringReplay.prototype.setFillStrokeStyle = function(fillStyle, strokeStyle) { - if (!goog.isNull(strokeStyle)) { + if (strokeStyle) { var strokeStyleColor = strokeStyle.getColor(); this.strokeColor_ = !goog.isNull(strokeStyleColor) ? ol.color.asArray(strokeStyleColor).map(function(c, i) { return i != 3 ? c / 255 : c; - }) : [0.0, 0.0, 0.0, 1.0]; + }) : ol.render.webgl.defaultStrokeStyle; } else { - this.strokeColor_ = null; + this.strokeColor_ = undefined; } }; @@ -1509,7 +1510,7 @@ ol.render.webgl.PolygonReplay.prototype.setFillStrokeStyle = this.fillColor_ = !goog.isNull(fillStyleColor) && Array.isArray(fillStyleColor) ? ol.color.asArray(fillStyleColor).map(function(c, i) { return i != 3 ? c / 255.0 : c; - }) : [0.0, 0.0, 0.0, 1.0]; + }) : ol.render.webgl.defaultFillStyle; } else { this.fillColor_ = undefined; } diff --git a/src/ol/render/webgl/immediate.js b/src/ol/render/webgl/immediate.js index 2d52703d85..b7cddd0da2 100644 --- a/src/ol/render/webgl/immediate.js +++ b/src/ol/render/webgl/immediate.js @@ -6,6 +6,7 @@ goog.require('ol.geom.GeometryType'); goog.require('ol.render.ReplayType'); goog.require('ol.render.VectorContext'); goog.require('ol.render.webgl.ReplayGroup'); +goog.require('ol.render.webgl'); /** diff --git a/src/ol/render/webgl/webgl.js b/src/ol/render/webgl/webgl.js new file mode 100644 index 0000000000..4fb7bfe513 --- /dev/null +++ b/src/ol/render/webgl/webgl.js @@ -0,0 +1,13 @@ +goog.provide('ol.render.webgl'); + +/** + * @const + * @type {ol.Color} + */ +ol.render.webgl.defaultFillStyle = [0.0, 0.0, 0.0, 1.0]; + +/** + * @const + * @type {ol.Color} + */ +ol.render.webgl.defaultStrokeStyle = [0.0, 0.0, 0.0, 1.0]; diff --git a/src/ol/render/webgl/webgl.jsdoc b/src/ol/render/webgl/webgl.jsdoc new file mode 100644 index 0000000000..2ac3cda5ae --- /dev/null +++ b/src/ol/render/webgl/webgl.jsdoc @@ -0,0 +1,3 @@ +/** + * @namespace ol.render.webgl + */