Add PlatformLocalFrame class.

Add PlatformLocalFrame class.

PlatformLocalFrame is used in PlatformChromeClient as the platform
base class for core/frame/LocalFrame

This removes a reference to FrameViewBase which is being deleted
in http://crrev.com/2855523002.

BUG=637460

[joelhockey, 2017-05-10 00:16:53]

Description was changed from

==========
Add PlatformLocalFrame class.

PlatformLocalFrame is used in PlatformChromeClient as the platform
base class for core/frame/LocalFrame

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

to

==========
Add PlatformLocalFrame class.

PlatformLocalFrame is used in PlatformChromeClient as the platform
base class for core/frame/LocalFrame

This removes a reference to FrameViewBase which is being deleted
in http://crrev.com/2855523002.

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

[joelhockey, 2017-05-10 00:17:01]

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

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

Dry run: CQ is trying da patch.

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

[joelhockey, 2017-05-10 00:18:49]

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

[joelhockey, 2017-05-10 00:18:50]

I have moved part of http://crrev.com/2855523002 into this separate CL.

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

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

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

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...)
  android_clang_dbg_recipe on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_clan...)
  android_compile_dbg on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/android_comp...)
  cast_shell_android on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/cast_shell_a...)
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[joelhockey, 2017-05-10 00:25:56]

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

[joelhockey, 2017-05-10 00:27:07]

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

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

Dry run: CQ is trying da patch.

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

[dcheng, 2017-05-10 00:44:36]

I'd prefer to keep plumbing this through as a FrameView rather than the Frame,
if possible.

In the long-run, I'm hoping we can expose WebWidgetClient directly so we don't
even need to plumb it through ChromeClient, but that's some time away.

[haraken, 2017-05-10 01:08:46]

On 2017/05/10 00:44:36, dcheng wrote:
> I'd prefer to keep plumbing this through as a FrameView rather than the Frame,
> if possible.
> 
> In the long-run, I'm hoping we can expose WebWidgetClient directly so we don't
> even need to plumb it through ChromeClient, but that's some time away.

Yeah, agreed. What we need here would be a base class in platform/ that handles
layout-related operations in a Frame. It is a responsibility of FrameView rather
than LocalFrame.

[joelhockey, 2017-05-10 01:33:26]

On 2017/05/10 at 01:08:46, haraken wrote:
> On 2017/05/10 00:44:36, dcheng wrote:
> > I'd prefer to keep plumbing this through as a FrameView rather than the
Frame,
> > if possible.
> > 
> > In the long-run, I'm hoping we can expose WebWidgetClient directly so we
don't
> > even need to plumb it through ChromeClient, but that's some time away.
> 
> Yeah, agreed. What we need here would be a base class in platform/ that
handles layout-related operations in a Frame. It is a responsibility of
FrameView rather than LocalFrame.

I'll close this CL.  See http://crrev.com/2873213002