Start out deferring commits in WebViewImpl

Start out deferring commits in WebViewImpl.

This is a re-land of https://codereview.chromium.org/1934513004/.
The two bugs leading to revert of that patch seem to be either ok (crbug.com/609033),
or exhibit flashing even without the patch (crbug.com/608623).

BUG=21798
Review-Url: https://codereview.chromium.org/2802503002
Cr-Commit-Position: refs/heads/master@{#469095}
Committed: https://chromium.googlesource.com/chromium/src/+/94fdfaa5a34534601337ce1780eb8fe381f74c3b

[chrishtr, 2017-04-04 23:31:57]

Description was changed from

==========
none

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

to

==========
Start out deferring commits in WebViewImpl.

This is a re-land of https://codereview.chromium.org/1934513004/.
The two bugs leading to revert of that patch seem to be either ok
(crbug.com/609033,
or exhibit flashing even without the patch (crbug.com/608623).

BUG=21798
==========

[chrishtr, 2017-04-04 23:32:00]

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

[chrishtr, 2017-04-04 23:32:34]

chrishtr@chromium.org changed reviewers:
+ enne@chromium.org, treib@chromium.org

[commit-bot: I haz the power, 2017-04-04 23:33:23]

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

[enne (OOO), 2017-04-05 00:08:55]

On 2017/04/04 at 23:32:35, chrishtr wrote:
>

[enne (OOO), 2017-04-05 00:09:05]

lgtm!

[chrishtr, 2017-04-05 01:50:39]

The CQ bit was unchecked by chrishtr@chromium.org

[chrishtr, 2017-04-05 01:50:44]

The CQ bit was checked by chrishtr@chromium.org

[commit-bot: I haz the power, 2017-04-05 01:51: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-04-05 03:35:47]

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

[commit-bot: I haz the power, 2017-04-05 03:35:48]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[chrishtr, 2017-04-06 23:36:52]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2017-04-06 23:37:15]

Description was changed from

==========
Start out deferring commits in WebViewImpl.

This is a re-land of https://codereview.chromium.org/1934513004/.
The two bugs leading to revert of that patch seem to be either ok
(crbug.com/609033,
or exhibit flashing even without the patch (crbug.com/608623).

BUG=21798
==========

to

==========
Start out deferring commits in WebViewImpl.

This is a re-land of https://codereview.chromium.org/1934513004/.
The two bugs leading to revert of that patch seem to be either ok
(crbug.com/609033),
or exhibit flashing even without the patch (crbug.com/608623).

BUG=21798
==========

[commit-bot: I haz the power, 2017-04-06 23:37:39]

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-07 01:20:08]

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

[commit-bot: I haz the power, 2017-04-07 01:20:09]

Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[chrishtr, 2017-04-14 18:05:18]

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

[commit-bot: I haz the power, 2017-04-14 18:05:32]

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-14 18:08:56]

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

[commit-bot: I haz the power, 2017-04-14 18:08:57]

Dry run: Try jobs failed on following builders:
  chromium_presubmit on master.tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/chromium_presub...)

[chrishtr, 2017-04-14 18:17:40]

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

[commit-bot: I haz the power, 2017-04-14 18:17:56]

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-14 19:45:45]

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

[commit-bot: I haz the power, 2017-04-14 19:45:46]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[Marc Treib, 2017-05-03 12:55:18]

Ping! What's the status of this CL?

[chrishtr, 2017-05-03 15:10:56]

I still need to debug one reproducible failure for Android webview. Will get
this
committed in time for M60. If you have cycles to take it over and debug that
last
failure, that would be great also!

[Marc Treib, 2017-05-03 15:36:46]

On 2017/05/03 15:10:56, chrishtr wrote:
> I still need to debug one reproducible failure for Android webview. Will get
> this
> committed in time for M60. If you have cycles to take it over and debug that
> last
> failure, that would be great also!

Not sure I'll be able to find the time.. if I do, I'll let you know!

Anyway, M60 sounds perfect!

[chrishtr, 2017-05-03 18:13:38]

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

[commit-bot: I haz the power, 2017-05-03 18:14:43]

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-05-03 19:54:18]

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

[commit-bot: I haz the power, 2017-05-03 19:54:19]

Dry run: This issue passed the CQ dry run.

[chrishtr, 2017-05-03 20:01:01]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2017-05-03 20:01:01]

The patchset sent to the CQ was uploaded after l-g-t-m from enne@chromium.org
Link to the patchset: https://codereview.chromium.org/2802503002/#ps40001
(title: "Merge branch 'master' into defercommits")

[commit-bot: I haz the power, 2017-05-03 20:01:35]

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-03 20:05:45]

CQ is committing da patch.

Bot data: {"patchset_id": 40001, "attempt_start_ts": 1493841661481670,
"parent_rev": "008b9b947994dd0311a14015660df1b30126247d", "commit_rev":
"94fdfaa5a34534601337ce1780eb8fe381f74c3b"}

[commit-bot: I haz the power, 2017-05-03 20:05:58]

Description was changed from

==========
Start out deferring commits in WebViewImpl.

This is a re-land of https://codereview.chromium.org/1934513004/.
The two bugs leading to revert of that patch seem to be either ok
(crbug.com/609033),
or exhibit flashing even without the patch (crbug.com/608623).

BUG=21798
==========

to

==========
Start out deferring commits in WebViewImpl.

This is a re-land of https://codereview.chromium.org/1934513004/.
The two bugs leading to revert of that patch seem to be either ok
(crbug.com/609033),
or exhibit flashing even without the patch (crbug.com/608623).

BUG=21798

Review-Url: https://codereview.chromium.org/2802503002
Cr-Commit-Position: refs/heads/master@{#469095}
Committed:
https://chromium.googlesource.com/chromium/src/+/94fdfaa5a34534601337ce1780eb...
==========

[commit-bot: I haz the power, 2017-05-03 20:05:59]

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