W3C Test: Import web-platform-tests/html/{iana,infrastructure}

W3C Test: Import web-platform-tests/html/{iana,infrastructure}

> update-w3c-deps import using blink 0a262a62e3c061286f29aef2115df7d08ef73f77:
>
> imported csswg-test@0a262a62e3c061286f29aef2115df7d08ef73f77
> imported web-platform-tests@0a262a62e3c061286f29aef2115df7d08ef73f77

BUG=482822
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=196262

[tkent, 2015-06-01 04:10:20]

Patchset #1 (id:1) has been deleted

[tkent, 2015-06-01 07:08:50]

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

[tkent, 2015-06-01 07:08:50]

jsbell or dpranke, please review this.

[jsbell, 2015-06-01 17:51:13]

lgtm

https://codereview.chromium.org/1157773008/diff/20001/LayoutTests/W3CImportEx...
File LayoutTests/W3CImportExpectations (right):

https://codereview.chromium.org/1157773008/diff/20001/LayoutTests/W3CImportEx...
LayoutTests/W3CImportExpectations:212: # crbug.com/493465: Requires the pipe
feature of wptserve
If this comment only applies to the following line, can you add a blank line to
separate the other tests and add comment for the reason(s) they don't work?

(If this applies to all three lines below, it's redundant with line 210?)

https://codereview.chromium.org/1157773008/diff/20001/LayoutTests/imported/we...
File
LayoutTests/imported/web-platform-tests/html/infrastructure/conformance-requirements/extensibility/foreign-expected.txt
(right):

https://codereview.chromium.org/1157773008/diff/20001/LayoutTests/imported/we...
LayoutTests/imported/web-platform-tests/html/infrastructure/conformance-requirements/extensibility/foreign-expected.txt:1:
Running test...
Can we tweak testharnessreport.js to suppress this so we don't need an
expected.txt file?

(I haven't looked at the test itself)

https://codereview.chromium.org/1157773008/diff/20001/LayoutTests/imported/we...
File
LayoutTests/imported/web-platform-tests/html/infrastructure/terminology/plugins/text-plain-expected.txt
(right):

https://codereview.chromium.org/1157773008/diff/20001/LayoutTests/imported/we...
LayoutTests/imported/web-platform-tests/html/infrastructure/terminology/plugins/text-plain-expected.txt:1:
Running test...
Ditto.

[tkent, 2015-06-01 23:30:45]

https://codereview.chromium.org/1157773008/diff/20001/LayoutTests/W3CImportEx...
File LayoutTests/W3CImportExpectations (right):

https://codereview.chromium.org/1157773008/diff/20001/LayoutTests/W3CImportEx...
LayoutTests/W3CImportExpectations:212: # crbug.com/493465: Requires the pipe
feature of wptserve
On 2015/06/01 17:51:12, jsbell wrote:
> If this comment only applies to the following line, can you add a blank line
to
> separate the other tests and add comment for the reason(s) they don't work?
> 
> (If this applies to all three lines below, it's redundant with line 210?)

Done.
window-security.sub.html has two problems.

https://codereview.chromium.org/1157773008/diff/20001/LayoutTests/imported/we...
File
LayoutTests/imported/web-platform-tests/html/infrastructure/conformance-requirements/extensibility/foreign-expected.txt
(right):

https://codereview.chromium.org/1157773008/diff/20001/LayoutTests/imported/we...
LayoutTests/imported/web-platform-tests/html/infrastructure/conformance-requirements/extensibility/foreign-expected.txt:1:
Running test...
On 2015/06/01 17:51:13, jsbell wrote:
> Can we tweak testharnessreport.js to suppress this so we don't need an
> expected.txt file?
> 
> (I haven't looked at the test itself)

Let me address this in a separated CL.
We need to hide <div id=log> too.

[tkent, 2015-06-01 23:32:29]

The CQ bit was checked by tkent@chromium.org

[tkent, 2015-06-01 23:32:30]

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/1157773008/#ps40001
(title: "W3CImportExpectations glitch")

[commit-bot: I haz the power, 2015-06-01 23:34:23]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1157773008/40001

[commit-bot: I haz the power, 2015-06-02 00:42:19]

Committed patchset #2 (id:40001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=196262