ServiceWorker: Check the equality of JS-level SW/SWR objects

third_party/WebKit/LayoutTests/http/tests/serviceworker/registration-service-worker-attributes.html

Index: third_party/WebKit/LayoutTests/http/tests/serviceworker/registration-service-worker-attributes.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/registration-service-worker-attributes.html b/third_party/WebKit/LayoutTests/http/tests/serviceworker/registration-service-worker-attributes.html
index dce8b1129d9969fc7eec09860fa75ecf84bb7728..cbeb4d3166936d2d32f10737978b4ec6094bf004 100644
--- a/third_party/WebKit/LayoutTests/http/tests/serviceworker/registration-service-worker-attributes.html
+++ b/third_party/WebKit/LayoutTests/http/tests/serviceworker/registration-service-worker-attributes.html
@@ -3,14 +3,17 @@
 <script src="../resources/testharnessreport.js"></script>
 <script src="resources/test-helpers.js"></script>
 <script>
-async_test(function(t) {
+promise_test(function(t) {
     var scope = 'resources/scope/installing-waiting-active-after-registration';
     var worker_url = 'resources/empty-worker.js';
     var expected_url = normalizeURL(worker_url);
+    var installing_worker;
+    var registration;
 
-    service_worker_unregister_and_register(t, worker_url, scope)
+    return service_worker_unregister_and_register(t, worker_url, scope)
       .then(function(r) {
           registration = r;

[nhiroki, 2016/04/18 03:26:53]

Can you add this here?

  add_completion_callback(function() { registration.unregister(); });

[shimazu (google), 2016/04/18 06:39:13]

Done.

+          installing_worker = registration.installing;
           assert_equals(registration.installing.scriptURL, expected_url,
                         'installing before updatefound');
           assert_equals(registration.waiting, null,
@@ -19,8 +22,8 @@ async_test(function(t) {
                         'active before updatefound');
           return wait_for_update(t, registration);
         })
-      .then(function(worker) {
-          assert_equals(registration.installing.scriptURL, expected_url,
+      .then(function() {
+          assert_equals(registration.installing, installing_worker,
                         'installing after updatefound');
           assert_equals(registration.waiting, null,
                         'waiting after updatefound');
@@ -31,7 +34,7 @@ async_test(function(t) {
       .then(function() {
           assert_equals(registration.installing, null,
                         'installing after installed');
-          assert_equals(registration.waiting.scriptURL, expected_url,
+          assert_equals(registration.waiting, installing_worker,
                         'waiting after installed');
           assert_equals(registration.active, null,
                         'active after installed');
@@ -42,7 +45,7 @@ async_test(function(t) {
                         'installing after activated');
           assert_equals(registration.waiting, null,
                         'waiting after activated');
-          assert_equals(registration.active.scriptURL, expected_url,
+          assert_equals(registration.active, installing_worker,
                         'active after activated');
           return Promise.all([
               wait_for_state(t, registration.active, 'redundant'),
@@ -60,8 +63,6 @@ async_test(function(t) {
           // resolved, this will be gone.
           assert_equals(registration.active, null,
                         'active should be null after redundant');
-          t.done();
-        })
-      .catch(unreached_rejection(t));
+        });
   }, 'installing/waiting/active after registration');
 </script>