Revert the debugging patch for bug 1157901
authorEhsan Akhgari <ehsan@mozilla.com>
Tue, 05 May 2015 21:43:53 -0400
changeset 273873 bf7256f12288f8444b608d447114cdfa44aab08c
parent 273872 380b0a052ef07f518ef411934b02c5db84ab790e
child 273874 5f0e9a84a57aa977ba43c52b6a809ccf0009f098
push id863
push userraliiev@mozilla.com
push dateMon, 03 Aug 2015 13:22:43 +0000
treeherdermozilla-release@f6321b14228d [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
bugs1157901
milestone40.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Revert the debugging patch for bug 1157901
dom/workers/test/serviceworkers/test_workerupdatefoundevent.html
dom/workers/test/serviceworkers/worker_updatefoundevent.js
dom/workers/test/serviceworkers/worker_updatefoundevent2.js
--- a/dom/workers/test/serviceworkers/test_workerupdatefoundevent.html
+++ b/dom/workers/test/serviceworkers/test_workerupdatefoundevent.html
@@ -12,34 +12,28 @@
 <body>
 <p id="display"></p>
 <div id="content"></div>
 <pre id="test"></pre>
 <script class="testbody" type="text/javascript">
   var registration;
   var promise;
 
-  SimpleTest.requestCompleteLog();
-
   function start() {
-    info("start got called");
     return navigator.serviceWorker.register("worker_updatefoundevent.js",
                                             { scope: "./updatefoundevent.html" })
       .then((swr) => registration = swr);
   }
 
   function startWaitForUpdateFound() {
-    info("startWaitForUpdateFound got called");
     registration.onupdatefound = function(e) {
-      info("onupdatefound");
     }
 
     promise = new Promise(function(resolve, reject) {
       window.onmessage = function(e) {
-        info("Got message " + e.data);
 
         if (e.data == "finish") {
           ok(true, "Received updatefound");
           resolve();
         }
       }
     });
 
@@ -47,28 +41,25 @@
     iframe = document.createElement("iframe");
     content.appendChild(iframe);
     iframe.setAttribute("src", "./updatefoundevent.html");
 
     return Promise.resolve();
   }
 
   function registerNext() {
-    info("registerNext got called");
     return navigator.serviceWorker.register("worker_updatefoundevent2.js",
                                             { scope: "./updatefoundevent.html" });
   }
 
   function waitForUpdateFound() {
-    info("waitForUpdateFound got called");
     return promise;
   }
 
   function unregister() {
-    info("unregister got called");
     return registration.unregister().then(function(result) {
       ok(result, "Unregister should return true.");
     });
   }
 
   function runTest() {
      start()
       .then(startWaitForUpdateFound)
@@ -80,15 +71,14 @@
       }).then(SimpleTest.finish);
   }
 
   SimpleTest.waitForExplicitFinish();
   SpecialPowers.pushPrefEnv({"set": [
     ["dom.serviceWorkers.exemptFromPerDomainMax", true],
     ["dom.serviceWorkers.enabled", true],
     ["dom.serviceWorkers.testing.enabled", true],
-    ["browser.dom.window.dump.enabled", true],
   ]}, runTest);
 </script>
 </pre>
 </body>
 </html>
 
--- a/dom/workers/test/serviceworkers/worker_updatefoundevent.js
+++ b/dom/workers/test/serviceworkers/worker_updatefoundevent.js
@@ -1,20 +1,18 @@
 /**
  * Any copyright is dedicated to the Public Domain.
  * http://creativecommons.org/publicdomain/zero/1.0/
  */
 
 onactivate = function(e) {
   e.waitUntil(new Promise(function(resolve, reject) {
     registration.onupdatefound = function(e) {
-      dump("Update found for scope " + registration.scope + "\n");
       clients.matchAll().then(function(clients) {
         if (!clients.length) {
-          dump("No clients found\n");
           reject("No clients found");
         }
 
         if (registration.scope.match(/updatefoundevent\.html$/)) {
           clients[0].postMessage("finish");
           resolve();
         } else {
           dump("Scope did not match");
--- a/dom/workers/test/serviceworkers/worker_updatefoundevent2.js
+++ b/dom/workers/test/serviceworkers/worker_updatefoundevent2.js
@@ -1,2 +1,1 @@
 // Not useful.
-dump("worker_updatefoundevent2.js loaded\n");