Remove FrameView::Root

Remove FrameView::Root

BUG=637460
Review-Url: https://codereview.chromium.org/2849383002
Cr-Original-Commit-Position: refs/heads/master@{#468588}
Committed: https://chromium.googlesource.com/chromium/src/+/561e38ff8dc24f23f0293c54777bbc2eda99a2b5
Review-Url: https://codereview.chromium.org/2849383002
Cr-Commit-Position: refs/heads/master@{#468817}
Committed: https://chromium.googlesource.com/chromium/src/+/95b2a74dcc579f928a8abb631a3ad00854924fd0

[joelhockey, 2017-05-02 04:57:52]

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

[commit-bot: I haz the power, 2017-05-02 04:58:07]

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-05-02 06:35:13]

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

[commit-bot: I haz the power, 2017-05-02 06:35:15]

Dry run: This issue passed the CQ dry run.

[joelhockey, 2017-05-02 07:31:44]

joelhockey@chromium.org changed reviewers:
+ dcheng@chromium.org, haraken@chromium.org

[joelhockey, 2017-05-02 07:31:44]

See patch 1 that I ran this testing that GetFrame()->LocalFrameRoot()->View is
same as Root().

[dcheng, 2017-05-02 07:38:57]

lgtm

[joelhockey, 2017-05-02 07:53:54]

The CQ bit was checked by joelhockey@chromium.org

[commit-bot: I haz the power, 2017-05-02 07:54:06]

CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[haraken, 2017-05-02 08:05:19]

LGTM

[commit-bot: I haz the power, 2017-05-02 09:38:47]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1493711634346770,
"parent_rev": "bed8311f2e1d29053f198a429b71653a6888088a", "commit_rev":
"561e38ff8dc24f23f0293c54777bbc2eda99a2b5"}

[commit-bot: I haz the power, 2017-05-02 09:39:01]

Description was changed from

==========
Remove FrameView::Root

BUG=637460
==========

to

==========
Remove FrameView::Root

BUG=637460

Review-Url: https://codereview.chromium.org/2849383002
Cr-Commit-Position: refs/heads/master@{#468588}
Committed:
https://chromium.googlesource.com/chromium/src/+/561e38ff8dc24f23f0293c54777b...
==========

[commit-bot: I haz the power, 2017-05-02 09:39:02]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/561e38ff8dc24f23f0293c54777b...

[vabr (Chromium), 2017-05-02 09:49:50]

A revert of this CL (patchset #2 id:20001) has been created in
https://codereview.chromium.org/2857523002/ by vabr@chromium.org.

The reason for reverting is: This broke compilation (details later).

Today's sheriff.

[findit-for-me, 2017-05-02 09:51:13]

Findit (https://goo.gl/kROfz5) confirmed this CL at revision 468588 as the
culprit for
failures in the build cycles as shown on:
https://findit-for-me.appspot.com/waterfall/culprit?key=ag9zfmZpbmRpdC1mb3Itb...

[joelhockey, 2017-05-02 10:01:41]

The CQ bit was checked by joelhockey@chromium.org

[joelhockey, 2017-05-02 10:01:41]

The patchset sent to the CQ was uploaded after l-g-t-m from
haraken@chromium.org, dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2849383002/#ps40001
(title: "merge")

[commit-bot: I haz the power, 2017-05-02 10:02:00]

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-05-02 11:33:27]

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

[commit-bot: I haz the power, 2017-05-02 11:33:31]

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_...)

[joelhockey, 2017-05-02 21:48:07]

The CQ bit was checked by joelhockey@chromium.org

[commit-bot: I haz the power, 2017-05-02 21:48: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-05-02 23:35:34]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1493761687712570,
"parent_rev": "63120471a1bf609a8fb4c37c4798db04eee980ef", "commit_rev":
"95b2a74dcc579f928a8abb631a3ad00854924fd0"}

[commit-bot: I haz the power, 2017-05-02 23:35:47]

Description was changed from

==========
Remove FrameView::Root

BUG=637460

Review-Url: https://codereview.chromium.org/2849383002
Cr-Commit-Position: refs/heads/master@{#468588}
Committed:
https://chromium.googlesource.com/chromium/src/+/561e38ff8dc24f23f0293c54777b...
==========

to

==========
Remove FrameView::Root

BUG=637460

Review-Url: https://codereview.chromium.org/2849383002
Cr-Original-Commit-Position: refs/heads/master@{#468588}
Committed:
https://chromium.googlesource.com/chromium/src/+/561e38ff8dc24f23f0293c54777b...
Review-Url: https://codereview.chromium.org/2849383002
Cr-Commit-Position: refs/heads/master@{#468817}
Committed:
https://chromium.googlesource.com/chromium/src/+/95b2a74dcc579f928a8abb631a3a...
==========

[commit-bot: I haz the power, 2017-05-02 23:35:49]

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