Move scheduleAnimation calls from WebViewClient to WebWidgetClient.

Move scheduleAnimation calls from WebViewClient to WebWidgetClient.

This is part of a refactor to split WebView/WebWidget.

BUG=419087
Committed: https://crrev.com/12136c6df9020eb8c418597887c6769b3a454111
Cr-Commit-Position: refs/heads/master@{#406580}

[lfg, 2016-07-15 18:42:38]

lfg@chromium.org changed reviewers:
+ dcheng@chromium.org

[lfg, 2016-07-15 18:42:38]

PTAL.

[dcheng, 2016-07-18 05:51:05]

LGTM

[lfg, 2016-07-18 15:12:40]

lfg@chromium.org changed reviewers:
+ avi@chromium.org

[lfg, 2016-07-18 15:12:41]

+avi, please take a look.

[Avi (use Gerrit), 2016-07-18 15:15:58]

lgtm

https://codereview.chromium.org/2128563002/diff/40001/third_party/WebKit/publ...
File third_party/WebKit/public/web/WebViewClient.h (right):

https://codereview.chromium.org/2128563002/diff/40001/third_party/WebKit/publ...
third_party/WebKit/public/web/WebViewClient.h:279: WebWidgetClient*
widgetClient() { return this; }
Do you see this becoming a permanent call on WebViewClient?

[lfg, 2016-07-18 15:20:25]

https://codereview.chromium.org/2128563002/diff/40001/third_party/WebKit/publ...
File third_party/WebKit/public/web/WebViewClient.h (right):

https://codereview.chromium.org/2128563002/diff/40001/third_party/WebKit/publ...
third_party/WebKit/public/web/WebViewClient.h:279: WebWidgetClient*
widgetClient() { return this; }
On 2016/07/18 15:15:58, Avi wrote:
> Do you see this becoming a permanent call on WebViewClient?

No. See my comment here: https://codereview.chromium.org/2121373002/ . I hope we
can remove this as soon as possible, but it's important until WebView can
instantiate a WebWidget.

[lfg, 2016-07-18 16:44:36]

The CQ bit was checked by lfg@chromium.org

[commit-bot: I haz the power, 2016-07-18 16:45:03]

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

[lfg, 2016-07-18 16:45:30]

The CQ bit was unchecked by lfg@chromium.org

[lfg, 2016-07-20 14:30:32]

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

[commit-bot: I haz the power, 2016-07-20 14:31:07]

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

[lfg, 2016-07-20 15:30:14]

The CQ bit was unchecked by lfg@chromium.org

[lfg, 2016-07-20 15:30:14]

The CQ bit was checked by lfg@chromium.org

[lfg, 2016-07-20 15:30:15]

The patchset sent to the CQ was uploaded after l-g-t-m from avi@chromium.org,
dcheng@chromium.org
Link to the patchset: https://codereview.chromium.org/2128563002/#ps60001
(title: "rebase")

[commit-bot: I haz the power, 2016-07-20 15:30:34]

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-20 16:26:00]

Committed patchset #4 (id:60001)

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

CQ bit was unchecked.

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

Description was changed from

==========
Move scheduleAnimation calls from WebViewClient to WebWidgetClient.

This is part of a refactor to split WebView/WebWidget.

BUG=419087
==========

to

==========
Move scheduleAnimation calls from WebViewClient to WebWidgetClient.

This is part of a refactor to split WebView/WebWidget.

BUG=419087

Committed: https://crrev.com/12136c6df9020eb8c418597887c6769b3a454111
Cr-Commit-Position: refs/heads/master@{#406580}
==========

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

Patchset 4 (id:??) landed as
https://crrev.com/12136c6df9020eb8c418597887c6769b3a454111
Cr-Commit-Position: refs/heads/master@{#406580}