Enable new wpt/referrer-policy tests.

Enable new wpt/referrer-policy tests.

R=jochen@chromium.org, estark@chromium.org
BUG=692105
Review-Url: https://codereview.chromium.org/2693703007
Cr-Commit-Position: refs/heads/master@{#451331}
Committed: https://chromium.googlesource.com/chromium/src/+/13e532004647b3d9fefe0c3643d3741a3c53184e

[qyearsley, 2017-02-14 23:22:52]

Looks like many of the referrer-policy tests fail when run with
run-webkit-tests; is this expected; does this look like it reflects the real
behavior of Blink, or might something be wrong with how the tests are run?

[qyearsley, 2017-02-14 23:23:07]

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

[commit-bot: I haz the power, 2017-02-14 23:24:06]

Dry run: 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-02-15 00:49:27]

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

[commit-bot: I haz the power, 2017-02-15 00:49:29]

Dry run: This issue passed the CQ dry run.

[jochen (gone - plz use gerrit), 2017-02-15 09:53:50]

Maybe just skip
external/wpt/referrer-policy/{strict-origin,strict-origin-when-cross-origin,same-origin}
referencing issue 627968 - those policies aren't implemented yet

I'll investigate origin-when-cross-origin

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

https://codereview.chromium.org/2693703007/diff/20001/third_party/WebKit/Layo...
third_party/WebKit/LayoutTests/TestExpectations:1855: crbug.com/626703
external/wpt/referrer-policy/generic/subresource-test/image-decoding.html [
Timeout ]
both fixed with this PR: https://github.com/w3c/web-platform-tests/pull/4823

[jochen (gone - plz use gerrit), 2017-02-15 09:57:10]

oh, the other tests are also testing the same-origin policy.

So I guess skipping
external/referrer-policy/origin-when-cross-origin/http-rp/same-origin as well
makes sense

for the first failure, I got a fix in the CQ that should land really soon now

[qyearsley, 2017-02-15 18:59:08]

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

[commit-bot: I haz the power, 2017-02-15 18:59:59]

Dry run: 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-02-15 20:15:52]

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

[commit-bot: I haz the power, 2017-02-15 20:15:53]

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

[qyearsley, 2017-02-15 22:39:04]

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

[commit-bot: I haz the power, 2017-02-15 22:39:58]

Dry run: 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-02-15 23:54:20]

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

[commit-bot: I haz the power, 2017-02-15 23:54:21]

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

[qyearsley, 2017-02-17 16:17:38]

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

[commit-bot: I haz the power, 2017-02-17 16:18:00]

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

[qyearsley, 2017-02-17 17:12:21]

On 2017/02/15 at 09:57:10, jochen wrote:
> oh, the other tests are also testing the same-origin policy.
> 
> So I guess skipping
external/referrer-policy/origin-when-cross-origin/http-rp/same-origin as well
makes sense
> 
> for the first failure, I got a fix in the CQ that should land really soon now

Alright! Does this CL look OK to commit now, with the skips for those four
sub-directories of referrer-policy?

[jochen (gone - plz use gerrit), 2017-02-17 17:17:48]

lgtm

[qyearsley, 2017-02-17 17:21:37]

The CQ bit was unchecked by qyearsley@chromium.org

[qyearsley, 2017-02-17 17:21:39]

The CQ bit was checked by qyearsley@chromium.org

[commit-bot: I haz the power, 2017-02-17 17:21:52]

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-02-17 17:31:31]

CQ is committing da patch.

Bot data: {"patchset_id": 110001, "attempt_start_ts": 1487352099303170,
"parent_rev": "87dd40c9b8ae1fb8c3eb15bf527ee464bd1c1bdd", "commit_rev":
"13e532004647b3d9fefe0c3643d3741a3c53184e"}

[commit-bot: I haz the power, 2017-02-17 17:33:18]

Description was changed from

==========
Enable new wpt/referrer-policy tests.

R=jochen@chromium.org, estark@chromium.org
BUG=692105
==========

to

==========
Enable new wpt/referrer-policy tests.

R=jochen@chromium.org, estark@chromium.org
BUG=692105

Review-Url: https://codereview.chromium.org/2693703007
Cr-Commit-Position: refs/heads/master@{#451331}
Committed:
https://chromium.googlesource.com/chromium/src/+/13e532004647b3d9fefe0c3643d3...
==========

[commit-bot: I haz the power, 2017-02-17 17:33:20]

Committed patchset #7 (id:110001) as
https://chromium.googlesource.com/chromium/src/+/13e532004647b3d9fefe0c3643d3...