Import wpt@5a1f188e0536ad023936cc62f9a00617299dc192

Import wpt@5a1f188e0536ad023936cc62f9a00617299dc192

Using update-w3c-deps in Chromium 2040bff2a904885249b1d58f3950b80ae45ebcbd.

Imports the service-workers/cache-storage as well as recent upstream changes.

BUG=666991, 666993, 432746, 667376
Committed: https://crrev.com/80cc8049a86123976cc7255f2bcd689322f596bf
Cr-Commit-Position: refs/heads/master@{#433651}

[jsbell, 2016-11-19 00:51:49]

The CQ bit was checked by jsbell@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-11-19 00:52:28]

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

[jsbell, 2016-11-19 00:54:38]

jsbell@chromium.org changed reviewers:
+ qyearsley@chromium.org, tkent@chromium.org

[jsbell, 2016-11-19 00:54:39]

qyearsley@, tkent@ - please take a look?

Filed https://crbug.com/666991 for the cache-storage flakes that'll need fixing
upstream. Several other cache-storage tests failed, added expectations for
those.

As follow up we'll need to file bugs for those failures and dedupe - we should
now be able to delete most of the tests in http/tests/serviceworker and
http/tests/cachestorage !!!

[jsbell, 2016-11-19 00:59:12]

Description was changed from

==========
Import wpt@5a1f188e0536ad023936cc62f9a00617299dc192

Using update-w3c-deps in Chromium 2040bff2a904885249b1d58f3950b80ae45ebcbd.

Imports the service-workers/cache-storage as well as recent upstream changes.

BUG=666991,666993
==========

to

==========
Import wpt@5a1f188e0536ad023936cc62f9a00617299dc192

Using update-w3c-deps in Chromium 2040bff2a904885249b1d58f3950b80ae45ebcbd.

Imports the service-workers/cache-storage as well as recent upstream changes.

BUG=666991,666993,432746
==========

[jsbell, 2016-11-19 01:02:29]

A few random notes on new expectations...

https://codereview.chromium.org/2517743002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/imported/wpt/service-workers/cache-storage/serviceworker/cache-add.https-expected.txt
(right):

https://codereview.chromium.org/2517743002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/imported/wpt/service-workers/cache-storage/serviceworker/cache-add.https-expected.txt:18:
FAIL Cache.addAll called with the same Request object specified twice
promise_test: Unhandled rejection with value: "assert_promise_rejects:
Cache.addAll should throw InvalidStateError if the same request is added twice.
Promise did not reject."
Looks like a legit bug; I thought we'd fixed all of these.

https://codereview.chromium.org/2517743002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/imported/wpt/service-workers/cache-storage/serviceworker/cache-storage.https-expected.txt
(right):

https://codereview.chromium.org/2517743002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/imported/wpt/service-workers/cache-storage/serviceworker/cache-storage.https-expected.txt:11:
FAIL CacheStorage names are DOMStrings not USVStrings assert_true: keys should
include cache with bad name expected true got false
This is https://crbug.com/432746

https://codereview.chromium.org/2517743002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/imported/wpt/service-workers/cache-storage/serviceworker/credentials-expected.txt
(right):

https://codereview.chromium.org/2517743002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/imported/wpt/service-workers/cache-storage/serviceworker/credentials-expected.txt:2:
FAIL Cache API matching includes credentials promise_test: Unhandled rejection
with value: object "SecurityError: Failed to execute 'delete' on 'CacheStorage':
Only secure origins are allowed (see: https://goo.gl/Y0ZkNV)."
I think this one needs to be renamed .https.html (upstream)

https://codereview.chromium.org/2517743002/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/imported/wpt/service-workers/cache-storage/window/cache-match.https-expected.txt
(right):

https://codereview.chromium.org/2517743002/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/imported/wpt/service-workers/cache-storage/window/cache-match.https-expected.txt:11:
FAIL Cache.match with Request and Response objects with different URLs
assert_array_equals: Cache.match should return a Response object that has the
same properties as the stored response. Headers differ. property 1, expected
"content-type: text/plain" but got "server: BaseHTTP/0.3 Python/2.7.6"
No idea about this one - header order? or are we plumbing through headers we
shouldn't be?

[commit-bot: I haz the power, 2016-11-19 01:41:20]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-19 01:41:21]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[tkent, 2016-11-21 01:04:11]

