Fix PrePaint UMA when SlimmingPaintInvalidation is disabled

Fix PrePaint UMA when SlimmingPaintInvalidation is disabled

Previously UMA of PrePaint with SlimmingPaintInvalidation disabled
count twice for the old path and prePaint() (which skips the real
pre-paint tree walk and so added many near 0 results).

BUG=646176
Review-Url: https://codereview.chromium.org/2670013004
Cr-Commit-Position: refs/heads/master@{#448001}
Committed: https://chromium.googlesource.com/chromium/src/+/385db906ffa3937dab92438ee6fc43b6485514ae

[Xianzhu, 2017-02-03 05:13:40]

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

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

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-03 05:14:42]

Description was changed from

==========
Fix PrePaint UMA and unskip virtual/stable/paint/invalidation

Previously UMA of PrePaint with slimming-paint-invalidation disabled
count twice for the old path and prePaint() (which early returns so
added many near 0 results). Now move the check for spinvalidation
out from prePaint() into updateLifecyclePhasesInternal().

BUG=646176
==========

to

==========
Fix PrePaint UMA when SlimmingPaintInvalidation is disabled

Previously UMA of PrePaint with SlimmingPaintInvalidation disabled
count twice for the old path and prePaint() (which early returns so
added many near 0 results). Now move the check for spinvalidation
out from prePaint() into updateLifecyclePhasesInternal().

BUG=646176
==========

[Xianzhu, 2017-02-03 05:14:58]

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

[Xianzhu, 2017-02-03 06:25:28]

Description was changed from

==========
Fix PrePaint UMA when SlimmingPaintInvalidation is disabled

Previously UMA of PrePaint with SlimmingPaintInvalidation disabled
count twice for the old path and prePaint() (which early returns so
added many near 0 results). Now move the check for spinvalidation
out from prePaint() into updateLifecyclePhasesInternal().

BUG=646176
==========

to

==========
Fix PrePaint UMA when SlimmingPaintInvalidation is disabled

Previously UMA of PrePaint with SlimmingPaintInvalidation disabled
count twice for the old path and prePaint() (which skips the real
pre-paint tree walk and so added many near 0 results).

BUG=646176
==========

[Xianzhu, 2017-02-03 06:27:43]

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

[commit-bot: I haz the power, 2017-02-03 06:28:02]

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-03 09:42:49]

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

[commit-bot: I haz the power, 2017-02-03 09:42:50]

Dry run: This issue passed the CQ dry run.

[chrishtr, 2017-02-03 16:09:21]

The CQ bit was checked by chrishtr@chromium.org

[chrishtr, 2017-02-03 16:09:22]

lgtm

[commit-bot: I haz the power, 2017-02-03 16:09:43]

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-03 16:37:11]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486138161676020,
"parent_rev": "9b51b5184179dd0a76fdc4e0e851b22a8aa842d5", "commit_rev":
"385db906ffa3937dab92438ee6fc43b6485514ae"}

[commit-bot: I haz the power, 2017-02-03 16:37:49]

Description was changed from

==========
Fix PrePaint UMA when SlimmingPaintInvalidation is disabled

Previously UMA of PrePaint with SlimmingPaintInvalidation disabled
count twice for the old path and prePaint() (which skips the real
pre-paint tree walk and so added many near 0 results).

BUG=646176
==========

to

==========
Fix PrePaint UMA when SlimmingPaintInvalidation is disabled

Previously UMA of PrePaint with SlimmingPaintInvalidation disabled
count twice for the old path and prePaint() (which skips the real
pre-paint tree walk and so added many near 0 results).

BUG=646176

Review-Url: https://codereview.chromium.org/2670013004
Cr-Commit-Position: refs/heads/master@{#448001}
Committed:
https://chromium.googlesource.com/chromium/src/+/385db906ffa3937dab92438ee6fc...
==========

[commit-bot: I haz the power, 2017-02-03 16:37:50]

Committed patchset #2 (id:20001) as
https://chromium.googlesource.com/chromium/src/+/385db906ffa3937dab92438ee6fc...