Fetch API: Don't exclude the URL fragment in Request#url

third_party/WebKit/LayoutTests/http/tests/fetch/script-tests/fetch.js

 if (self.importScripts) {
   importScripts('../resources/fetch-test-helpers.js');
 }
 
 promise_test(function(t) {
     return fetch('http://')
       .then(
         t.unreached_func('fetch of invalid URL must fail'),
         function() {});
   }, 'Fetch invalid URL');
@@ skipping 149 matching lines @@
                 self.internals.getInternalResponseURLList(response),
                 [request.url]);
           }
         });
   }, 'Fetch result of 404 response');
 
 promise_test(function(t) {
     var request = new Request(
       '/fetch/resources/fetch-status.php?status=200#fragment');
 
-    // The url attribute's getter must return request's url,
-    // serialized with the exclude fragment flag set.
+    // The url attribute's getter must return request's url with the fragment
+    // flag set.

[falken, 2017/03/31 06:40:49]

There's no "fragment flag", it's just the "exclude fragment flag" from the URL
spec: https://url.spec.whatwg.org/#url-serializing

This comment can just deleted: it's a bit confusing to just point out that the
"exclude fragment flag is not set" since that's the default.

[yiyix, 2017/03/31 07:24:02]

True, the term "flag set" is not used in the spec. I will delete the comment.

     assert_equals(request.url,
-      BASE_ORIGIN + '/fetch/resources/fetch-status.php?status=200');
+      BASE_ORIGIN + '/fetch/resources/fetch-status.php?status=200#fragment');
 
     return fetch(request)
       .then(function(response) {
           assert_equals(response.status, 200);
           assert_equals(response.statusText, 'OK');
           // The url attribute's getter must return the empty string
           // if response's url is null and response's url,
           // serialized with the exclude fragment flag set, otherwise.
           assert_equals(response.url,
             BASE_ORIGIN + '/fetch/resources/fetch-status.php?status=200');
@@ skipping 259 matching lines @@
     assert_true(req.bodyUsed);
 }, 'Calling fetch() disturbs body if not null');
 
 test(t => {
     var req = new Request('/', {method: 'POST'});
     fetch(req);
     assert_false(req.bodyUsed);
 }, 'Calling fetch() doesn\'t disturb body if null');
 
 done();