Revert of Skip PaintLayer empty paint phases if it previously painted nothing

Revert of Skip PaintLayer empty paint phases if it previously painted nothing (patchset #12 id:220001 of https://codereview.chromium.org/1739893006/ )

Reason for revert:
BUG=595839
Original issue's description:
> Skip PaintLayer empty paint phases if it previously painted nothing
>
> This avoids later painting of empty phases if the layer is marked to need
> a paint phase but the phase painted nothing in the following cases:
> - the object marking the layer needing the paint phase is not visible
>   (e.g. because it is clipped)
> - the layer was marked needing the paint phase by some object that
>   has been deleted or moved to another layer.
>
> Instead of directly clearing the needsPaintPhase flags, this CL adds
> a new set of flags to indicate if the previous paint of the phases painted
> nothing. This is because invalidations of the needsPaintPhase flags and the
> new flags need different conditions. The new flags are cleared when scroll
> offset, interest rect or clipping changes.
>
> Cluster telemetry runs show that this will improve non-cached painting
> by 1.7% and fully-cached painting (with subsequence-caching disabled) by
> 45%.
>
> BUG=574938
>
> Committed: https://crrev.com/8ad6e62af5301a7e8dacd12fc1ae7914258b9e7a
> Cr-Commit-Position: refs/heads/master@{#380841}

TBR=chrishtr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=574938
Committed: https://crrev.com/e425e3a8c5204d2d75f34c885c33f077372d1ff8
Cr-Commit-Position: refs/heads/master@{#382018}

[Xianzhu, 2016-03-18 17:02:35]

Created Revert of Skip PaintLayer empty paint phases if it previously painted
nothing

[commit-bot: I haz the power, 2016-03-18 17:02:53]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1815293002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1815293002/1

[chrishtr, 2016-03-18 17:04:32]

lgtm

[commit-bot: I haz the power, 2016-03-18 17:04:47]

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

[commit-bot: I haz the power, 2016-03-18 17:04:48]

Try jobs failed on following builders:
  ios_dbg_simulator_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  ios_rel_device_gn on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_gn...)
  ios_rel_device_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_rel_device_ni...)
  mac_chromium_compile_dbg_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_comp...)

[Xianzhu, 2016-03-18 17:15:57]

The CQ bit was checked by wangxianzhu@chromium.org

[Xianzhu, 2016-03-18 17:15:58]

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/1815293002/#ps140001
(title: "Rebase")

[commit-bot: I haz the power, 2016-03-18 17:16:22]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1815293002/140001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1815293002/140001

[commit-bot: I haz the power, 2016-03-18 18:25:03]

Committed patchset #2 (id:140001)

[commit-bot: I haz the power, 2016-03-18 18:26:14]

Description was changed from

==========
Revert of Skip PaintLayer empty paint phases if it previously painted nothing
(patchset #12 id:220001 of https://codereview.chromium.org/1739893006/ )

Reason for revert:
BUG=595839

Original issue's description:
> Skip PaintLayer empty paint phases if it previously painted nothing
> 
> This avoids later painting of empty phases if the layer is marked to need
> a paint phase but the phase painted nothing in the following cases:
> - the object marking the layer needing the paint phase is not visible
>   (e.g. because it is clipped)
> - the layer was marked needing the paint phase by some object that
>   has been deleted or moved to another layer.
> 
> Instead of directly clearing the needsPaintPhase flags, this CL adds
> a new set of flags to indicate if the previous paint of the phases painted
> nothing. This is because invalidations of the needsPaintPhase flags and the
> new flags need different conditions. The new flags are cleared when scroll
> offset, interest rect or clipping changes.
> 
> Cluster telemetry runs show that this will improve non-cached painting
> by 1.7% and fully-cached painting (with subsequence-caching disabled) by
> 45%.
> 
> BUG=574938
> 
> Committed: https://crrev.com/8ad6e62af5301a7e8dacd12fc1ae7914258b9e7a
> Cr-Commit-Position: refs/heads/master@{#380841}

TBR=chrishtr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=574938
==========

to

==========
Revert of Skip PaintLayer empty paint phases if it previously painted nothing
(patchset #12 id:220001 of https://codereview.chromium.org/1739893006/ )

Reason for revert:
BUG=595839

Original issue's description:
> Skip PaintLayer empty paint phases if it previously painted nothing
>
> This avoids later painting of empty phases if the layer is marked to need
> a paint phase but the phase painted nothing in the following cases:
> - the object marking the layer needing the paint phase is not visible
>   (e.g. because it is clipped)
> - the layer was marked needing the paint phase by some object that
>   has been deleted or moved to another layer.
>
> Instead of directly clearing the needsPaintPhase flags, this CL adds
> a new set of flags to indicate if the previous paint of the phases painted
> nothing. This is because invalidations of the needsPaintPhase flags and the
> new flags need different conditions. The new flags are cleared when scroll
> offset, interest rect or clipping changes.
>
> Cluster telemetry runs show that this will improve non-cached painting
> by 1.7% and fully-cached painting (with subsequence-caching disabled) by
> 45%.
>
> BUG=574938
>
> Committed: https://crrev.com/8ad6e62af5301a7e8dacd12fc1ae7914258b9e7a
> Cr-Commit-Position: refs/heads/master@{#380841}

TBR=chrishtr@chromium.org
# Not skipping CQ checks because original CL landed more than 1 days ago.
BUG=574938

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

[commit-bot: I haz the power, 2016-03-18 18:26:15]

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

[Xianzhu, 2016-08-26 22:17:06]

A revert of this CL (patchset #2 id:140001) has been created in
https://codereview.chromium.org/2285633005/ by wangxianzhu@chromium.org.

The reason for reverting is: Create a reland CL for debugging..