Merge pull request #195 from aogilvie/submissions/aogilvie

Fixed a bug where if testharness.js sends a postMessage before either of the iframes reply, the test ends with a fail.
web-platform-tests · Jun 8, 2013 · 96fd027 · 96fd027
2 parents 3000064 + 5a767ac
commit 96fd027
Showing 1 changed file with 8 additions and 6 deletions.
diff --git a/webmessaging/event.data.htm b/webmessaging/event.data.htm
@@ -43,13 +43,15 @@
 
     window.onmessage = t.step_func(function(e)
     {
-        ActualResult.push(e.data, typeof(e.data));
+        if (e.data.toString() === "STRING") {
+        	ActualResult.push(e.data, typeof(e.data));
 
-        if (ActualResult.length >= ExpectedResult.length)
-        {
-            assert_array_equals(ActualResult, ExpectedResult, "ActualResult");
-            t.done();
-        }
+	        if (ActualResult.length >= ExpectedResult.length)
+	        {
+	            assert_array_equals(ActualResult, ExpectedResult, "ActualResult");
+	            t.done();
+	        }
+		}
     });
 </script>
 </body>