Merge pull request #5165 from fredj/rm_goog.addSingletonGetter

Remove goog.addSingletonGetter function

src/ol/render/webgl/imagereplay/defaultshader.js

@@ -14,7 +14,6 @@ ol.render.webgl.imagereplay.defaultshader.Fragment = function() {
   ol.webgl.Fragment.call(this, ol.render.webgl.imagereplay.defaultshader.Fragment.SOURCE);
 };
 ol.inherits(ol.render.webgl.imagereplay.defaultshader.Fragment, ol.webgl.Fragment);
-goog.addSingletonGetter(ol.render.webgl.imagereplay.defaultshader.Fragment);
 
 
 /**
@@ -40,6 +39,9 @@ ol.render.webgl.imagereplay.defaultshader.Fragment.SOURCE = goog.DEBUG ?
     ol.render.webgl.imagereplay.defaultshader.Fragment.OPTIMIZED_SOURCE;
 
 
+ol.render.webgl.imagereplay.defaultshader.fragment = new ol.render.webgl.imagereplay.defaultshader.Fragment();
+
+
 /**
  * @constructor
  * @extends {ol.webgl.Vertex}
@@ -49,7 +51,6 @@ ol.render.webgl.imagereplay.defaultshader.Vertex = function() {
   ol.webgl.Vertex.call(this, ol.render.webgl.imagereplay.defaultshader.Vertex.SOURCE);
 };
 ol.inherits(ol.render.webgl.imagereplay.defaultshader.Vertex, ol.webgl.Vertex);
-goog.addSingletonGetter(ol.render.webgl.imagereplay.defaultshader.Vertex);
 
 
 /**
@@ -75,6 +76,9 @@ ol.render.webgl.imagereplay.defaultshader.Vertex.SOURCE = goog.DEBUG ?
     ol.render.webgl.imagereplay.defaultshader.Vertex.OPTIMIZED_SOURCE;
 
 
+ol.render.webgl.imagereplay.defaultshader.vertex = new ol.render.webgl.imagereplay.defaultshader.Vertex();
+
+
 /**
  * @constructor
  * @param {WebGLRenderingContext} gl GL.

src/ol/render/webgl/imagereplay/index.js

@@ -515,10 +515,8 @@ ol.render.webgl.ImageReplay.prototype.replay = function(context,
   context.bindBuffer(ol.webgl.ELEMENT_ARRAY_BUFFER, this.indicesBuffer_);
 
   // get the program
-  var fragmentShader =
+  var fragmentShader = ol.render.webgl.imagereplay.defaultshader.fragment;
-      ol.render.webgl.imagereplay.defaultshader.Fragment.getInstance();
+  var vertexShader = ol.render.webgl.imagereplay.defaultshader.vertex;
-  var vertexShader =
-      ol.render.webgl.imagereplay.defaultshader.Vertex.getInstance();
   var program = context.getProgram(fragmentShader, vertexShader);
 
   // get the locations

src/ol/renderer/map.js

@@ -9,7 +9,7 @@ goog.require('ol.events.EventType');
 goog.require('ol.extent');
 goog.require('ol.functions');
 goog.require('ol.layer.Layer');
-goog.require('ol.style.IconImageCache');
+goog.require('ol.style.iconImageCache');
 
 
 /**
@@ -104,7 +104,7 @@ ol.renderer.Map.prototype.disposeInternal = function() {
  * @private
  */
 ol.renderer.Map.expireIconCache_ = function(map, frameState) {
-  ol.style.IconImageCache.getInstance().expire();
+  ol.style.iconImageCache.expire();
 };
 
 

src/ol/renderer/webgl/defaultmapshader.js

@@ -14,7 +14,6 @@ ol.renderer.webgl.defaultmapshader.Fragment = function() {
   ol.webgl.Fragment.call(this, ol.renderer.webgl.defaultmapshader.Fragment.SOURCE);
 };
 ol.inherits(ol.renderer.webgl.defaultmapshader.Fragment, ol.webgl.Fragment);
