Early out PrePaintTreeWalk::walk() earlier

Early out PrePaintTreeWalk::walk() earlier

We should have paint invalidation flags set when auxiliary object
properties and/or paint offset will change (otherwise we will not
reach some of such objects because of the early return of the upper
level), so they can be below the early return.

BUG=685179
TEST=all existing tests pass
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
R=chrishtr@chromium.org

Review-Url: https://codereview.chromium.org/2694193004 .
Cr-Commit-Position: refs/heads/master@{#450884}
Committed: https://chromium.googlesource.com/chromium/src/+/5b35a06876222a78be4405d86e712fb17bf6fa8b

[Xianzhu, 2017-02-14 20:01:23]

Description was changed from

==========
Early out PrePaintTreeWalk::walk() earlier

We should have paint invalidation flags set when auxiliary object
properties and/or paint offset will change (otherwise we will not
reach some of such objects because of the early return of the upper
level), so they can be below the early return.
==========

to

==========
Early out PrePaintTreeWalk::walk() earlier

We should have paint invalidation flags set when auxiliary object
properties and/or paint offset will change (otherwise we will not
reach some of such objects because of the early return of the upper
level), so they can be below the early return.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Xianzhu, 2017-02-14 21:35:52]

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

[commit-bot: I haz the power, 2017-02-14 21:36:31]

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-02-14 21:41:57]

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

[commit-bot: I haz the power, 2017-02-14 21:41:58]

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...)

[Xianzhu, 2017-02-14 23:15:59]

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

[commit-bot: I haz the power, 2017-02-14 23:16:50]

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

[Xianzhu, 2017-02-14 23:17:10]

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

[commit-bot: I haz the power, 2017-02-14 23:17:27]

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

[Xianzhu, 2017-02-14 23:26:17]

Description was changed from

==========
Early out PrePaintTreeWalk::walk() earlier

We should have paint invalidation flags set when auxiliary object
properties and/or paint offset will change (otherwise we will not
reach some of such objects because of the early return of the upper
level), so they can be below the early return.
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Early out PrePaintTreeWalk::walk() earlier

We should have paint invalidation flags set when auxiliary object
properties and/or paint offset will change (otherwise we will not
reach some of such objects because of the early return of the upper
level), so they can be below the early return.

BUG=685179
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Xianzhu, 2017-02-14 23:53:29]

Description was changed from

==========
Early out PrePaintTreeWalk::walk() earlier

We should have paint invalidation flags set when auxiliary object
properties and/or paint offset will change (otherwise we will not
reach some of such objects because of the early return of the upper
level), so they can be below the early return.

BUG=685179
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Early out PrePaintTreeWalk::walk() earlier

We should have paint invalidation flags set when auxiliary object
properties and/or paint offset will change (otherwise we will not
reach some of such objects because of the early return of the upper
level), so they can be below the early return.

BUG=685179
TEST=all existing tests pass
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[Xianzhu, 2017-02-14 23:53:45]

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

[commit-bot: I haz the power, 2017-02-15 01:22:03]

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

[commit-bot: I haz the power, 2017-02-15 01:22:04]

Dry run: This issue passed the CQ dry run.

[chrishtr, 2017-02-15 02:05:27]

lgtm

Please wait for pdr's review also though.

[Xianzhu, 2017-02-16 05:14:21]

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

[commit-bot: I haz the power, 2017-02-16 05:15:33]

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-02-16 06:58:11]

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

[commit-bot: I haz the power, 2017-02-16 06:58:12]

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

[Xianzhu, 2017-02-16 07:05:02]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2017-02-16 07:05:02]

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

[commit-bot: I haz the power, 2017-02-16 07:05:38]

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

[Xianzhu, 2017-02-16 07:08:32]

Description was changed from

==========
Early out PrePaintTreeWalk::walk() earlier

We should have paint invalidation flags set when auxiliary object
properties and/or paint offset will change (otherwise we will not
reach some of such objects because of the early return of the upper
level), so they can be below the early return.

BUG=685179
TEST=all existing tests pass
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Early out PrePaintTreeWalk::walk() earlier

We should have paint invalidation flags set when auxiliary object
properties and/or paint offset will change (otherwise we will not
reach some of such objects because of the early return of the upper
level), so they can be below the early return.

BUG=685179
TEST=all existing tests pass
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
R=chrishtr@chromium.org

Review-Url: https://codereview.chromium.org/2694193004 .
Cr-Commit-Position: refs/heads/master@{#450884}
Committed:
https://chromium.googlesource.com/chromium/src/+/5b35a06876222a78be4405d86e71...
==========

[Xianzhu, 2017-02-16 07:08:33]

Committed patchset #4 (id:60001) manually as
5b35a06876222a78be4405d86e712fb17bf6fa8b (presubmit successful).