diff --git a/src/ol/parser/kml.js b/src/ol/parser/kml.js index 16a0d53a36..3de797d27f 100644 --- a/src/ol/parser/kml.js +++ b/src/ol/parser/kml.js @@ -3,6 +3,7 @@ goog.require('goog.array'); goog.require('goog.async.Deferred'); goog.require('goog.async.DeferredList'); goog.require('goog.date'); +goog.require('goog.dispose'); goog.require('goog.dom.xml'); goog.require('goog.events'); goog.require('goog.net.EventType'); @@ -881,7 +882,7 @@ ol.parser.KML.prototype.parseLinks = function(deferreds, obj, done) { goog.events.listen(xhr, goog.net.EventType.COMPLETE, function(e) { if (e.target.isSuccess()) { var data = e.target.getResponseXml(); - e.target.dispose(); + goog.dispose(e.target); if (data && data.nodeType == 9) { data = data.documentElement; } diff --git a/test/spec/ol/control/control.test.js b/test/spec/ol/control/control.test.js index 1f9e1de825..4b4f25bd00 100644 --- a/test/spec/ol/control/control.test.js +++ b/test/spec/ol/control/control.test.js @@ -13,17 +13,18 @@ describe('ol.control.Control', function() { }); afterEach(function() { - map.dispose(); + goog.dispose(map); }); describe('dispose', function() { it('removes the control element from its parent', function() { - control.dispose(); + goog.dispose(control); expect(goog.dom.getParentElement(control.element)).to.be(null); }); }); }); +goog.require('goog.dispose'); goog.require('goog.dom'); goog.require('goog.dom.TagName'); goog.require('ol.Map'); diff --git a/test/spec/ol/control/zoomslidercontrol.test.js b/test/spec/ol/control/zoomslidercontrol.test.js index c1cb4134d6..893d735482 100644 --- a/test/spec/ol/control/zoomslidercontrol.test.js +++ b/test/spec/ol/control/zoomslidercontrol.test.js @@ -15,8 +15,8 @@ describe('ol.control.ZoomSlider', function() { }); afterEach(function() { - zoomslider.dispose(); - map.dispose(); + goog.dispose(zoomslider); + goog.dispose(map); document.body.removeChild(target); zoomslider = null; map = null; @@ -72,7 +72,7 @@ describe('ol.control.ZoomSlider', function() { var horizontal = ol.control.ZoomSlider.direction.HORIZONTAL; expect(control.direction_).to.be(horizontal); - control.dispose(); + goog.dispose(control); }); it('is vertical for tall containers', function() { @@ -85,12 +85,13 @@ describe('ol.control.ZoomSlider', function() { var vertical = ol.control.ZoomSlider.direction.VERTICAL; expect(control.direction_).to.be(vertical); - control.dispose(); + goog.dispose(control); }); }); }); +goog.require('goog.dispose'); goog.require('goog.dom'); goog.require('goog.dom.classes'); goog.require('goog.fx.Dragger'); diff --git a/test/spec/ol/layer/layer.test.js b/test/spec/ol/layer/layer.test.js index 9778950b4c..2476c8e3cb 100644 --- a/test/spec/ol/layer/layer.test.js +++ b/test/spec/ol/layer/layer.test.js @@ -15,7 +15,7 @@ describe('ol.layer.Layer', function() { }); afterEach(function() { - layer.dispose(); + goog.dispose(layer); }); it('creates an instance', function() { @@ -70,7 +70,7 @@ describe('ol.layer.Layer', function() { expect(layer.getSaturation()).to.be(5); expect(layer.getVisible()).to.be(false); - layer.dispose(); + goog.dispose(layer); }); }); @@ -88,7 +88,7 @@ describe('ol.layer.Layer', function() { }); afterEach(function() { - layer.dispose(); + goog.dispose(layer); }); it('accepts a positive number', function() { @@ -126,7 +126,7 @@ describe('ol.layer.Layer', function() { }); afterEach(function() { - layer.dispose(); + goog.dispose(layer); }); it('accepts a small positive number', function() { @@ -160,7 +160,7 @@ describe('ol.layer.Layer', function() { }); afterEach(function() { - layer.dispose(); + goog.dispose(layer); }); it('accepts a small positive number', function() { @@ -199,7 +199,7 @@ describe('ol.layer.Layer', function() { }); afterEach(function() { - layer.dispose(); + goog.dispose(layer); }); it('accepts a positive number', function() { @@ -233,7 +233,7 @@ describe('ol.layer.Layer', function() { }); afterEach(function() { - layer.dispose(); + goog.dispose(layer); }); it('accepts a small positive number', function() { @@ -269,13 +269,14 @@ describe('ol.layer.Layer', function() { layer.setVisible(true); expect(layer.getVisible()).to.be(true); - layer.dispose(); + goog.dispose(layer); }); }); }); +goog.require('goog.dispose'); goog.require('ol.layer.Layer'); goog.require('ol.projection'); goog.require('ol.source.Source'); diff --git a/test/spec/ol/layer/vectorlayer.test.js b/test/spec/ol/layer/vectorlayer.test.js index 598213b2ed..943ffbb059 100644 --- a/test/spec/ol/layer/vectorlayer.test.js +++ b/test/spec/ol/layer/vectorlayer.test.js @@ -170,12 +170,13 @@ describe('ol.layer.Vector', function() { }); - layer.dispose(); + goog.dispose(layer); }); }); +goog.require('goog.dispose'); goog.require('ol.Expression'); goog.require('ol.Feature'); goog.require('ol.filter.Extent'); diff --git a/test/spec/ol/map.test.js b/test/spec/ol/map.test.js index 7f2e5f5530..2775487034 100644 --- a/test/spec/ol/map.test.js +++ b/test/spec/ol/map.test.js @@ -61,7 +61,7 @@ describe('ol.Map', function() { }); it('removes the viewport from its parent', function() { - map.dispose(); + goog.dispose(map); expect(goog.dom.getParentElement(map.getViewport())).to.be(null); }); }); @@ -223,6 +223,7 @@ describe('ol.Map', function() { }); goog.require('goog.async.AnimationDelay'); +goog.require('goog.dispose'); goog.require('goog.dom'); goog.require('ol.Collection'); goog.require('ol.Map'); diff --git a/test/spec/ol/renderer/webgl/webglimagelayerrenderer.test.js b/test/spec/ol/renderer/webgl/webglimagelayerrenderer.test.js index 21bbafab9f..71894d3c4d 100644 --- a/test/spec/ol/renderer/webgl/webglimagelayerrenderer.test.js +++ b/test/spec/ol/renderer/webgl/webglimagelayerrenderer.test.js @@ -37,7 +37,7 @@ describe('ol.renderer.webgl.ImageLayer', function() { }); afterEach(function() { - map.dispose(); + goog.dispose(map); }); it('produces a correct matrix', function() { @@ -77,6 +77,7 @@ describe('ol.renderer.webgl.ImageLayer', function() { }); }); +goog.require('goog.dispose'); goog.require('goog.vec.Mat4'); goog.require('goog.vec.Vec4'); goog.require('ol.Map');