Bug 863813 - Only call the done function once per test, failure or success. r=smaug

Author: Guilherme Gonçalves

content/media/webspeech/recognition/test/head.js

@@ -43,6 +43,8 @@ function EventManager(sr) {
     "audioend": "audiostart"
   };
 
+  var isDone = false;
+
   // AUDIO_DATA events are asynchronous,
   // so we queue events requested while they are being
   // issued to make them seem synchronous
@@ -59,7 +61,10 @@ function EventManager(sr) {
         }
 
         ok(false, message);
-        if (self.done) self.done();
+        if (self.doneFunc && !isDone) {
+          isDone = true;
+          self.doneFunc();
+        }
       };
     })(allEvents[i]);
   }
@@ -78,8 +83,10 @@ function EventManager(sr) {
       }
 
       cb && cb(evt, sr);
-      if (self.done && nEventsExpected === self.eventsReceived.length) {
-        self.done();
+      if (self.doneFunc && !isDone &&
+          nEventsExpected === self.eventsReceived.length) {
+        isDone = true;
+        self.doneFunc();
       }
     }
   }
@@ -150,7 +157,7 @@ function performTest(options) {
       em.expect(eventName, cb);
     }
 
-    em.done = function() {
+    em.doneFunc = function() {
       em.requestTestEnd();
       options.doneFunc();
     }