Eliminate FrameMojoShell.

Eliminate FrameMojoShell.

R=rockot@chromium.org
BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/dd157e1ae8d937ab67ad8550f83b53bfc93947cc
Cr-Commit-Position: refs/heads/master@{#407010}

[Ben Goodger (Google), 2016-07-21 19:39:39]

Description was changed from

==========
Eliminate FrameMojoShell.

R=rockot@chromium.org
BUG=
==========

to

==========
Eliminate FrameMojoShell.

R=rockot@chromium.org
BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

[Ben Goodger (Google), 2016-07-21 20:36:22]

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

[commit-bot: I haz the power, 2016-07-21 20:36:37]

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-07-21 20:48:00]

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

[commit-bot: I haz the power, 2016-07-21 20:48:18]

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-07-21 21:02:43]

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

[commit-bot: I haz the power, 2016-07-21 21:02:44]

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

[Ben Goodger (Google), 2016-07-21 21:05:17]

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

[commit-bot: I haz the power, 2016-07-21 21:06:10]

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-07-21 21:16:07]

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

[commit-bot: I haz the power, 2016-07-21 21:16:20]

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-07-21 21:56:13]

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

[commit-bot: I haz the power, 2016-07-21 21:56:14]

Dry run: Try jobs failed on following builders:
  win8_chromium_gyp_rel on master.tryserver.chromium.win (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.win/builders/win8_chromium_gyp...)

[Ben Goodger (Google), 2016-07-21 22:19:50]

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

[commit-bot: I haz the power, 2016-07-21 22:20:42]

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-07-21 22:21:33]

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

[commit-bot: I haz the power, 2016-07-21 22:21:49]

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

[Ken Rockot(use gerrit already), 2016-07-21 22:34:33]

lgtm

https://codereview.chromium.org/2166953006/diff/110001/content/browser/frame_...
File content/browser/frame_host/render_frame_host_impl.cc (right):

https://codereview.chromium.org/2166953006/diff/110001/content/browser/frame_...
content/browser/frame_host/render_frame_host_impl.cc:2108:
base::Unretained(GetProcess()->GetBrowserContext()),
nit: You don't need base::Unretained when not binding the |this| argument of a
class method. So you can just Bind(&Create...,
GetProcess()->GetBrowserContext(), this);

https://codereview.chromium.org/2166953006/diff/110001/content/browser/mojo/m...
File content/browser/mojo/mojo_shell_context.cc (left):

https://codereview.chromium.org/2166953006/diff/110001/content/browser/mojo/m...
content/browser/mojo/mojo_shell_context.cc:177: class MojoShellContext::Proxy {
\o/

[Ben Goodger (Google), 2016-07-21 22:39:03]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-07-21 22:39:04]

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

[commit-bot: I haz the power, 2016-07-21 22:39:38]

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

[Ben Goodger (Google), 2016-07-21 22:56:02]

The CQ bit was checked by ben@chromium.org

[Ben Goodger (Google), 2016-07-21 22:56:02]

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

[commit-bot: I haz the power, 2016-07-21 22:56:30]

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-07-22 03:16:13]

Committed patchset #9 (id:150001)

[commit-bot: I haz the power, 2016-07-22 03:17:43]

Description was changed from

==========
Eliminate FrameMojoShell.

R=rockot@chromium.org
BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation
==========

to

==========
Eliminate FrameMojoShell.

R=rockot@chromium.org
BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_site_isolation

Committed: https://crrev.com/dd157e1ae8d937ab67ad8550f83b53bfc93947cc
Cr-Commit-Position: refs/heads/master@{#407010}
==========

[commit-bot: I haz the power, 2016-07-22 03:17:44]

Patchset 9 (id:??) landed as
https://crrev.com/dd157e1ae8d937ab67ad8550f83b53bfc93947cc
Cr-Commit-Position: refs/heads/master@{#407010}