Fix bug in request-init-003.sub.html

Fix bug in request-init-003.sub.html

According to the Fetch spec, the Request constructor is called
with |input| only, then referrer attribute will be set to 
"about:client".

From fetch spec, https://fetch.spec.whatwg.org/#concept-request:
Unless the referrer is set to anther value in request, it is set to 
"about:client" by default. 

BUG=624278
Review-Url: https://codereview.chromium.org/2802153002
Cr-Commit-Position: refs/heads/master@{#462811}
Committed: https://chromium.googlesource.com/chromium/src/+/3602528ca6d0a7b9bf3b8569c1974ced69249eb7

[yiyix, 2017-04-07 02:02:24]

Description was changed from

==========
fix test cases

BUG=
==========

to

==========
Fix bug in request-init-003.sub.html

According to the service worker spec, if the fetch request is sent 
with url only, then referrer attribute will be set to "about:client".

BUG=624278
==========

[yiyix, 2017-04-07 02:02:24]

yiyix@chromium.org changed reviewers:
+ falken@chromium.org

[yiyix, 2017-04-07 02:02:49]

@falken, could you please review this change? thank you.

[falken, 2017-04-07 02:56:08]

Could you link to the spec and quote the relevant section in the bug
description? I just want to confirm the change is correct and you seem to have
done the spec research already.

I think it'd be the Fetch API spec, not the service worker spec.

[yiyix, 2017-04-07 06:57:15]

Description was changed from

==========
Fix bug in request-init-003.sub.html

According to the service worker spec, if the fetch request is sent 
with url only, then referrer attribute will be set to "about:client".

BUG=624278
==========

to

==========
Fix bug in request-init-003.sub.html

According to the service worker spec, if the fetch request is sent 
with url only, then referrer attribute will be set to "about:client".

From fetch spec, https://fetch.spec.whatwg.org/#concept-request:
Unless the referrer is set to anther value in request, it is set to 
"about:client" by default. 

BUG=624278
==========

[yiyix, 2017-04-07 06:58:37]

On 2017/04/07 02:56:08, falken wrote:
> Could you link to the spec and quote the relevant section in the bug
> description? I just want to confirm the change is correct and you seem to have
> done the spec research already.
> 
> I think it'd be the Fetch API spec, not the service worker spec.

I paraphrased the spec and linked the section in the description. Do you think
it is clear now?

[falken, 2017-04-07 07:08:21]

I think this is right. lgtm.

nits in CL description:
s/the service worker spec/the Fetch spec
s/if the fetch request is sent with url only/the Request constructor is called
with |input| only

[yiyix, 2017-04-07 07:40:03]

Description was changed from

==========
Fix bug in request-init-003.sub.html

According to the service worker spec, if the fetch request is sent 
with url only, then referrer attribute will be set to "about:client".

From fetch spec, https://fetch.spec.whatwg.org/#concept-request:
Unless the referrer is set to anther value in request, it is set to 
"about:client" by default. 

BUG=624278
==========

to

==========
Fix bug in request-init-003.sub.html

According to the Fetch spec, the Request constructor is called
with |input| only, then referrer attribute will be set to 
"about:client".

From fetch spec, https://fetch.spec.whatwg.org/#concept-request:
Unless the referrer is set to anther value in request, it is set to 
"about:client" by default. 

BUG=624278
==========

[yiyix, 2017-04-07 07:40:18]

On 2017/04/07 07:08:21, falken wrote:
> I think this is right. lgtm.
> 
> nits in CL description:
> s/the service worker spec/the Fetch spec
> s/if the fetch request is sent with url only/the Request constructor is called
> with |input| only

Thanks. Corrected.

[yiyix, 2017-04-07 07:40:29]

The CQ bit was checked by yiyix@chromium.org

[commit-bot: I haz the power, 2017-04-07 07:40:45]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-04-07 08:40:32]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1491550829592180, "parent_rev":
"7a66ab96e524e57f6717edb13f375d50f9cd89e8", "commit_rev":
"3602528ca6d0a7b9bf3b8569c1974ced69249eb7"}

[commit-bot: I haz the power, 2017-04-07 08:41:20]

Description was changed from

==========
Fix bug in request-init-003.sub.html

According to the Fetch spec, the Request constructor is called
with |input| only, then referrer attribute will be set to 
"about:client".

From fetch spec, https://fetch.spec.whatwg.org/#concept-request:
Unless the referrer is set to anther value in request, it is set to 
"about:client" by default. 

BUG=624278
==========

to

==========
Fix bug in request-init-003.sub.html

According to the Fetch spec, the Request constructor is called
with |input| only, then referrer attribute will be set to 
"about:client".

From fetch spec, https://fetch.spec.whatwg.org/#concept-request:
Unless the referrer is set to anther value in request, it is set to 
"about:client" by default. 

BUG=624278

Review-Url: https://codereview.chromium.org/2802153002
Cr-Commit-Position: refs/heads/master@{#462811}
Committed:
https://chromium.googlesource.com/chromium/src/+/3602528ca6d0a7b9bf3b8569c197...
==========

[commit-bot: I haz the power, 2017-04-07 08:41:22]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/3602528ca6d0a7b9bf3b8569c197...