Move more classes to WebLocalFrameBase instead of WebLocalFrameImpl.

Move more classes to WebLocalFrameBase instead of WebLocalFrameImpl.

This CL uses WebLocalFrameBase to break dependencies on WebLocalFrameImpl at the
web/ layer. In some case, methods have been made pure virtual in WebLocalFrameBase
and overriden in WebLocalFrameImpl to assist with the decoupling.

We need to break these dependencies so we can Move WebLocalFrameImpl to core.

This CL updates:
- TextFinder
- WebEmbeddedWorkerImpl
- WebFrameSerializer
- WebFrameSerializerImpl
- WebSharedWorkerImpl


BUG=708879
Review-Url: https://codereview.chromium.org/2881693002
Cr-Commit-Position: refs/heads/master@{#471308}
Committed: https://chromium.googlesource.com/chromium/src/+/ee9b642ef6c3c43f8342cd3522a1367b1a9dcd76

[slangley, 2017-05-12 04:19:25]

Description was changed from

==========
Move more classes to WebLocalFrameBase instead of WebLocalFrameImpl.

git cl format


Fix WebSharedWorkerImpl.


WebScopedUserGesture.


git cl format


Move some methods to WebLocalFrameBase.


Move WebFrameSerializerImpl to WebLocalFrameBase.


Move WebFrameSerializer.


Update WebEmbeddedWorkerImpl to WebLocalFrameBase.


Use WebLocalFrameBase for TextFinder.

BUG=
==========

to

==========
Move more classes to WebLocalFrameBase instead of WebLocalFrameImpl.

This CL uses WebLocalFrameBase to break dependencies on WebLocalFrameImpl at the
web/ layer. In some case, methods have been made pure virtual in
WebLocalFrameBase
and overriden in WebLocalFrameImpl to assist with the decoupling.

We need to break these dependencies so we can Move WebLocalFrameImpl to core.

This CL updates:
- TextFinder
- WebEmbeddedWorkerImpl
- WebFrameSerializer
- WebFrameSerializerImpl


BUG=708879
==========

[slangley, 2017-05-12 04:19:25]

slangley@chromium.org changed reviewers:
+ haraken@chromium.org, kinuko@chromium.org, tkent@chromium.org

[slangley, 2017-05-12 04:19:43]

Description was changed from

==========
Move more classes to WebLocalFrameBase instead of WebLocalFrameImpl.

This CL uses WebLocalFrameBase to break dependencies on WebLocalFrameImpl at the
web/ layer. In some case, methods have been made pure virtual in
WebLocalFrameBase
and overriden in WebLocalFrameImpl to assist with the decoupling.

We need to break these dependencies so we can Move WebLocalFrameImpl to core.

This CL updates:
- TextFinder
- WebEmbeddedWorkerImpl
- WebFrameSerializer
- WebFrameSerializerImpl


BUG=708879
==========

to

==========
Move more classes to WebLocalFrameBase instead of WebLocalFrameImpl.

This CL uses WebLocalFrameBase to break dependencies on WebLocalFrameImpl at the
web/ layer. In some case, methods have been made pure virtual in
WebLocalFrameBase
and overriden in WebLocalFrameImpl to assist with the decoupling.

We need to break these dependencies so we can Move WebLocalFrameImpl to core.

This CL updates:
- TextFinder
- WebEmbeddedWorkerImpl
- WebFrameSerializer
- WebFrameSerializerImpl
- WebSharedWorkerImpl


BUG=708879
==========

[haraken, 2017-05-12 04:37:07]

LGTM

[kinuko, 2017-05-12 04:58:00]

(lgtm/2)

[slangley, 2017-05-12 05:33:11]

The CQ bit was checked by slangley@chromium.org

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

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-12 06:32:35]

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

[commit-bot: I haz the power, 2017-05-12 06:32:37]

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

[slangley, 2017-05-12 10:36:49]

The CQ bit was checked by slangley@chromium.org

[commit-bot: I haz the power, 2017-05-12 10:37: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-12 15:37:50]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1494585409280260, "parent_rev":
"37f3bb60cad9165bae952b7f95746ecb03342ff3", "commit_rev":
"ee9b642ef6c3c43f8342cd3522a1367b1a9dcd76"}

[commit-bot: I haz the power, 2017-05-12 15:38:05]

Description was changed from

==========
Move more classes to WebLocalFrameBase instead of WebLocalFrameImpl.

This CL uses WebLocalFrameBase to break dependencies on WebLocalFrameImpl at the
web/ layer. In some case, methods have been made pure virtual in
WebLocalFrameBase
and overriden in WebLocalFrameImpl to assist with the decoupling.

We need to break these dependencies so we can Move WebLocalFrameImpl to core.

This CL updates:
- TextFinder
- WebEmbeddedWorkerImpl
- WebFrameSerializer
- WebFrameSerializerImpl
- WebSharedWorkerImpl


BUG=708879
==========

to

==========
Move more classes to WebLocalFrameBase instead of WebLocalFrameImpl.

This CL uses WebLocalFrameBase to break dependencies on WebLocalFrameImpl at the
web/ layer. In some case, methods have been made pure virtual in
WebLocalFrameBase
and overriden in WebLocalFrameImpl to assist with the decoupling.

We need to break these dependencies so we can Move WebLocalFrameImpl to core.

This CL updates:
- TextFinder
- WebEmbeddedWorkerImpl
- WebFrameSerializer
- WebFrameSerializerImpl
- WebSharedWorkerImpl


BUG=708879

Review-Url: https://codereview.chromium.org/2881693002
Cr-Commit-Position: refs/heads/master@{#471308}
Committed:
https://chromium.googlesource.com/chromium/src/+/ee9b642ef6c3c43f8342cd3522a1...
==========

[commit-bot: I haz the power, 2017-05-12 15:38:06]

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