Merge pull request #5888 from fredj/rm_unused

Remove unused functions

src/ol/color.js

@@ -189,18 +189,6 @@ ol.color.fromStringInternal_ = function(s) {
 };
 
 
-/**
- * @param {ol.Color} color Color.
- * @return {boolean} Is valid.
- */
-ol.color.isValid = function(color) {
-  return 0 <= color[0] && color[0] < 256 &&
-      0 <= color[1] && color[1] < 256 &&
-      0 <= color[2] && color[2] < 256 &&
-      0 <= color[3] && color[3] <= 1;
-};
-
-
 /**
  * @param {ol.Color} color Color.
  * @param {ol.Color=} opt_color Color.

src/ol/tilecoord.js

@@ -1,22 +1,5 @@
 goog.provide('ol.tilecoord');
 
-goog.require('ol');
-
-
-/**
- * @param {string} str String that follows pattern “z/x/y” where x, y and z are
- *   numbers.
- * @return {ol.TileCoord} Tile coord.
- */
-ol.tilecoord.createFromString = function(str) {
-  var v = str.split('/');
-  ol.DEBUG && console.assert(v.length === 3,
-      'must provide a string in "z/x/y" format, got "%s"', str);
-  return v.map(function(e) {
-    return parseInt(e, 10);
-  });
-};
-
 
 /**
  * @param {number} z Z.

test/spec/ol/color.test.js

@@ -103,28 +103,6 @@ describe('ol.color', function() {
 
   });
 
-  describe('ol.color.isValid', function() {
-
-    it('identifies valid colors', function() {
-      expect(ol.color.isValid([0, 0, 0, 0])).to.be(true);
-      expect(ol.color.isValid([255, 255, 255, 1])).to.be(true);
-    });
-
-    it('identifies out-of-range channels', function() {
-      expect(ol.color.isValid([-1, 0, 0, 0])).to.be(false);
-      expect(ol.color.isValid([256, 0, 0, 0])).to.be(false);
-      expect(ol.color.isValid([0, -1, 0, 0])).to.be(false);
-      expect(ol.color.isValid([0, 256, 0, 0])).to.be(false);
-      expect(ol.color.isValid([0, 0, -1, 0])).to.be(false);
-      expect(ol.color.isValid([0, 0, 256, 0])).to.be(false);
-      expect(ol.color.isValid([0, 0, -1, 0])).to.be(false);
-      expect(ol.color.isValid([0, 0, 256, 0])).to.be(false);
-      expect(ol.color.isValid([0, 0, 0, -1])).to.be(false);
-      expect(ol.color.isValid([0, 0, 0, 2])).to.be(false);
-    });
-
-  });
-
   describe('ol.color.normalize', function() {
 
     it('clamps out-of-range channels', function() {

test/spec/ol/tilecoord.test.js

@@ -15,16 +15,6 @@ describe('ol.TileCoord', function() {
     });
   });
 
-  describe('create from string', function() {
-    it('sets x y z properties as expected', function() {
-      var str = '1/2/3';
-      var tileCoord = ol.tilecoord.createFromString(str);
-      expect(tileCoord[0]).to.eql(1);
-      expect(tileCoord[1]).to.eql(2);
-      expect(tileCoord[2]).to.eql(3);
-    });
-  });
-
   describe('call quadKey', function() {
     it('returns expected string', function() {
       var tileCoord = [3, 3, 5];