Import web-platform-tests@5a8700479d98852455bee6117558897867eb278a

Import web-platform-tests@5a8700479d98852455bee6117558897867eb278a

Using update-w3c-deps in Blink 7bebfa461eab56d6e2f78d8c42a2130ab538c305.

After the import, some paths in TestExpectations needed to be updated
(due to structural changes in web-platform-tests/mediacapture-streams),
plus there are some new and updated expected failures and timeouts.

BUG=600261
Committed: https://crrev.com/d90c1eafcbf95049f1abb81c7df0377a0a822747
Cr-Commit-Position: refs/heads/master@{#385996}

[suzyh_UTC10 (ex-contributor), 2016-04-04 08:24:53]

suzyh@chromium.org changed reviewers:
+ dpranke@chromium.org, jsbell@chromium.org

[suzyh_UTC10 (ex-contributor), 2016-04-04 08:30:45]

Hi Dirk & Joshua,

I'm not sure of the protocols for updating test expectations after syncing the
W3C web-platform-tests, so let me know if there is a better approach than I've
done here.

The manual changes in this patch (i.e. those that are not just due to syncing
web-platform-tests) are changes to LayoutTests/TestExpectations and updates to 
- imported/web-platform-tests/dom/interfaces-expected.txt
- imported/web-platform-tests/dom/nodes/Element-classlist-expected.txt
- imported/web-platform-tests/dom/nodes/Node-replaceChild-expected.txt
- imported/web-platform-tests/dom/nodes/attributes-expected.txt

Thanks,
Suzy

[suzyh_UTC10 (ex-contributor), 2016-04-04 08:30:55]

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

[commit-bot: I haz the power, 2016-04-04 08:31:13]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1854003004/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1854003004/1

[tkent, 2016-04-04 08:48:49]

tkent@chromium.org changed reviewers:
+ tkent@chromium.org

[tkent, 2016-04-04 08:48:51]

https://codereview.chromium.org/1854003004/diff/1/third_party/WebKit/LayoutTe...
File third_party/WebKit/LayoutTests/TestExpectations (right):

https://codereview.chromium.org/1854003004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/TestExpectations:1373: crbug.com/600261
imported/web-platform-tests/dom/events/Event-dispatch-bubbles-false.html [
Failure ]
For wpt/dom and wpt/html, please check in failing -expected.txt unless it
contains local file paths.

[commit-bot: I haz the power, 2016-04-04 09:18:08]

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

[commit-bot: I haz the power, 2016-04-04 09:18:10]

Dry run: Try jobs failed on following builders:
  linux_blink_oilpan_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_blink_oil...)

[jsbell, 2016-04-04 20:19:42]

https://codereview.chromium.org/1854003004/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/imported/web-platform-tests/IndexedDB/idbcursor-direction-index-keyrange.htm
(left):

https://codereview.chromium.org/1854003004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/imported/web-platform-tests/IndexedDB/idbcursor-direction-index-keyrange.htm:2:
<meta charset="utf-8">
Do you happen to know why this line went away? Were we injecting it until
recently?

