ServiceWorker: throw when close() or terminate() called

LayoutTests/http/tests/serviceworker/resources/fetch-worker.js

 self.onmessage = function(e) {
     var message = e.data;
     if ('port' in message) {
         port = message.port;
         doNextFetchTest(port);
     }
 };
 
 var testTargets = [
     'other.html',
     'http://',
     'http://www.example.com/foo',
     'fetch-status.php?status=200',
     'fetch-status.php?status=404'
 ];
 
 function doNextFetchTest(port) {
     if (testTargets.length == 0) {
         port.postMessage('quit');
         // Destroying the execution context while fetch is happening should not cause a crash.
         fetch('dummy.html').then(function() {}).catch(function() {});
-        self.close();
+        // FIXME: Since we can't ensure the context is torn down while

[horo, 2014/08/27 02:24:50]

Could you please add this method

function runInfiniteFetchLoop() {
    fetch('dummy.html')
      .then(function() { runInfiniteFetchLoop(); })
}

and call this before port.postMessage('quit');?

[jsbell, 2014/08/27 16:27:09]

Done. Let's see what the try bots think.

+        // this fetch is outstanding, consider making it re-fetch on
+        // success.
         return;
     }
     var target = testTargets.shift();
     fetch(target)
     .then(function(response) {
         port.postMessage('Resolved: ' + target + ' [' + response.status + ']' + response.statusText);
         doNextFetchTest(port);
     }).catch(function(e) {
         port.postMessage('Rejected: ' + target + ' : '+ e.message);
         doNextFetchTest(port);
     });
 };