-goog.addSingletonGetter(ol.renderer.webgl.defaultmapshader.Fragment);
 
 
 /**
@@ -40,6 +39,9 @@ ol.renderer.webgl.defaultmapshader.Fragment.SOURCE = goog.DEBUG ?
     ol.renderer.webgl.defaultmapshader.Fragment.OPTIMIZED_SOURCE;
 
 
+ol.renderer.webgl.defaultmapshader.fragment = new ol.renderer.webgl.defaultmapshader.Fragment();
+
+
 /**
  * @constructor
  * @extends {ol.webgl.Vertex}
@@ -49,7 +51,6 @@ ol.renderer.webgl.defaultmapshader.Vertex = function() {
   ol.webgl.Vertex.call(this, ol.renderer.webgl.defaultmapshader.Vertex.SOURCE);
 };
 ol.inherits(ol.renderer.webgl.defaultmapshader.Vertex, ol.webgl.Vertex);
-goog.addSingletonGetter(ol.renderer.webgl.defaultmapshader.Vertex);
 
 
 /**
@@ -75,6 +76,9 @@ ol.renderer.webgl.defaultmapshader.Vertex.SOURCE = goog.DEBUG ?
     ol.renderer.webgl.defaultmapshader.Vertex.OPTIMIZED_SOURCE;
 
 
+ol.renderer.webgl.defaultmapshader.vertex = new ol.renderer.webgl.defaultmapshader.Vertex();
+
+
 /**
  * @constructor
  * @param {WebGLRenderingContext} gl GL.

src/ol/renderer/webgl/layer.js

@@ -145,8 +145,8 @@ ol.renderer.webgl.Layer.prototype.composeFrame = function(frameState, layerState
 
   var gl = context.getGL();
 
-  var fragmentShader = ol.renderer.webgl.defaultmapshader.Fragment.getInstance();
+  var fragmentShader = ol.renderer.webgl.defaultmapshader.fragment;
-  var vertexShader = ol.renderer.webgl.defaultmapshader.Vertex.getInstance();
+  var vertexShader = ol.renderer.webgl.defaultmapshader.vertex;
 
   var program = context.getProgram(fragmentShader, vertexShader);
 

src/ol/renderer/webgl/tilelayer.js

@@ -30,14 +30,13 @@ ol.renderer.webgl.TileLayer = function(mapRenderer, tileLayer) {
    * @private
    * @type {ol.webgl.Fragment}
    */
-  this.fragmentShader_ =
+  this.fragmentShader_ = ol.renderer.webgl.tilelayershader.fragment;
-      ol.renderer.webgl.tilelayershader.Fragment.getInstance();
 
   /**
    * @private
    * @type {ol.webgl.Vertex}
    */