(Just asking because I don't see the file having been modified upstream and I'm
not seeing a change in our import scripts, but obviously one of those changed
and I'm missing it, so I feel dumb!)

https://codereview.chromium.org/1854003004/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/imported/web-platform-tests/html/semantics/scripting-1/the-script-element/external-script-windows1250.js
(right):

https://codereview.chromium.org/1854003004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/imported/web-platform-tests/html/semantics/scripting-1/the-script-element/external-script-windows1250.js:3:
return "œæ¹¿Ÿ"; //<- these are five Polish letters, similar to scazz. It can be
read correctly only with windows 1250 encoding.
Watch out - since this file is not UTF-8 it may require landing it manually.

(If so, ideally land this file first manually then the rest of the CL via the
CQ)

[jsbell, 2016-04-04 20:46:56]

https://codereview.chromium.org/1854003004/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/imported/web-platform-tests/IndexedDB/idbcursor-direction-index-keyrange.htm
(left):

https://codereview.chromium.org/1854003004/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/imported/web-platform-tests/IndexedDB/idbcursor-direction-index-keyrange.htm:2:
<meta charset="utf-8">
On 2016/04/04 20:19:41, jsbell wrote:
> Do you happen to know why this line went away? Were we injecting it until
> recently?
> 
> (Just asking because I don't see the file having been modified upstream and
I'm
> not seeing a change in our import scripts, but obviously one of those changed
> and I'm missing it, so I feel dumb!)

Huh, and I just saw https://github.com/w3c/web-platform-tests/issues/2747
appear. I'm probably behind on email or code reviews or something...

[Dirk Pranke, 2016-04-04 21:06:38]

I defer to tkent@ and jsbell@ here.

[suzyh_UTC10 (ex-contributor), 2016-04-04 23:53:06]

On 2016/04/04 at 08:48:51, tkent wrote:
>
https://codereview.chromium.org/1854003004/diff/1/third_party/WebKit/LayoutTe...
> File third_party/WebKit/LayoutTests/TestExpectations (right):
> 
>
https://codereview.chromium.org/1854003004/diff/1/third_party/WebKit/LayoutTe...
> third_party/WebKit/LayoutTests/TestExpectations:1373: crbug.com/600261
imported/web-platform-tests/dom/events/Event-dispatch-bubbles-false.html [
Failure ]
> For wpt/dom and wpt/html, please check in failing -expected.txt unless it
contains local file paths.

Done.

[suzyh_UTC10 (ex-contributor), 2016-04-04 23:54:47]

suzyh@chromium.org changed reviewers:
- dpranke@chromium.org

[suzyh_UTC10 (ex-contributor), 2016-04-04 23:55:50]

On 2016/04/04 at 20:46:56, jsbell wrote:
>
https://codereview.chromium.org/1854003004/diff/1/third_party/WebKit/LayoutTe...
> File
third_party/WebKit/LayoutTests/imported/web-platform-tests/IndexedDB/idbcursor-direction-index-keyrange.htm
(left):
> 
>
https://codereview.chromium.org/1854003004/diff/1/third_party/WebKit/LayoutTe...
>
third_party/WebKit/LayoutTests/imported/web-platform-tests/IndexedDB/idbcursor-direction-index-keyrange.htm:2:
<meta charset="utf-8">
> On 2016/04/04 20:19:41, jsbell wrote:
> > Do you happen to know why this line went away? Were we injecting it until
> > recently?
> > 
> > (Just asking because I don't see the file having been modified upstream and
I'm
> > not seeing a change in our import scripts, but obviously one of those
changed
> > and I'm missing it, so I feel dumb!)
> 
> Huh, and I just saw https://github.com/w3c/web-platform-tests/issues/2747
appear. I'm probably behind on email or code reviews or something...

Yeah, looks like https://codereview.chromium.org/1854503002 added it.

+jshin FYI

[suzyh_UTC10 (ex-contributor), 2016-04-04 23:58:47]

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

[commit-bot: I haz the power, 2016-04-04 23:59:46]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1854003004/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1854003004/20001

[commit-bot: I haz the power, 2016-04-05 00:05:10]

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

[commit-bot: I haz the power, 2016-04-05 00:05:12]

Dry run: Try jobs failed on following builders:
  android_chromium_gn_compile_rel on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_chro...)
  android_clang_dbg_recipe on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  ios_dbg_simulator_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)

[jsbell, 2016-04-05 00:14:27]

lgtm, assuming (as tkent requested) the new [Failure] entries correspond to
failures we can't *-expect.txt and yet we don't want to skip the import (via
W3CImportExpectations)

Undoing jshin's <meta charset=utf-8> add should be safe since those weren't
there previously, they're not actually needed for these tests, and he's pursuing
it upstream.

Spot check elsewhere looks good.

[suzyh_UTC10 (ex-contributor), 2016-04-05 00:35:46]

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

[commit-bot: I haz the power, 2016-04-05 00:36:05]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1854003004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1854003004/40001

[commit-bot: I haz the power, 2016-04-05 01:49:07]

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

[commit-bot: I haz the power, 2016-04-05 01:49:08]

Dry run: Try jobs failed on following builders:
  win_chromium_rel_ng on tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_rel_...)

[suzyh_UTC10 (ex-contributor), 2016-04-06 04:53:53]

Description was changed from

==========
Import web-platform-tests@5a8700479d98852455bee6117558897867eb278a

Using update-w3c-deps in Blink 7bebfa461eab56d6e2f78d8c42a2130ab538c305.

After the import, some paths in TestExpectations needed to be updated
(due to structural changes in web-platform-tests/mediacapture-streams),
plus there are some new failures to ignore.

