Merge branch '5819_fix_for_empty_icon_tag' of github.com:josh-bernstein/cesium into 5819_fix_for_empty_icon_tag

Author: Josh Bernstein

CHANGES.md

@@ -15,6 +15,8 @@ Change Log
 * Fixed CZML processing of `velocityReference` within an interval. [#5738](https://github.com/AnalyticalGraphicsInc/cesium/issues/5738)
 * Zoom about mouse now maintains camera heading, pitch, and roll [#4639](https://github.com/AnalyticalGraphicsInc/cesium/pull/5603)
 * Fixed a bug in `PolylineCollection` preventing the display of more than 16K points in a single collection [#5538](https://github.com/AnalyticalGraphicsInc/cesium/pull/5782)
+* Fixed removing multiple event listeners within event callbacks. [#5827](https://github.com/AnalyticalGraphicsInc/cesium/issues/5827)
+* Fixed a 3D Tiles point cloud bug causing a stray point to appear at the center of the screen on certain hardware. [#5599](https://github.com/AnalyticalGraphicsInc/cesium/issues/5599)
 * Fixed bug with placemarks in imported KML: placemarks with no specified icon would be displayed with default icon (https://github.com/AnalyticalGraphicsInc/cesium/issues/5819)
 
 ### 1.37 - 2017-09-01

Source/Core/Event.js

@@ -120,6 +120,10 @@ define([
         return false;
     };
 
+    function compareNumber(a,b) {
+        return b - a;
+    }
+
     /**
      * Raises the event by calling each registered listener with all supplied arguments.
      *
@@ -146,12 +150,15 @@ define([
         //Actually remove items removed in removeEventListener.
         var toRemove = this._toRemove;
         length = toRemove.length;
-        for (i = 0; i < length; i++) {
-            var index = toRemove[i];
-            listeners.splice(index, 1);
-            scopes.splice(index, 1);
+        if (length > 0) {
+            toRemove.sort(compareNumber);
+            for (i = 0; i < length; i++) {
+                var index = toRemove[i];
+                listeners.splice(index, 1);
+                scopes.splice(index, 1);
+            }
+            toRemove.length = 0;
         }
-        toRemove.length = 0;
 
         this._insideRaiseEvent = false;
     };

Specs/Core/EventSpec.js

@@ -45,7 +45,7 @@ defineSuite([
         expect(spyListener).not.toHaveBeenCalled();
     });
 
-    it('can remove from withing a callback', function() {
+    it('can remove from within a callback', function() {
         var doNothing = function(evt) {
         };
 
@@ -67,6 +67,25 @@ defineSuite([
         expect(event.numberOfListeners).toEqual(0);
     });
 
+    it('can remove multiple listeners within a callback', function() {
+        var removeEvent0 =  event.addEventListener(function() { removeEvent0(); });
+                            event.addEventListener(function() {});
+        var removeEvent2 =  event.addEventListener(function() { removeEvent2(); });
+                            event.addEventListener(function() {});
+        var removeEvent4 =  event.addEventListener(function() { removeEvent4(); });
+                            event.addEventListener(function() {});
+        var removeEvent6 =  event.addEventListener(function() { removeEvent6(); });
+                            event.addEventListener(function() {});
+        var removeEvent8 =  event.addEventListener(function() { removeEvent8(); });
+                            event.addEventListener(function() {});
+
+        expect(event.numberOfListeners).toEqual(10);
+        event.raiseEvent();
+        expect(event.numberOfListeners).toEqual(5);
+        event.raiseEvent();
+        expect(event.numberOfListeners).toEqual(5);
+    });
+
     it('addEventListener and removeEventListener works with same function of different scopes', function() {
         var Scope = function() {
             this.timesCalled = 0;