ServiceWorker: Add a test to check if worker is deleted after reloading iframe following unregister

third_party/WebKit/LayoutTests/http/tests/serviceworker/unregister-then-register.html

Index: third_party/WebKit/LayoutTests/http/tests/serviceworker/unregister-then-register.html
diff --git a/third_party/WebKit/LayoutTests/http/tests/serviceworker/unregister-then-register.html b/third_party/WebKit/LayoutTests/http/tests/serviceworker/unregister-then-register.html
index cec2ddc85410777c300e6c0b14e1fdf958173378..78ca7a3f5b7dc97eaa6b7bdb07c87a256b65a302 100644
--- a/third_party/WebKit/LayoutTests/http/tests/serviceworker/unregister-then-register.html
+++ b/third_party/WebKit/LayoutTests/http/tests/serviceworker/unregister-then-register.html
@@ -57,6 +57,46 @@ async_test(function(t) {
          'registration is in use.');
 
 async_test(function(t) {
+    var scope = 'resources/scope/re-register-after-reloading-iframe-' +
+                'with-old-registration-in-use';
+    var old_registration;
+    var new_registration;

[nhiroki, 2016/04/11 09:51:36]

unused

[shimazu (google), 2016/04/12 03:42:30]

Done.

+    var frame;
+    var service_worker;

[nhiroki, 2016/04/11 09:51:36]

unused

[shimazu (google), 2016/04/12 03:42:30]

Done.

+    service_worker_unregister_and_register(t, worker_url, scope)
+      .then(function(r) {
+          old_registration = r;
+          return wait_for_state(t, r.installing, 'activated');
+        })
+      .then(function() {
+          return with_iframe(scope);
+        })
+      .then(function(f) {
+          frame = f;
+          return old_registration.unregister();
+        })
+      .then(function() {
+          return new Promise(function(resolve) {
+              frame.onload = resolve;
+              frame.contentWindow.location.reload();
+            });
+        })
+      .then(function() {
+          var c = frame.contentWindow.navigator.serviceWorker.controller;
+          assert_equals(c, null, 'a page after unregistration should not be ' +
+                                 'provided from service worker');

[nhiroki, 2016/04/11 09:51:36]

"controlled by" would be more common phrase than "provided from" in SW.

[shimazu (google), 2016/04/12 03:42:30]

Done.

+          return navigator.serviceWorker.getRegistration(scope);
+        })
+      .then(function(r) {
+          assert_equals(r, undefined, 'getRegistration should return ' +
+                                      'undefined after unregistration');
+          service_worker_unregister_and_done(t, scope);
+        })
+      .catch(unreached_rejection(t));
+}, 'Unregistration followed by reloading the iframe provided by a ' +
+   'service worker should unregister the service worker successfully');

[nhiroki, 2016/04/11 09:51:36]

I think the main action of this test is 'reloading' (not 'unregistration'), so
how about make it the subject? For example...

  'Reloading the last controlled iframe after unregistration should delete the
registration'

[shimazu (google), 2016/04/12 03:42:30]

Done.

+
+async_test(function(t) {
     var scope = 'resources/scope/re-register-does-not-affect-existing-controllee';
     var iframe;
     var registration;