lgtm

[qyearsley, 2016-11-21 16:48:07]

lgtm too although I didn't look very closely

[jsbell, 2016-11-21 18:08:21]

Description was changed from

==========
Import wpt@5a1f188e0536ad023936cc62f9a00617299dc192

Using update-w3c-deps in Chromium 2040bff2a904885249b1d58f3950b80ae45ebcbd.

Imports the service-workers/cache-storage as well as recent upstream changes.

BUG=666991,666993,432746
==========

to

==========
Import wpt@5a1f188e0536ad023936cc62f9a00617299dc192

Using update-w3c-deps in Chromium 2040bff2a904885249b1d58f3950b80ae45ebcbd.

Imports the service-workers/cache-storage as well as recent upstream changes.

BUG=666991,666993,432746,667376
==========

[jsbell, 2016-11-21 18:10:50]

The CQ bit was checked by jsbell@chromium.org

[jsbell, 2016-11-21 18:10:51]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org,
qyearsley@chromium.org
Link to the patchset: https://codereview.chromium.org/2517743002/#ps20001
(title: "Add TestExpectations entry for crbug.com/667376")

[commit-bot: I haz the power, 2016-11-21 18:11:24]

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, 2016-11-21 19:09:18]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-11-21 19:09:21]

Failed to apply patch for third_party/WebKit/LayoutTests/TestExpectations:
While running git apply --index -p1;
  error: patch failed: third_party/WebKit/LayoutTests/TestExpectations:1705
  error: third_party/WebKit/LayoutTests/TestExpectations: patch does not apply

Patch:       third_party/WebKit/LayoutTests/TestExpectations
Index: third_party/WebKit/LayoutTests/TestExpectations
diff --git a/third_party/WebKit/LayoutTests/TestExpectations
b/third_party/WebKit/LayoutTests/TestExpectations
index
0f7b89d6bfc4605f81722f9e9b72240420c8c06f..4c092695c131e61e32d31b15bfe4d5465d2c6e9b
100644
--- a/third_party/WebKit/LayoutTests/TestExpectations
+++ b/third_party/WebKit/LayoutTests/TestExpectations
@@ -1454,7 +1454,7 @@ crbug.com/660580 [ Precise ]
paint/invalidation/table/composited-table-backgroun
 crbug.com/660580 [ Precise ]
fast/xmlhttprequest/xmlhttprequest-responsetype-arraybuffer.html [ Skip ]
 crbug.com/660580 [ Precise ] images/imagemap-case.html [ Skip ]
 crbug.com/660580 [ Precise ]
virtual/gpu-rasterization/images/imagemap-case.html [ Skip ]
-crbug.com/660580 [ Precise ]
http/tests/inspector-enabled/console-stack-overflow-source-url.html [ Skip ] 
+crbug.com/660580 [ Precise ]
http/tests/inspector-enabled/console-stack-overflow-source-url.html [ Skip ]
 
 # ====== New tests from w3c-test-autoroller added here ======
 crbug.com/626703
imported/csswg-test/vendor-imports/mozilla/mozilla-central-reftests/flexbox/flexbox-align-self-baseline-horiz-001a.xhtml
[ Failure ]
@@ -1705,3 +1705,17 @@ crbug.com/666602
inspector-enabled/sources/debugger/script-formatter-breakpoints
 crbug.com/666628 [ Mac Win Trusty ]
