Remove apparently redundant uses of updatePageOverlays in WebViewImpl.

Remove apparently redundant uses of updatePageOverlays in WebViewImpl.
BUG=591468
Committed: https://crrev.com/f66f629cc6107328e6a445d158e7a27fa91668df
Cr-Commit-Position: refs/heads/master@{#411792}

[chrishtr, 2016-08-12 19:42:28]

Description was changed from

==========
none

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

to

==========
Clean up use of updatePageOverlays in WebViewImpl.
==========

[chrishtr, 2016-08-12 19:42:43]

Description was changed from

==========
Clean up use of updatePageOverlays in WebViewImpl.
==========

to

==========
Clean up use of updatePageOverlays in WebViewImpl.

BUG=591468
==========

[chrishtr, 2016-08-12 19:42:48]

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

[commit-bot: I haz the power, 2016-08-12 19:43:12]

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

[chrishtr, 2016-08-12 20:39:16]

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

[chrishtr, 2016-08-12 20:39:22]

chrishtr@chromium.org changed reviewers:
+ wangxianzhu@chromium.org

[commit-bot: I haz the power, 2016-08-12 20:39:39]

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

[Xianzhu, 2016-08-12 20:49:08]

Is there any behavior change or is this a pure cleanup? If the latter, do all
removed invocations have equivalent new code path, or were some of them
unnecessary?

[chrishtr, 2016-08-12 20:51:45]

On 2016/08/12 at 20:49:08, wangxianzhu wrote:
> Is there any behavior change or is this a pure cleanup? If the latter, do all
removed invocations have equivalent new code path, or were some of them
unnecessary?

It's not pure cleanup. I think the other call sites were unnecessary.

[Xianzhu, 2016-08-12 20:55:07]

lgtm.

Please add some descriptions.

[chrishtr, 2016-08-12 20:56:48]

Description was changed from

==========
Clean up use of updatePageOverlays in WebViewImpl.

BUG=591468
==========

to

==========
Remove apparently redundant uses of updatePageOverlays in WebViewImpl.
BUG=591468
==========

[chrishtr, 2016-08-12 20:56:54]

The CQ bit was unchecked by chrishtr@chromium.org

[chrishtr, 2016-08-12 20:56:59]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2016-08-12 20:57:06]

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-08-12 21:59:39]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-12 22:02:33]

Description was changed from

==========
Remove apparently redundant uses of updatePageOverlays in WebViewImpl.
BUG=591468
==========

to

==========
Remove apparently redundant uses of updatePageOverlays in WebViewImpl.
BUG=591468

Committed: https://crrev.com/f66f629cc6107328e6a445d158e7a27fa91668df
Cr-Commit-Position: refs/heads/master@{#411792}
==========

[commit-bot: I haz the power, 2016-08-12 22:02:34]

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