BUG=600261
==========

to

==========
Import web-platform-tests@5a8700479d98852455bee6117558897867eb278a

Using update-w3c-deps in Blink 7bebfa461eab56d6e2f78d8c42a2130ab538c305.

After the import, some paths in TestExpectations needed to be updated
(due to structural changes in web-platform-tests/mediacapture-streams),
plus there are some new and updated expected failures and timeouts.

BUG=600261
==========

[suzyh_UTC10 (ex-contributor), 2016-04-06 04:55:53]

tkent: PTAL

Aside from the changes caused by the update-w3c-deps script itself, this patch
now includes the following changes:

Existing TestExpectations entries for
imported/web-platform-tests/mediacapture-streams
- 8 updated to new paths
- 2 deleted
(due to upstream moves/deletions)

New TestExpectations entries
- imported/web-platform-tests/mediacapture-streams/GUM-deny.html [ Failure ]
-
imported/web-platform-tests/mediacapture-streams/MediaDevices-getUserMedia.html
[ Failure ]
-
imported/web-platform-tests/mediacapture-streams/MediaStream-MediaElement-srcObject.html
[ Failure ]
- imported/web-platform-tests/dom/nodes/Node-isEqualNode-xhtml.xhtml [ Timeout
Failure ]
-
imported/web-platform-tests/html/semantics/forms/form-submission-0/getactionurl.html
[ Timeout Failure ]

New -expected files
-
imported/web-platform-tests/dom/events/Event-dispatch-bubbles-false-expected.txt
- imported/web-platform-tests/dom/lists/DOMTokenList-value-expected.txt
-
imported/web-platform-tests/html/semantics/interactive-elements/the-details-element/toggleEvent-expected.txt
-
imported/web-platform-tests/html/semantics/scripting-1/the-script-element/script-charset-01-expected.txt
-
imported/web-platform-tests/html/semantics/scripting-1/the-script-element/script-charset-02-expected.txt
-
imported/web-platform-tests/html/webappapis/scripting/events/event-handler-processing-algorithm-expected.txt

Updated -expected files
- imported/web-platform-tests/dom/interfaces-expected.txt
- imported/web-platform-tests/dom/nodes/Element-classlist-expected.txt
- imported/web-platform-tests/dom/nodes/Node-replaceChild-expected.txt
- imported/web-platform-tests/dom/nodes/attributes-expected.txt

[suzyh_UTC10 (ex-contributor), 2016-04-06 04:56:06]

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

[commit-bot: I haz the power, 2016-04-06 04:56:37]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1854003004/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1854003004/60001

[commit-bot: I haz the power, 2016-04-06 05:01:03]

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

[commit-bot: I haz the power, 2016-04-06 05:01:06]

Dry run: Try jobs failed on following builders:
  cast_shell_linux on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/cast_shell_linu...)
  chromeos_x86-generic_chromium_compile_only_ng on tryserver.chromium.linux
(JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromeos_x86-ge...)
  linux_chromium_chromeos_ozone_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_chromeos_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_compile_dbg_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_gn_chromeos_rel on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  linux_chromium_rel_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_rel_device_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_gn...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)
  mac_chromium_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[suzyh_UTC10 (ex-contributor), 2016-04-06 05:12:55]

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

[commit-bot: I haz the power, 2016-04-06 05:13:33]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1854003004/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1854003004/80001

[commit-bot: I haz the power, 2016-04-06 06:16:06]

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

[commit-bot: I haz the power, 2016-04-06 06:16:08]

Dry run: This issue passed the CQ dry run.

[tkent, 2016-04-07 23:44:42]

https://codereview.chromium.org/1854003004/diff/80001/third_party/WebKit/Layo...
File third_party/WebKit/LayoutTests/TestExpectations (right):

https://codereview.chromium.org/1854003004/diff/80001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/TestExpectations:1371: crbug.com/600261
imported/web-platform-tests/dom/nodes/Node-isEqualNode-xhtml.xhtml [ Timeout
Failure ]
This test requires extra XML files. So we should skip importing this by adding
this to W3CImportExpectations at crbug.com/490940 section.

https://codereview.chromium.org/1854003004/diff/80001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/TestExpectations:1372: crbug.com/600261
imported/web-platform-tests/html/semantics/forms/form-submission-0/getactionurl.html
[ Timeout Failure ]
This test requires /common/.  So we should skip importing this by adding this to
W3CImportExpectations at crbug.com/360762 section.