http/tests/inspector-enabled/console-stack-overflow-source-url.html [
NeedsRebaseline ]
 
 crbug.com/619427 [ Linux ]
fast/overflow/overflow-height-float-not-removed-crash3.html [ Pass Failure ]
+
+# Fail when run in random order.
+crbug.com/666991
imported/wpt/service-workers/cache-storage/window/cache-storage.https.html [
Pass Failure ]
+crbug.com/666991
imported/wpt/service-workers/cache-storage/worker/cache-storage.https.html [
Pass Failure ]
+crbug.com/666991
imported/wpt/service-workers/cache-storage/serviceworker/cache-storage.https.html
[ Pass Failure ]
+
+# Newly imported tests that time out.
+crbug.com/666993
imported/wpt/html/webappapis/idle-callbacks/callback-idle-periods.html [ Timeout
]
+crbug.com/666993
imported/wpt/html/webappapis/idle-callbacks/callback-timeout-with-raf.html [
Timeout ]
+crbug.com/666993 imported/wpt/html/webappapis/idle-callbacks/basic.html [
Timeout ]
+
+# These tests (erroneously?) see a platform-specific 'server' header
+crbug.com/667376
imported/wpt/service-workers/cache-storage/window/cache-match.https.html [ Pass
Failure ]
+crbug.com/667376
imported/wpt/service-workers/cache-storage/window/cache-put.https.html [ Pass
Failure ]

[jsbell, 2016-11-21 20:01:52]

The CQ bit was checked by jsbell@chromium.org

[jsbell, 2016-11-21 20:01:53]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org,
qyearsley@chromium.org
Link to the patchset: https://codereview.chromium.org/2517743002/#ps40001
(title: "rebased")

[commit-bot: I haz the power, 2016-11-21 20:03:53]

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, 2016-11-21 21:11:37]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1479758512692720,
"parent_rev": "a9a768ab07383ca06b3ff7e3f551cae3fb56e050", "commit_rev":
"fb779f01d1ec8e1c8913e3eb71224f36f302cf42"}

[commit-bot: I haz the power, 2016-11-21 21:12:04]

Description was changed from

==========
Import wpt@5a1f188e0536ad023936cc62f9a00617299dc192

Using update-w3c-deps in Chromium 2040bff2a904885249b1d58f3950b80ae45ebcbd.

Imports the service-workers/cache-storage as well as recent upstream changes.

BUG=666991,666993,432746,667376
==========

to

==========
Import wpt@5a1f188e0536ad023936cc62f9a00617299dc192

Using update-w3c-deps in Chromium 2040bff2a904885249b1d58f3950b80ae45ebcbd.

Imports the service-workers/cache-storage as well as recent upstream changes.

BUG=666991,666993,432746,667376
==========

[commit-bot: I haz the power, 2016-11-21 21:12:07]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-11-21 21:16:24]

Description was changed from

==========
Import wpt@5a1f188e0536ad023936cc62f9a00617299dc192

Using update-w3c-deps in Chromium 2040bff2a904885249b1d58f3950b80ae45ebcbd.

Imports the service-workers/cache-storage as well as recent upstream changes.

BUG=666991,666993,432746,667376
==========

to

==========
Import wpt@5a1f188e0536ad023936cc62f9a00617299dc192

Using update-w3c-deps in Chromium 2040bff2a904885249b1d58f3950b80ae45ebcbd.

Imports the service-workers/cache-storage as well as recent upstream changes.

BUG=666991,666993,432746,667376

Committed: https://crrev.com/80cc8049a86123976cc7255f2bcd689322f596bf
Cr-Commit-Position: refs/heads/master@{#433651}
==========

[commit-bot: I haz the power, 2016-11-21 21:16:25]

Patchset 3 (id:??) landed as
https://crrev.com/80cc8049a86123976cc7255f2bcd689322f596bf
Cr-Commit-Position: refs/heads/master@{#433651}