Request construction spec changes to use the 'used' flag.

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

 importScripts('worker-test-harness.js');
 importScripts('test-helpers.js');
 
 var URL = 'https://www.example.com/test.html';
 
 test(function() {
     var headers = new Headers;
     headers.set('User-Agent', 'Mozilla/5.0');
     headers.set('Accept', 'text/html');
     headers.set('X-ServiceWorker-Test', 'request test field');
@@ skipping 79 matching lines @@
                 return;
             }
             request1 = new Request(URL, init1);
             assert_equals(request1.method, method1 ? method1 : 'GET', 'Request.method should match');
             assert_equals(request1.mode, mode1 ? mode1 : 'cors', 'Request.mode should match');
             request1 = new Request(request1);
             assert_equals(request1.method, method1 ? method1 : 'GET', 'Request.method should match');
             assert_equals(request1.mode, mode1 ? mode1 : 'cors', 'Request.mode should match');
             METHODS.forEach(function(method2) {
                 MODES.forEach(function(mode2) {
+                    // We need to construct a new request1 because as soon as it
+                    // is used in a constructor it will be flagged as 'used',
+                    // and we can no longer construct objects with it.
+                    request1 = new Request(URL, init1);
                     var init2 = {};
                     if (method2 != undefined) { init2['method'] = method2; }
                     if (mode2 != undefined) { init2['mode'] = mode2; }
                     if (!isSimpleMethod(effectiveMethod(method1, method2)) && effectiveMode(mode1, mode2) == 'no-cors') {
                         assert_throws(
                             {name:'TypeError'},
                             function() { request2 = new Request(request1, init2); },
                             'new no-cors Request with non simple method should throw');
                         return;
                     }
@@ skipping 15 matching lines @@
     var request2 = {};
     var CREDENTIALS = ['omit', 'same-origin', 'include', '', undefined];
     CREDENTIALS.forEach(function(credentials1) {
         var init1 = {};
         if (credentials1 != undefined) { init1['credentials'] = credentials1; }
         request1 = new Request(URL, init1);
         assert_equals(request1.credentials, credentials1 ? credentials1 : 'omit', 'Request.credentials should match');
         request1 = new Request(request1);
         assert_equals(request1.credentials, credentials1 ? credentials1 : 'omit', 'Request.credentials should match');
         CREDENTIALS.forEach(function(credentials2) {
+            request1 = new Request(URL, init1);
             var init2 = {};
             if (credentials2 != undefined) { init2['credentials'] = credentials2; }
             request2 = new Request(request1, init2);
             assert_equals(request2.credentials,
                           credentials2 ? credentials2 : request1.credentials,
                           'Request.credentials should be overridden');
         });
     });
 }, 'Request credentials test in ServiceWorkerGlobalScope');
 
@@ skipping 132 matching lines @@
 
   ['PATCH', 'MKCOL', 'CUSTOM', 'X-FILES'].forEach(function(method) {
     assert_equals(new Request(url, {method: method}).method, method,
                   'method should not be changed when normalized: ' + method);
     method = method.toLowerCase();
     assert_equals(new Request(url, {method: method}).method, method,
                   'method should not be changed when normalized: ' + method);
   });
 }, 'Request method names are normalized');
 
+
+test(function() {
+    var req = new Request(URL);
+    assert_false(req.bodyUsed,
+      "Request should not be flagged as used if it has not been consumed.");
+    var req2 = new Request(req);
+    assert_true(req.bodyUsed,
+      "Request should be flagged as used if it is used as a construction " +
+      "argument of another Request.");
+    assert_false(req2.bodyUsed,
+      "Request should not be flagged as used if it has not been consumed.");
+    assert_throws(new TypeError(), function() { new Request(req); },
+      "Request cannot be constructed with a request that has been flagged as used.");
+  }, 'Request construction behavior regarding "used" body flag and exceptions.');
+
 async_test(function(t) {
     var getContentType = function(headers) {
         var content_type = '';
         headers.forEach(function(value, key) {
             if (key == 'content-type') {
               content_type = value;
             }
           });
         return content_type;
       };
@@ skipping 81 matching lines @@
             result, expected_body,
             'Creating a Request with FormData body should success.');
         })
       .then(function() {
           t.done();
         })
       .catch(unreached_rejection(t));
     assert_true(request.bodyUsed,
                 'bodyUsed must be true after calling text()');
   }, 'Request body test in ServiceWorkerGlobalScope');