Make EventHandlerRegistry store Page instead of FrameHost

Make EventHandlerRegistry store Page instead of FrameHost

This is pre-work to move FrameHost::m_eventHandlerRegistry to Page. Also
renamed didMove{Into,OutOf}FrameHost to didMove{Into,OutOf}Page, which
now makes more sense.

BUG=691794
Review-Url: https://codereview.chromium.org/2736753003
Cr-Commit-Position: refs/heads/master@{#455379}
Committed: https://chromium.googlesource.com/chromium/src/+/5c4056a61c3d24b5df4ebdba406467b96ff68866

[sashab, 2017-03-07 05:24:28]

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

[commit-bot: I haz the power, 2017-03-07 05:24:57]

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

[sashab, 2017-03-07 05:25:00]

Description was changed from

==========
Make EventHandlerRegistry store Page instead of FrameHost

BUG=691794
==========

to

==========
Make EventHandlerRegistry store Page instead of FrameHost

This is pre-work to move FrameHost::m_eventHandlerRegistry to Page.

BUG=691794
==========

[sashab, 2017-03-07 05:25:07]

sashab@chromium.org changed reviewers:
+ joelhockey@chromium.org

[sashab, 2017-03-07 05:25:51]

Description was changed from

==========
Make EventHandlerRegistry store Page instead of FrameHost

This is pre-work to move FrameHost::m_eventHandlerRegistry to Page.

BUG=691794
==========

to

==========
Make EventHandlerRegistry store Page instead of FrameHost

This is pre-work to move FrameHost::m_eventHandlerRegistry to Page. Also
renamed didMove{Into,OutOf}FrameHost to didMove{Into,OutOf}Page, which
now makes more sense.

BUG=691794
==========

[joelhockey, 2017-03-07 05:43:22]

lgtm

[haraken, 2017-03-07 06:30:48]

LGTM

[commit-bot: I haz the power, 2017-03-07 06:58:02]

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

[commit-bot: I haz the power, 2017-03-07 06:58:03]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[sashab, 2017-03-08 00:59:35]

The CQ bit was checked by sashab@chromium.org

[commit-bot: I haz the power, 2017-03-08 01:00:51]

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-03-08 04:33:21]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1488934775323900, "parent_rev":
"f386bcddbe277ef1cf0845124146948afb251ee2", "commit_rev":
"5c4056a61c3d24b5df4ebdba406467b96ff68866"}

[commit-bot: I haz the power, 2017-03-08 04:34:23]

Description was changed from

==========
Make EventHandlerRegistry store Page instead of FrameHost

This is pre-work to move FrameHost::m_eventHandlerRegistry to Page. Also
renamed didMove{Into,OutOf}FrameHost to didMove{Into,OutOf}Page, which
now makes more sense.

BUG=691794
==========

to

==========
Make EventHandlerRegistry store Page instead of FrameHost

This is pre-work to move FrameHost::m_eventHandlerRegistry to Page. Also
renamed didMove{Into,OutOf}FrameHost to didMove{Into,OutOf}Page, which
now makes more sense.

BUG=691794

Review-Url: https://codereview.chromium.org/2736753003
Cr-Commit-Position: refs/heads/master@{#455379}
Committed:
https://chromium.googlesource.com/chromium/src/+/5c4056a61c3d24b5df4ebdba4064...
==========

[commit-bot: I haz the power, 2017-03-08 04:34:24]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/5c4056a61c3d24b5df4ebdba4064...

[Peter Mayo, 2017-03-08 16:00:07]

On 2017/03/08 04:34:24, commit-bot: I haz the power wrote:
> Committed patchset #1 (id:1) as
>
https://chromium.googlesource.com/chromium/src/+/5c4056a61c3d24b5df4ebdba4064...

This looks like it has caused a unit test failure on Windows.

[Peter Mayo, 2017-03-08 16:01:05]

On 2017/03/08 16:00:07, Peter Mayo wrote:
> On 2017/03/08 04:34:24, commit-bot: I haz the power wrote:
> > Committed patchset #1 (id:1) as
> >
>
https://chromium.googlesource.com/chromium/src/+/5c4056a61c3d24b5df4ebdba4064...
> 
> This looks like it has caused a unit test failure on Windows.

See https://bugs.chromium.org/p/chromium/issues/detail?id=699573

[Peter Mayo, 2017-03-08 16:04:49]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2737863005/ by petermayo@chromium.org.

The reason for reverting is:
virtual/gpu-rasterization/images/cross-fade-background-size.html failed after
this change.

No idea why, perhaps it is too sensitive, or needs a concurrent rebaseline.

Tracked in bug 699573.