Update page scale factor when layout updates

Update page scale factor when layout updates

Page scale factor is updated when a main frame is committed. But that is
not enough because when WebView is invisible, there won't be main frame
commits.

In order to get page scale factor updates even when WebView is
invisible, check page scale factor change when the layout is updated.

BUG=520089
Committed: https://crrev.com/f4e647c7be4db5bbea755042ee965b7accd2adf3
Cr-Commit-Position: refs/heads/master@{#351116}

[hush (inactive), 2015-09-26 01:53:51]

hush@chromium.org changed reviewers:
+ boliu@chromium.org

[hush (inactive), 2015-09-26 01:53:52]

PTAL

[hush (inactive), 2015-09-26 02:13:11]

On 2015/09/26 01:53:52, hush wrote:
> PTAL

Per offline discussion, we can explore the possibility of fetching the page
scale factor from the compositor frame meta data. That will be done later

[boliu, 2015-09-28 16:33:06]

https://codereview.chromium.org/1368193002/diff/1/android_webview/renderer/aw...
File android_webview/renderer/aw_render_view_ext.cc (right):

https://codereview.chromium.org/1368193002/diff/1/android_webview/renderer/aw...
android_webview/renderer/aw_render_view_ext.cc:191: CheckContentsSizeAndScale();
This implementation should be indentical to DidUpdateLayout, ie also check the
timer.

[hush (inactive), 2015-09-28 18:21:40]

https://codereview.chromium.org/1368193002/diff/1/android_webview/renderer/aw...
File android_webview/renderer/aw_render_view_ext.cc (right):

https://codereview.chromium.org/1368193002/diff/1/android_webview/renderer/aw...
android_webview/renderer/aw_render_view_ext.cc:191: CheckContentsSizeAndScale();
On 2015/09/28 16:33:05, boliu wrote:
> This implementation should be indentical to DidUpdateLayout, ie also check the
> timer.

Done. I don't know why it was added though. It seems like a way to spam the
browser process with IPC less often.

[boliu, 2015-09-28 18:30:47]

lgtm % code dedup

https://codereview.chromium.org/1368193002/diff/20001/android_webview/rendere...
File android_webview/renderer/aw_render_view_ext.cc (right):

https://codereview.chromium.org/1368193002/diff/20001/android_webview/rendere...
android_webview/renderer/aw_render_view_ext.cc:191: if
(check_contents_size_timer_.IsRunning())
no code duplication so close! factor out into a private method

[hush (inactive), 2015-09-28 18:34:40]

https://codereview.chromium.org/1368193002/diff/20001/android_webview/rendere...
File android_webview/renderer/aw_render_view_ext.cc (right):

https://codereview.chromium.org/1368193002/diff/20001/android_webview/rendere...
android_webview/renderer/aw_render_view_ext.cc:191: if
(check_contents_size_timer_.IsRunning())
On 2015/09/28 18:30:47, boliu wrote:
> no code duplication so close! factor out into a private method

Done.

[hush (inactive), 2015-09-28 18:34:43]

The CQ bit was checked by hush@chromium.org

[hush (inactive), 2015-09-28 18:34:43]

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

[commit-bot: I haz the power, 2015-09-28 18:42:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1368193002/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1368193002/40001

[commit-bot: I haz the power, 2015-09-28 19:33:38]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-09-28 19:34:54]

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