Import the baseURI test from web-platform-tests

Import the baseURI test from web-platform-tests

Import web-platform-tests@0bb3d73f26aa5a21326b6c1c7aaf35029222223f

Using update-w3c-deps in Blink 731a1238920ae9b7216cd2ced12267deb0d63e19.

This fails, but is imported so that it can be fixed.

BUG=570679
Committed: https://crrev.com/504d39f51972ae114ddfb97656ca0a009b911d16
Cr-Commit-Position: refs/heads/master@{#365828}

[philipj_slow, 2015-12-16 21:45:41]

The CQ bit was checked by philipj@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2015-12-16 21:46:25]

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

[commit-bot: I haz the power, 2015-12-16 22:44:34]

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

[commit-bot: I haz the power, 2015-12-16 22:44:34]

Dry run: This issue passed the CQ dry run.

[philipj_slow, 2015-12-17 15:19:01]

Description was changed from

==========
Import the baseURI test from web-platform-tests

Import web-platform-tests@0bb3d73f26aa5a21326b6c1c7aaf35029222223f

Using update-w3c-deps in Blink 731a1238920ae9b7216cd2ced12267deb0d63e19.

This fails, but is imported so that it can be fixed.

BUG=484930
==========

to

==========
Import the baseURI test from web-platform-tests

Import web-platform-tests@0bb3d73f26aa5a21326b6c1c7aaf35029222223f

Using update-w3c-deps in Blink 731a1238920ae9b7216cd2ced12267deb0d63e19.

This fails, but is imported so that it can be fixed.

BUG=570679
==========

[philipj_slow, 2015-12-17 15:21:27]

philipj@opera.com changed reviewers:
+ fs@opera.com, tkent@chromium.org

[philipj_slow, 2015-12-17 15:21:28]

PTAL

[philipj_slow, 2015-12-17 15:22:02]

Next CL is https://codereview.chromium.org/1529363004/

[philipj_slow, 2015-12-17 15:31:24]

The way this test fails is:

This is a testharness.js-based test.
PASS For elements belonging to document, baseURI should be document url 
FAIL For elements unassigned to document, baseURI should be document url
assert_equals: expected (string)
"file:///home/philip/src/chromium/src/third_party/WebKit/LayoutTests/imported/web-platform-tests/dom/nodes/Node-baseURI.html"
but got (object) null
FAIL For elements belonging to document fragments, baseURI should be document
url assert_equals: expected (string)
"file:///home/philip/src/chromium/src/third_party/WebKit/LayoutTests/imported/web-platform-tests/dom/nodes/Node-baseURI.html"
but got (object) null
PASS After inserting fragment into document, element baseURI should be document
url 
Harness: the test ran to completion.

In other words, it's the two tests that check that the node document is used
instead of walking the ancestor chain, which is the bug to fix.

This had to be skipped before the failure output includes the local path.

[fs, 2015-12-17 15:33:54]

LGTM

On 2015/12/17 at 15:22:02, philipj wrote:
> Next CL is https://codereview.chromium.org/1529363004/

Why not land that first then?

[philipj_slow, 2015-12-17 15:35:42]

On 2015/12/17 15:33:54, fs wrote:
> LGTM
> 
> On 2015/12/17 at 15:22:02, philipj wrote:
> > Next CL is https://codereview.chromium.org/1529363004/
> 
> Why not land that first then?

The test imported here is the test coverage for the changes made in the other
CL. They could have been combined, I guess, but in case of revert it seems
better that the wpt import is untouched.

I'll go ahead and land this so that the other can run the bots, and wait for
tkent's review.

[philipj_slow, 2015-12-17 15:35:51]

The CQ bit was checked by philipj@opera.com

[commit-bot: I haz the power, 2015-12-17 15:36:26]

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

[commit-bot: I haz the power, 2015-12-17 15:40:47]

Description was changed from

==========
Import the baseURI test from web-platform-tests

Import web-platform-tests@0bb3d73f26aa5a21326b6c1c7aaf35029222223f

Using update-w3c-deps in Blink 731a1238920ae9b7216cd2ced12267deb0d63e19.

This fails, but is imported so that it can be fixed.

BUG=570679
==========

to

==========
Import the baseURI test from web-platform-tests

Import web-platform-tests@0bb3d73f26aa5a21326b6c1c7aaf35029222223f

Using update-w3c-deps in Blink 731a1238920ae9b7216cd2ced12267deb0d63e19.

This fails, but is imported so that it can be fixed.

BUG=570679
==========

[commit-bot: I haz the power, 2015-12-17 15:40:47]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2015-12-17 15:42:02]

Description was changed from

==========
Import the baseURI test from web-platform-tests

Import web-platform-tests@0bb3d73f26aa5a21326b6c1c7aaf35029222223f

Using update-w3c-deps in Blink 731a1238920ae9b7216cd2ced12267deb0d63e19.

This fails, but is imported so that it can be fixed.

BUG=570679
==========

to

==========
Import the baseURI test from web-platform-tests

Import web-platform-tests@0bb3d73f26aa5a21326b6c1c7aaf35029222223f

Using update-w3c-deps in Blink 731a1238920ae9b7216cd2ced12267deb0d63e19.

This fails, but is imported so that it can be fixed.

BUG=570679

Committed: https://crrev.com/504d39f51972ae114ddfb97656ca0a009b911d16
Cr-Commit-Position: refs/heads/master@{#365828}
==========

[commit-bot: I haz the power, 2015-12-17 15:42:03]

Patchset 1 (id:??) landed as
https://crrev.com/504d39f51972ae114ddfb97656ca0a009b911d16
Cr-Commit-Position: refs/heads/master@{#365828}