[ServiceWorker] Update formatting for tests.

LayoutTests/http/tests/serviceworker/resources/test-helpers.js

Index: LayoutTests/http/tests/serviceworker/resources/test-helpers.js
diff --git a/LayoutTests/http/tests/serviceworker/resources/test-helpers.js b/LayoutTests/http/tests/serviceworker/resources/test-helpers.js
index 6b9f79fec6dd0b7c2d9948de53f25614793c6196..75dd45f21762be516db847c6a0458a3c045fd1e6 100644
--- a/LayoutTests/http/tests/serviceworker/resources/test-helpers.js
+++ b/LayoutTests/http/tests/serviceworker/resources/test-helpers.js
@@ -1,46 +1,45 @@
 // Adapter for testharness.js-style tests with Service Workers
 
 function service_worker_unregister_and_register(test, url, scope) {
-    var options = scope ? { scope: scope } : {};
-    return navigator.serviceWorker.unregister(scope).then(
-        test.step_func(function() {
-            return navigator.serviceWorker.register(url, options);
-        }),
-        unreached_rejection(test, 'Unregister should not fail')
-    ).then(test.step_func(function(worker) {
-          return Promise.resolve(worker);
-        }),
-        unreached_rejection(test, 'Registration should not fail')
-    );
+  var options = scope ? {scope: scope} : {};
+  return navigator.serviceWorker.unregister(scope)
+    .then(test.step_func(
+      function() {
+        return navigator.serviceWorker.register(url, options);
+      }),
+          unreached_rejection(test, 'Unregister should not fail'))
+    .then(
+      test.step_func(function(worker) { return Promise.resolve(worker); }),

[jsbell, 2014/07/31 20:37:23]

Is this .then() clause necessary, or can we drop it and just have a .catch()

[asanka, 2014/08/01 05:15:52]

Replaced with a .catch().then().catch() chain.

+      unreached_rejection(test, 'Registration should not fail'));
 }
 
 function service_worker_unregister_and_done(test, scope) {
-    return navigator.serviceWorker.unregister(scope).then(
-        test.done.bind(test),
-        unreached_rejection(test, 'Unregister should not fail'));
+  return navigator.serviceWorker.unregister(scope)
+    .then(test.done.bind(test),
+          unreached_rejection(test, 'Unregister should not fail'));
 }
 
 // Rejection-specific helper that provides more details
 function unreached_rejection(test, prefix) {
-    return test.step_func(function(error) {
-        var reason = error.name ? error.name : error;
-        var prefix = prefix ? prefix : "unexpected rejection";
-        assert_unreached(prefix + ': ' + reason);
+  return test.step_func(function(error) {
+      var reason = error.message || error.name || error;
+      var error_prefix = prefix || 'unexpected rejection';
+      assert_unreached(error_prefix + ': ' + reason);
     });
 }
 
 // FIXME: Clean up the iframe when the test completes.
 function with_iframe(url, f) {
-    return new Promise(function(resolve, reject) {
-        var frame = document.createElement('iframe');
-        frame.src = url;
-        frame.onload = function() {
-            if (f) {
-              f(frame);
-            }
-            resolve(frame);
-        };
-        document.body.appendChild(frame);
+  return new Promise(function(resolve, reject) {
+      var frame = document.createElement('iframe');
+      frame.src = url;
+      frame.onload = function() {
+        if (f) {
+          f(frame);
+        }
+        resolve(frame);
+      };
+      document.body.appendChild(frame);
     });
 }
 
@@ -49,11 +48,12 @@ function normalizeURL(url) {
 }
 
 function wait_for_state(test, worker, state) {
-    return new Promise(test.step_func(function(resolve, reject) {
-        worker.addEventListener('statechange', test.step_func(function() {
+  return new Promise(test.step_func(function(resolve, reject) {
+      worker.addEventListener('statechange',
+                              test.step_func(function() {
             if (worker.state === state)
-                resolve(state);
-        }));
+              resolve(state);
+          }));
     }));
 }