Refactor WebView resize and update-after-layout methods

Refactor WebView resize and update-after-layout methods

This changes some method names to make them more accurate, and splits
out ResizeAfterLayout from LayoutUpdated.  In a subsequent patch, these
two methods will be called independently through ChromeClient.

BUG=701575
Review-Url: https://codereview.chromium.org/2848043002
Cr-Commit-Position: refs/heads/master@{#468236}
Committed: https://chromium.googlesource.com/chromium/src/+/b6c07cb297c05e54ce3865a63b9325fdd8222017

[szager1, 2017-04-28 18:19:20]

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

[commit-bot: I haz the power, 2017-04-28 18:20:26]

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, 2017-04-28 20:04:56]

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

[commit-bot: I haz the power, 2017-04-28 20:04:57]

Dry run: This issue passed the CQ dry run.

[szager1, 2017-04-28 20:14:32]

Description was changed from

==========
Refactor WebView resize and update-after-layout methods

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

to

==========
Refactor WebView resize and update-after-layout methods

This changes some method names to make them more accurate, and splits
out ResizeAfterLayout from LayoutUpdated.  In a subsequent patch, these
two methods will be called independently through ChromeClient.

BUG=701575
==========

[szager1, 2017-04-28 20:14:32]

szager@chromium.org changed reviewers:
+ pdr@chromium.org, skobes@chromium.org

[szager1, 2017-04-28 20:14:47]

pdr@ for web/OWNERS

[szager1, 2017-04-28 20:37:39]

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

[commit-bot: I haz the power, 2017-04-28 20:38:46]

Dry run: CQ is trying da patch.

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

[skobes, 2017-04-28 20:52:13]

lgtm w/ nit

https://codereview.chromium.org/2848043002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/web/WebViewImpl.cpp (right):

https://codereview.chromium.org/2848043002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/web/WebViewImpl.cpp:3702: FrameView* view =
webframe->GetFrame()->View();
nit: this could just be frame->View()

[pdr., 2017-04-28 21:29:21]

On 2017/04/28 at 20:52:13, skobes wrote:
> lgtm w/ nit
> 
>
https://codereview.chromium.org/2848043002/diff/20001/third_party/WebKit/Sour...
> File third_party/WebKit/Source/web/WebViewImpl.cpp (right):
> 
>
https://codereview.chromium.org/2848043002/diff/20001/third_party/WebKit/Sour...
> third_party/WebKit/Source/web/WebViewImpl.cpp:3702: FrameView* view =
webframe->GetFrame()->View();
> nit: this could just be frame->View()

LGTM

[commit-bot: I haz the power, 2017-04-28 22:22:13]

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

[commit-bot: I haz the power, 2017-04-28 22:22:14]

Dry run: This issue passed the CQ dry run.

[szager1, 2017-04-29 09:11:00]

https://codereview.chromium.org/2848043002/diff/20001/third_party/WebKit/Sour...
File third_party/WebKit/Source/web/WebViewImpl.cpp (right):

https://codereview.chromium.org/2848043002/diff/20001/third_party/WebKit/Sour...
third_party/WebKit/Source/web/WebViewImpl.cpp:3702: FrameView* view =
webframe->GetFrame()->View();
On 2017/04/28 20:52:13, skobes (OOO till 5-16) wrote:
> nit: this could just be frame->View()

Done.

[szager1, 2017-04-29 09:11:04]

The CQ bit was checked by szager@chromium.org

[szager1, 2017-04-29 09:11:04]

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

[commit-bot: I haz the power, 2017-04-29 09:11:13]

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-04-29 11:04:42]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1493457064405990,
"parent_rev": "ac1207df09b815b8ccc78dfead13410d2eefca68", "commit_rev":
"b6c07cb297c05e54ce3865a63b9325fdd8222017"}

[commit-bot: I haz the power, 2017-04-29 11:04:58]

Description was changed from

==========
Refactor WebView resize and update-after-layout methods

This changes some method names to make them more accurate, and splits
out ResizeAfterLayout from LayoutUpdated.  In a subsequent patch, these
two methods will be called independently through ChromeClient.

BUG=701575
==========

to

==========
Refactor WebView resize and update-after-layout methods

This changes some method names to make them more accurate, and splits
out ResizeAfterLayout from LayoutUpdated.  In a subsequent patch, these
two methods will be called independently through ChromeClient.

BUG=701575

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

[commit-bot: I haz the power, 2017-04-29 11:04:58]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/src/+/b6c07cb297c05e54ce3865a63b93...