[suzyh_UTC10 (ex-contributor), 2016-04-08 03:09:26]

Thanks!

On 2016/04/07 at 23:44:42, tkent wrote:
>
https://codereview.chromium.org/1854003004/diff/80001/third_party/WebKit/Layo...
> File third_party/WebKit/LayoutTests/TestExpectations (right):
> 
>
https://codereview.chromium.org/1854003004/diff/80001/third_party/WebKit/Layo...
> third_party/WebKit/LayoutTests/TestExpectations:1371: crbug.com/600261
imported/web-platform-tests/dom/nodes/Node-isEqualNode-xhtml.xhtml [ Timeout
Failure ]
> This test requires extra XML files. So we should skip importing this by adding
this to W3CImportExpectations at crbug.com/490940 section.

Turns out this was already in W3CImportExpectations, but the filename had
changed, so I've updated that line.


>
https://codereview.chromium.org/1854003004/diff/80001/third_party/WebKit/Layo...
> third_party/WebKit/LayoutTests/TestExpectations:1372: crbug.com/600261
imported/web-platform-tests/html/semantics/forms/form-submission-0/getactionurl.html
[ Timeout Failure ]
> This test requires /common/.  So we should skip importing this by adding this
to W3CImportExpectations at crbug.com/360762 section.

Done.

[suzyh_UTC10 (ex-contributor), 2016-04-08 03:09:43]

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

[commit-bot: I haz the power, 2016-04-08 03:10:02]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1854003004/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1854003004/100001

[tkent, 2016-04-08 03:23:06]

lgtm

[commit-bot: I haz the power, 2016-04-08 04:18:14]

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

[commit-bot: I haz the power, 2016-04-08 04:18:16]

Dry run: This issue passed the CQ dry run.

[suzyh_UTC10 (ex-contributor), 2016-04-08 04:19:21]

The CQ bit was checked by suzyh@chromium.org

[suzyh_UTC10 (ex-contributor), 2016-04-08 04:19:22]

The patchset sent to the CQ was uploaded after l-g-t-m from jsbell@chromium.org
Link to the patchset: https://codereview.chromium.org/1854003004/#ps100001
(title: "Skip two tests using W3CImportExpectations instead")

[commit-bot: I haz the power, 2016-04-08 04:19:41]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1854003004/100001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1854003004/100001

[suzyh_UTC10 (ex-contributor), 2016-04-08 04:28:31]

The CQ bit was checked by suzyh@chromium.org

[suzyh_UTC10 (ex-contributor), 2016-04-08 04:28:32]

The patchset sent to the CQ was uploaded after l-g-t-m from tkent@chromium.org,
jsbell@chromium.org
Link to the patchset: https://codereview.chromium.org/1854003004/#ps120001
(title: "Rebase")

[commit-bot: I haz the power, 2016-04-08 04:28:49]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1854003004/120001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1854003004/120001

[commit-bot: I haz the power, 2016-04-08 05:43:54]

Committed patchset #7 (id:120001)

[commit-bot: I haz the power, 2016-04-08 05:44:48]

Description was changed from

==========
Import web-platform-tests@5a8700479d98852455bee6117558897867eb278a

Using update-w3c-deps in Blink 7bebfa461eab56d6e2f78d8c42a2130ab538c305.

After the import, some paths in TestExpectations needed to be updated
(due to structural changes in web-platform-tests/mediacapture-streams),
plus there are some new and updated expected failures and timeouts.

BUG=600261
==========

to

==========
Import web-platform-tests@5a8700479d98852455bee6117558897867eb278a

Using update-w3c-deps in Blink 7bebfa461eab56d6e2f78d8c42a2130ab538c305.

After the import, some paths in TestExpectations needed to be updated
(due to structural changes in web-platform-tests/mediacapture-streams),
plus there are some new and updated expected failures and timeouts.

BUG=600261

Committed: https://crrev.com/d90c1eafcbf95049f1abb81c7df0377a0a822747
Cr-Commit-Position: refs/heads/master@{#385996}
==========

[commit-bot: I haz the power, 2016-04-08 05:44:51]

Patchset 7 (id:??) landed as
https://crrev.com/d90c1eafcbf95049f1abb81c7df0377a0a822747
Cr-Commit-Position: refs/heads/master@{#385996}