Make registration.update() no longer force bypassing the HTTP cache (1/2)

LayoutTests/http/tests/serviceworker/ServiceWorkerGlobalScope/resources/update-worker.php

 <?php
-// Force the browser to cache this script. update() should always bypass this
-// cache and fetch a new version.
-header('Cache-Control: max-age=86400');
+if(!isset($_COOKIE['mode']))
+  $mode = 'no-cache'; // Set mode to 'no-cache' for initial register().
+else
+  $mode = $_COOKIE['mode'];
 
+if ($mode == 'no-cache') {
+  // Set max-age to 0 so the next update will go to the network.
+  // Set cookie value to 'cache' so the next fetch will work in 'cache' mode.
+  header('Cache-Control: max-age=0');
+  setcookie('mode', 'cache');
+} else if ($mode == 'cache') {
+  // Set max-age to non-zero value so the next update will be served from cache.
+  // Unset cookie value.
+  header('Cache-Control: max-age=3600');
+  unset($_COOKIE['mode']);
+  setcookie('mode', '', time() - 3600);
+}
 // Return a different script for each access.
 header('Content-Type:application/javascript');
 echo '// ' . microtime();
 ?>
 
 importScripts('../../resources/test-helpers.js');
 importScripts('../../resources/worker-testharness.js');
 
 var events_seen = [];

[falken, 2015/10/01 14:58:03]

BTW a lot of our tests do this but ideally we shouldn't rely on global state in
the sw.

 
 self.registration.addEventListener('updatefound', function() {
     events_seen.push('updatefound');
   });
 
 self.addEventListener('activate', function(e) {
     events_seen.push('activate');
   });
 
 self.addEventListener('fetch', function(e) {
     events_seen.push('fetch');
     e.respondWith(new Response(events_seen));
+    events_seen = []; // Init events_seen for the next update.
   });
 
 self.addEventListener('message', function(e) {
     events_seen.push('message');
-    self.registration.update();
+    self.registration.update()
+      .then(function() {
+          if (self.registration.installing) // A new version found.
+            events_seen.push('new-version');
+          else                              // The same version served.
+            events_seen.push('cached-version');
+
+          self.clients.matchAll().then(function(clients) {
+              clients.forEach(function(c) {
+                  if (c.url == e.data) {
+                    c.postMessage('update resolved');
+                  }
+                });
+            });
+        });
   });
 
 // update() during the script evaluation should be ignored.
 self.registration.update();