WPT tests of allow-top-navigation-by-user-activation for iframe@sandbox.

WPT tests for allow-top-navigation-by-user-activation for iframe@sandbox.

Move the test files for disallowing top navigation given no gesture to WPT, and also adding a manual test for allowing top navigation with user gesture.

BUG=662506
Review-Url: https://codereview.chromium.org/2711073002
Cr-Commit-Position: refs/heads/master@{#452609}
Committed: https://chromium.googlesource.com/chromium/src/+/290032568531b964b232611699a358900bdb789f

[Bin Lu, 2017-02-23 05:18:24]

Description was changed from

==========
WPT tests for allow-top-navigation-by-user-activation for iframe@sandbox.

Move the test files for disallowing top navigation given no gesture to WPT, and
also adding a manual test for allowing top navigation with user gesture.

BUG=662506
==========

to

==========
WPT tests for allow-top-navigation-by-user-activation for iframe@sandbox.

Move the test files for disallowing top navigation given no gesture to WPT, and
also adding a manual test for allowing top navigation with user gesture.

BUG=662506
==========

[Bin Lu, 2017-02-23 05:18:24]

binlu@google.com changed reviewers:
+ domenic@chromium.org, ojan@chromium.org, rbyers@chromium.org

[Bin Lu, 2017-02-23 06:38:54]

The CQ bit was checked by binlu@google.com to run a CQ dry run

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

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

[Bin Lu, 2017-02-23 06:40:19]

Does this look good? Thanks.

[commit-bot: I haz the power, 2017-02-23 07:40:16]

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

[commit-bot: I haz the power, 2017-02-23 07:40:17]

Dry run: This issue passed the CQ dry run.

[domenic, 2017-02-23 19:01:05]

lgtm!

[Bin Lu, 2017-02-23 20:00:53]

The CQ bit was checked by binlu@google.com

[commit-bot: I haz the power, 2017-02-23 20:01:32]

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-23 20:01:33]

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

[commit-bot: I haz the power, 2017-02-23 20:01:34]

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[Rick Byers, 2017-02-23 20:25:31]

On 2017/02/23 20:01:34, commit-bot: I haz the power wrote:
> No L-G-T-M from a valid reviewer yet. 
> CQ run can only be started by full committers or once the patch has
> received an L-G-T-M from a full committer.
> Even if an L-G-T-M may have been provided, it was from a non-committer,
> _not_ a full super star committer.
> Committers are members of the group "project-chromium-committers".
> Note that this has nothing to do with OWNERS files.

What?  Domenic isn't a committer??
 
Anyway RS LGTM.

Note that WPT-only changes like this might be sligtly more convenient to do
directly in GitHub (you get some additional tooling benefit, avoid our stupid
manifest.json issue, and domenic definitely has push access there), but
whichever workflow you find most convenient is fine.

[Rick Byers, 2017-02-23 20:26:14]

On 2017/02/23 20:25:31, Rick Byers wrote:
> On 2017/02/23 20:01:34, commit-bot: I haz the power wrote:
> > No L-G-T-M from a valid reviewer yet. 
> > CQ run can only be started by full committers or once the patch has
> > received an L-G-T-M from a full committer.
> > Even if an L-G-T-M may have been provided, it was from a non-committer,
> > _not_ a full super star committer.
> > Committers are members of the group "project-chromium-committers".
> > Note that this has nothing to do with OWNERS files.
> 
> What?  Domenic isn't a committer??
>  
> Anyway RS LGTM.
> 
> Note that WPT-only changes like this might be sligtly more convenient to do
> directly in GitHub (you get some additional tooling benefit, avoid our stupid
> manifest.json issue, and domenic definitely has push access there), but
> whichever workflow you find most convenient is fine.

Oh whoops, I see this also deletes the blink LayoutTest, so it's not "WPT only".
 Makes sense...

[Rick Byers, 2017-02-23 20:26:21]

The CQ bit was checked by rbyers@chromium.org

[commit-bot: I haz the power, 2017-02-23 20:26:42]

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-23 20:35:53]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1487881581956720,
"parent_rev": "316805c14bf56b93f6251983af8f84a18470a0b8", "commit_rev":
"290032568531b964b232611699a358900bdb789f"}

[commit-bot: I haz the power, 2017-02-23 20:36:31]

Description was changed from

==========
WPT tests for allow-top-navigation-by-user-activation for iframe@sandbox.

Move the test files for disallowing top navigation given no gesture to WPT, and
also adding a manual test for allowing top navigation with user gesture.

BUG=662506
==========

to

==========
WPT tests for allow-top-navigation-by-user-activation for iframe@sandbox.

Move the test files for disallowing top navigation given no gesture to WPT, and
also adding a manual test for allowing top navigation with user gesture.

BUG=662506

Review-Url: https://codereview.chromium.org/2711073002
Cr-Commit-Position: refs/heads/master@{#452609}
Committed:
https://chromium.googlesource.com/chromium/src/+/290032568531b964b232611699a3...
==========

[commit-bot: I haz the power, 2017-02-23 20:36:32]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/290032568531b964b232611699a3...