Removes WebContentsViewMus/RenderWidgetHostViewMus

Removes WebContentsViewMus/RenderWidgetHostViewMus

I updated the aura-mus classes to support mus.

BUG=674575
TEST=none
R=ben@chromium.org, sadrul@chromium.org

Committed: https://crrev.com/52a39e34c68e7e37138c666eccdd836a1162415a
Cr-Commit-Position: refs/heads/master@{#439344}

[sadrul, 2016-12-16 01:56:20]

Nice! lgtm (I am still reviewing the first one)

https://codereview.chromium.org/2577023003/diff/20001/content/browser/rendere...
File content/browser/renderer_host/render_widget_host_view_aura.cc (right):

https://codereview.chromium.org/2577023003/diff/20001/content/browser/rendere...
content/browser/renderer_host/render_widget_host_view_aura.cc:373:
DelegatedFrameHostClientImpl(
explicit

https://codereview.chromium.org/2577023003/diff/20001/content/browser/rendere...
content/browser/renderer_host/render_widget_host_view_aura.cc:1716:
*transformed_point =
We should ideally not even reach here in mus (since this is used for
hit-testing, and in mus, we will not be doing hit-testing in chrome browser).

https://codereview.chromium.org/2577023003/diff/20001/content/browser/rendere...
content/browser/renderer_host/render_widget_host_view_aura.cc:1758: if
(delegated_frame_host_ &&
ditto

[sky, 2016-12-16 22:30:52]

https://codereview.chromium.org/2577023003/diff/20001/content/browser/rendere...
File content/browser/renderer_host/render_widget_host_view_aura.cc (right):

https://codereview.chromium.org/2577023003/diff/20001/content/browser/rendere...
content/browser/renderer_host/render_widget_host_view_aura.cc:373:
DelegatedFrameHostClientImpl(
On 2016/12/16 01:56:20, sadrul wrote:
> explicit

Done.

https://codereview.chromium.org/2577023003/diff/20001/content/browser/rendere...
content/browser/renderer_host/render_widget_host_view_aura.cc:1716:
*transformed_point =
On 2016/12/16 01:56:20, sadrul wrote:
> We should ideally not even reach here in mus (since this is used for
> hit-testing, and in mus, we will not be doing hit-testing in chrome browser).

I added a TODO.

https://codereview.chromium.org/2577023003/diff/20001/content/browser/rendere...
content/browser/renderer_host/render_widget_host_view_aura.cc:1758: if
(delegated_frame_host_ &&
On 2016/12/16 01:56:20, sadrul wrote:
> ditto

TODO here too.

[sky, 2016-12-16 22:40:59]

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

[commit-bot: I haz the power, 2016-12-16 22:41:36]

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

[Ben Goodger (Google), 2016-12-16 22:42:30]

lgtm

[sky, 2016-12-16 22:44:06]

The CQ bit was unchecked by sky@chromium.org

[sky, 2016-12-16 22:44:11]

The CQ bit was checked by sky@chromium.org

[sky, 2016-12-16 22:44:12]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/2577023003/#ps60001
(title: "merge")

[commit-bot: I haz the power, 2016-12-16 22:45: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, 2016-12-16 23:15:47]

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

[commit-bot: I haz the power, 2016-12-16 23:15:48]

Try jobs failed on following builders:
  linux_chromium_asan_rel_ng on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  win_chromium_x64_rel_ng on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win_chromium_x64_...)

[sky, 2016-12-17 00:42:02]

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

[commit-bot: I haz the power, 2016-12-17 00:42:52]

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, 2016-12-17 02:11:14]

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

[commit-bot: I haz the power, 2016-12-17 02:11:14]

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

[sky, 2016-12-17 02:43:27]

I had to move the DelegatedFrameHostClient into its own header because a test
was subclassing the functions. Take another look?

[Ben Goodger (Google), 2016-12-17 04:51:02]

lgtm

[sky, 2016-12-17 16:52:32]

The CQ bit was checked by sky@chromium.org

[sky, 2016-12-17 16:52:33]

The patchset sent to the CQ was uploaded after l-g-t-m from sadrul@chromium.org
Link to the patchset: https://codereview.chromium.org/2577023003/#ps80001
(title: "fix test")

[commit-bot: I haz the power, 2016-12-17 16:52:38]

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, 2016-12-17 17:23:25]

CQ is committing da patch.

Bot data: {"patchset_id": 80001, "attempt_start_ts": 1481993552657790,
"parent_rev": "acc4a2a4a7dfbaea3b07912c1430065155673959", "commit_rev":
"c8d3efd3d2f1667fb26790100bbb04f6d9f309e3"}

[commit-bot: I haz the power, 2016-12-17 17:23:52]

Description was changed from

==========
Removes WebContentsViewMus/RenderWidgetHostViewMus

I updated the aura-mus classes to support mus.

BUG=674575
TEST=none
R=ben@chromium.org, sadrul@chromium.org
==========

to

==========
Removes WebContentsViewMus/RenderWidgetHostViewMus

I updated the aura-mus classes to support mus.

BUG=674575
TEST=none
R=ben@chromium.org, sadrul@chromium.org

Review-Url: https://codereview.chromium.org/2577023003
==========

[commit-bot: I haz the power, 2016-12-17 17:23:53]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-12-17 17:26:25]

Description was changed from

==========
Removes WebContentsViewMus/RenderWidgetHostViewMus

I updated the aura-mus classes to support mus.

BUG=674575
TEST=none
R=ben@chromium.org, sadrul@chromium.org

Review-Url: https://codereview.chromium.org/2577023003
==========

to

==========
Removes WebContentsViewMus/RenderWidgetHostViewMus

I updated the aura-mus classes to support mus.

BUG=674575
TEST=none
R=ben@chromium.org, sadrul@chromium.org

Committed: https://crrev.com/52a39e34c68e7e37138c666eccdd836a1162415a
Cr-Commit-Position: refs/heads/master@{#439344}
==========

[commit-bot: I haz the power, 2016-12-17 17:26:26]

Patchset 5 (id:??) landed as
https://crrev.com/52a39e34c68e7e37138c666eccdd836a1162415a
Cr-Commit-Position: refs/heads/master@{#439344}