Bug 1522246 - Fix browser_test_performance_metrics.js test, r=tarek.

--HG--
rename : dom/tests/browser/shared_worker.js => dom/tests/browser/perfmetrics/shared_worker.js
extra : rebase_source : f9f6b44886fffe48bddcb1907213e04a24718b55

dom/tests/browser/perfmetrics/browser.ini

@@ -10,6 +10,7 @@ support-files =
   ping_worker.js
   setinterval.html
   settimeout.html
+  shared_worker.js
   unresponsive.html
   hello.ogg
   sound.html

dom/tests/browser/perfmetrics/browser_test_performance_metrics.js

@@ -138,9 +138,7 @@ add_task(async function test() {
     Assert.ok(isTopLevel, "example.com as a top level window");
     Assert.ok(aboutMemoryFound, "about:memory");
     Assert.ok(heapUsage > 0, "got some memory value reported");
-    // FIXME bug 1522246. the worker performance improvements in this bug cause
-    // the shared worker to shut down too quickly to report any info.
-    // Assert.ok(sharedWorker, "We got some info from a shared worker");
+    Assert.ok(sharedWorker, "We got some info from a shared worker");
     let numCounters = counterIds.length;
     Assert.ok(numCounters > 5, "This test generated at least " + numCounters + " unique counters");
 

dom/tests/browser/perfmetrics/ping_worker2.html

@@ -9,7 +9,7 @@
   function init() {
    shared = new SharedWorker("shared_worker.js");
    shared.port.start();
-   for (let i = 0; i++; i < 10) shared.port.postMessage(["ok"]);
+   for (let i = 0; i < 10; i++) shared.port.postMessage(["ok"]);
   }
 
   </script>