Upstream service worker `fetch` test to WPT

Upstream service worker `fetch` test to WPT

**fetch-request-html-imports**

Use HTTPS version of absolute URLs and schedule iframe element to be
removed at the completion of the test.

**fetch-request-resources**

Extend the WPT version of this test with the novel aspects of the
Chromium version:

- assertions for the `redirect` attribute of the Request object
- 1 additional test case for image resource requests
- 2 additional test cases for stylesheet resource requests
- 15 test cases for "fetch" requests
- 6 test cases for audio resource requests

Note that while the Chromium version also contains test cases for image
resources loaded via CSS, equivalent tests are already present upstream
(under the name `fetch-request-css-images.https.html`) and therefore do
not need to be migrated.

Chromium continues to fail the upstream version due to its additional
assertions for the `integrity` attribute of the Request object. Due to
this, it cannot be used as a substitute for the Chromium-specific
version. Persist the Chromium-specific version under a new file name
documenting its deprecated status.

BUG=688116
R=mek@chromium.org

Review-Url: https://codereview.chromium.org/2858093002
Cr-Commit-Position: refs/heads/master@{#469726}
Committed: https://chromium.googlesource.com/chromium/src/+/be8f71f78b95b5d953e1f385120f9aa7f797ed0d

[Marijn Kruisselbrink, 2017-05-04 18:38:20]

lgtm, although it might be nice if there would be a bug associated whenever you
keep a chromium copy of a test around?

https://codereview.chromium.org/2858093002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/fetch-request-html-imports.https.html
(right):

https://codereview.chromium.org/2858093002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/fetch-request-html-imports.https.html:5:
<script src="/common/get-host-info.sub.js?pipe=sub"></script>
nit: no need for the ?pipe=sub

https://codereview.chromium.org/2858093002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/fetch-request-resources.https.html
(right):

https://codereview.chromium.org/2858093002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/fetch-request-resources.https.html:6:
<script src="resources/test-helpers.sub.js?pipe=sub"></script>
nit: get rid of the pipe=sub here too?

https://codereview.chromium.org/2858093002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium.fetch-request-resources.html
(right):

https://codereview.chromium.org/2858093002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium.fetch-request-resources.html:2:
<!-- This test is prefixed with `chromium.` because the equivalent version
is there a bug you can refer to in this comment?

[mike3, 2017-05-05 16:44:02]

All set, Mek. Thanks!

https://codereview.chromium.org/2858093002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/fetch-request-html-imports.https.html
(right):

https://codereview.chromium.org/2858093002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/fetch-request-html-imports.https.html:5:
<script src="/common/get-host-info.sub.js?pipe=sub"></script>
On 2017/05/04 18:38:19, Marijn Kruisselbrink wrote:
> nit: no need for the ?pipe=sub

Acknowledged.

https://codereview.chromium.org/2858093002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/fetch-request-resources.https.html
(right):

https://codereview.chromium.org/2858093002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/external/wpt/service-workers/service-worker/fetch-request-resources.https.html:6:
<script src="resources/test-helpers.sub.js?pipe=sub"></script>
On 2017/05/04 18:38:19, Marijn Kruisselbrink wrote:
> nit: get rid of the pipe=sub here too?

Acknowledged.

https://codereview.chromium.org/2858093002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium.fetch-request-resources.html
(right):

https://codereview.chromium.org/2858093002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/http/tests/serviceworker/chromium.fetch-request-resources.html:2:
<!-- This test is prefixed with `chromium.` because the equivalent version
On 2017/05/04 18:38:20, Marijn Kruisselbrink wrote:
> is there a bug you can refer to in this comment?

Acknowledged.

[mike3, 2017-05-05 16:45:12]

The CQ bit was checked by mike@mikepennisi.com

[mike3, 2017-05-05 16:45:13]

The patchset sent to the CQ was uploaded after l-g-t-m from mek@chromium.org
Link to the patchset: https://codereview.chromium.org/2858093002/#ps20001
(title: "Incorporate review feedback")

[commit-bot: I haz the power, 2017-05-05 16:45:37]

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-05-05 18:20:01]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1494002712956970,
"parent_rev": "e520c4590bfdf0c571bf534ababda17939b9e4ef", "commit_rev":
"be8f71f78b95b5d953e1f385120f9aa7f797ed0d"}

[commit-bot: I haz the power, 2017-05-05 18:22:14]

Description was changed from

==========
Upstream service worker `fetch` test to WPT

**fetch-request-html-imports**

Use HTTPS version of absolute URLs and schedule iframe element to be
removed at the completion of the test.

**fetch-request-resources**

Extend the WPT version of this test with the novel aspects of the
Chromium version:

- assertions for the `redirect` attribute of the Request object
- 1 additional test case for image resource requests
- 2 additional test cases for stylesheet resource requests
- 15 test cases for "fetch" requests
- 6 test cases for audio resource requests

Note that while the Chromium version also contains test cases for image
resources loaded via CSS, equivalent tests are already present upstream
(under the name `fetch-request-css-images.https.html`) and therefore do
not need to be migrated.

Chromium continues to fail the upstream version due to its additional
assertions for the `integrity` attribute of the Request object. Due to
this, it cannot be used as a substitute for the Chromium-specific
version. Persist the Chromium-specific version under a new file name
documenting its deprecated status.

BUG=688116
R=mek@chromium.org
==========

to

==========
Upstream service worker `fetch` test to WPT

**fetch-request-html-imports**

Use HTTPS version of absolute URLs and schedule iframe element to be
removed at the completion of the test.

**fetch-request-resources**

Extend the WPT version of this test with the novel aspects of the
Chromium version:

- assertions for the `redirect` attribute of the Request object
- 1 additional test case for image resource requests
- 2 additional test cases for stylesheet resource requests
- 15 test cases for "fetch" requests
- 6 test cases for audio resource requests

Note that while the Chromium version also contains test cases for image
resources loaded via CSS, equivalent tests are already present upstream
(under the name `fetch-request-css-images.https.html`) and therefore do
not need to be migrated.

Chromium continues to fail the upstream version due to its additional
assertions for the `integrity` attribute of the Request object. Due to
this, it cannot be used as a substitute for the Chromium-specific
version. Persist the Chromium-specific version under a new file name
documenting its deprecated status.

BUG=688116
R=mek@chromium.org

Review-Url: https://codereview.chromium.org/2858093002
Cr-Commit-Position: refs/heads/master@{#469726}
Committed:
https://chromium.googlesource.com/chromium/src/+/be8f71f78b95b5d953e1f385120f...
==========

[commit-bot: I haz the power, 2017-05-05 18:22:15]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/be8f71f78b95b5d953e1f385120f...