-  this.vertexShader_ = ol.renderer.webgl.tilelayershader.Vertex.getInstance();
+  this.vertexShader_ = ol.renderer.webgl.tilelayershader.vertex;
 
   /**
    * @private

src/ol/renderer/webgl/tilelayershader.js

@@ -14,7 +14,6 @@ ol.renderer.webgl.tilelayershader.Fragment = function() {
   ol.webgl.Fragment.call(this, ol.renderer.webgl.tilelayershader.Fragment.SOURCE);
 };
 ol.inherits(ol.renderer.webgl.tilelayershader.Fragment, ol.webgl.Fragment);
-goog.addSingletonGetter(ol.renderer.webgl.tilelayershader.Fragment);
 
 
 /**
@@ -40,6 +39,9 @@ ol.renderer.webgl.tilelayershader.Fragment.SOURCE = goog.DEBUG ?
     ol.renderer.webgl.tilelayershader.Fragment.OPTIMIZED_SOURCE;
 
 
+ol.renderer.webgl.tilelayershader.fragment = new ol.renderer.webgl.tilelayershader.Fragment();
+
+
 /**
  * @constructor
  * @extends {ol.webgl.Vertex}
@@ -49,7 +51,6 @@ ol.renderer.webgl.tilelayershader.Vertex = function() {
   ol.webgl.Vertex.call(this, ol.renderer.webgl.tilelayershader.Vertex.SOURCE);
 };
 ol.inherits(ol.renderer.webgl.tilelayershader.Vertex, ol.webgl.Vertex);
-goog.addSingletonGetter(ol.renderer.webgl.tilelayershader.Vertex);
 
 
 /**
@@ -75,6 +76,9 @@ ol.renderer.webgl.tilelayershader.Vertex.SOURCE = goog.DEBUG ?
     ol.renderer.webgl.tilelayershader.Vertex.OPTIMIZED_SOURCE;
 
 
+ol.renderer.webgl.tilelayershader.vertex = new ol.renderer.webgl.tilelayershader.Vertex();
+
+
 /**
  * @constructor
  * @param {WebGLRenderingContext} gl GL.

src/ol/style/icon.js

@@ -1,7 +1,7 @@
 goog.provide('ol.style.Icon');
 goog.provide('ol.style.IconAnchorUnits');
-goog.provide('ol.style.IconImageCache');
 goog.provide('ol.style.IconOrigin');
+goog.provide('ol.style.iconImageCache');
 
 goog.require('ol.events');
 goog.require('ol.events.EventTarget');
@@ -458,7 +458,7 @@ ol.inherits(ol.style.IconImage_, ol.events.EventTarget);
  */
 ol.style.IconImage_.get = function(image, src, size, crossOrigin, imageState,
                                    color) {
-  var iconImageCache = ol.style.IconImageCache.getInstance();
+  var iconImageCache = ol.style.iconImageCache;
   var iconImage = iconImageCache.get(src, crossOrigin, color);
   if (!iconImage) {
     iconImage = new ol.style.IconImage_(
@@ -638,8 +638,9 @@ ol.style.IconImage_.prototype.unlistenImage_ = function() {
 
 /**
  * @constructor
+ * @private
  */
-ol.style.IconImageCache = function() {
+ol.style.IconImageCache_ = function() {
 
   /**
    * @type {Object.<string, ol.style.IconImage_>}
@@ -660,7 +661,6 @@ ol.style.IconImageCache = function() {
    */
   this.maxCacheSize_ = 32;
 };
-goog.addSingletonGetter(ol.style.IconImageCache);
 
 
 /**
@@ -669,7 +669,7 @@ goog.addSingletonGetter(ol.style.IconImageCache);
  * @param {ol.Color} color Color.
  * @return {string} Cache key.
  */
-ol.style.IconImageCache.getKey = function(src, crossOrigin, color) {
+ol.style.IconImageCache_.getKey = function(src, crossOrigin, color) {
   goog.DEBUG && console.assert(crossOrigin !== undefined,
       'argument crossOrigin must be defined');
   var colorString = color ? ol.color.asString(color) : 'null';
@@ -680,7 +680,7 @@ ol.style.IconImageCache.getKey = function(src, crossOrigin, color) {
 /**
  * FIXME empty description for jsdoc
  */
-ol.style.IconImageCache.prototype.clear = function() {
+ol.style.IconImageCache_.prototype.clear = function() {
   this.cache_ = {};
   this.cacheSize_ = 0;
 };
@@ -689,7 +689,7 @@ ol.style.IconImageCache.prototype.clear = function() {
 /**
  * FIXME empty description for jsdoc
  */
-ol.style.IconImageCache.prototype.expire = function() {
+ol.style.IconImageCache_.prototype.expire = function() {
   if (this.cacheSize_ > this.maxCacheSize_) {
     var i = 0;
     var key, iconImage;
@@ -710,8 +710,8 @@ ol.style.IconImageCache.prototype.expire = function() {
  * @param {ol.Color} color Color.
  * @return {ol.style.IconImage_} Icon image.
  */
-ol.style.IconImageCache.prototype.get = function(src, crossOrigin, color) {
+ol.style.IconImageCache_.prototype.get = function(src, crossOrigin, color) {
-  var key = ol.style.IconImageCache.getKey(src, crossOrigin, color);
+  var key = ol.style.IconImageCache_.getKey(src, crossOrigin, color);
   return key in this.cache_ ? this.cache_[key] : null;
 };
 
@@ -722,9 +722,12 @@ ol.style.IconImageCache.prototype.get = function(src, crossOrigin, color) {
  * @param {ol.Color} color Color.
  * @param {ol.style.IconImage_} iconImage Icon image.
  */
-ol.style.IconImageCache.prototype.set = function(src, crossOrigin, color,
+ol.style.IconImageCache_.prototype.set = function(src, crossOrigin, color,
                                                  iconImage) {
-  var key = ol.style.IconImageCache.getKey(src, crossOrigin, color);
+  var key = ol.style.IconImageCache_.getKey(src, crossOrigin, color);
   this.cache_[key] = iconImage;
   ++this.cacheSize_;
 };
+
+
+ol.style.iconImageCache = new ol.style.IconImageCache_();

src/ol/webgl/shader.mustache

@@ -14,7 +14,6 @@ goog.require('ol.webgl.Vertex');
   ol.webgl.Fragment.call(this, {{className}}.Fragment.SOURCE);
 };
 ol.inherits({{className}}.Fragment, ol.webgl.Fragment);
-goog.addSingletonGetter({{className}}.Fragment);
 
 
 /**
@@ -40,6 +39,9 @@ goog.addSingletonGetter({{className}}.Fragment);
     {{className}}.Fragment.OPTIMIZED_SOURCE;
 
 
+{{className}}.fragment = new {{className}}.Fragment();
+
+
 /**
  * @constructor
  * @extends {ol.webgl.Vertex}
@@ -49,7 +51,6 @@ goog.addSingletonGetter({{className}}.Fragment);
   ol.webgl.Vertex.call(this, {{className}}.Vertex.SOURCE);
 };
 ol.inherits({{className}}.Vertex, ol.webgl.Vertex);
-goog.addSingletonGetter({{className}}.Vertex);
 
 
 /**
@@ -75,6 +76,9 @@ goog.addSingletonGetter({{className}}.Vertex);
     {{className}}.Vertex.OPTIMIZED_SOURCE;
 
 
+{{className}}.vertex = new {{className}}.Vertex();
+
+
 /**
  * @constructor
  * @param {WebGLRenderingContext} gl GL.

test/spec/ol/style/icon.test.js

@@ -2,7 +2,7 @@ goog.provide('ol.test.style.Icon');
 goog.provide('ol.test.style.IconImageCache');
 
 goog.require('ol.events');
-goog.require('ol.style.IconImageCache');
+goog.require('ol.style.iconImageCache');
 goog.require('ol.style.Icon');
 
 
@@ -145,7 +145,7 @@ describe('ol.style.Icon', function() {
     it('takes the real image size', function() {
       // pretend that the image is already in the cache,
       // this image will be used for the icon.
-      var cache = ol.style.IconImageCache.getInstance();
+      var cache = ol.style.iconImageCache;
       var src = 'test.png';
       var iconImage = new ol.style.IconImage_(null, 'test.png', imgSize);
       cache.set(src, null, null, iconImage);
@@ -170,23 +170,23 @@ describe('ol.style.IconImageCache', function() {
   var originalMaxCacheSize;
 
   beforeEach(function() {
-    var cache = ol.style.IconImageCache.getInstance();
+    var cache = ol.style.iconImageCache;
     cache.clear();
     originalMaxCacheSize = cache.maxCacheSize;
     cache.maxCacheSize_ = 4;
   });
 
   afterEach(function() {
-    var cache = ol.style.IconImageCache.getInstance();
+    var cache = ol.style.iconImageCache;
     cache.maxCacheSize_ = originalMaxCacheSize;
     cache.clear();
   });
 
   describe('#expire', function() {
     it('expires images when expected', function() {
-      var cache = ol.style.IconImageCache.getInstance();
+      var cache = ol.style.iconImageCache;
 
-      var i, src, iconImage, key;
+      var i, src, iconImage;
 
       for (i = 0; i < 4; ++i) {
         src = i + '';
@@ -223,11 +223,8 @@ describe('ol.style.IconImageCache', function() {
 
       // check that '0' and '4' are not removed from the cache
       cache.expire();
-      key = ol.style.IconImageCache.getKey('0', null, null);
+      expect(cache.get('0', null, null)).to.not.be(null);
-      expect(key in cache.cache_).to.be.ok();
+      expect(cache.get('4', null, null)).to.not.be(null);
-      key = ol.style.IconImageCache.getKey('4', null, null);
-      expect(key in cache.cache_).to.be.ok();
-
     });
   });
 });