Revert "Box shadow should be recorded as main thread scrolling reasons for non-root layers."

Bisect job found this CL several times suspecting that it caused regression on nexus5p(crbug.com/685182). Revert it for now.

This reverts commit c83ddee4e5dcb0fdce873021c2701c9c3a2c547a.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2676333002
Cr-Commit-Position: refs/heads/master@{#448326}
Committed: https://chromium.googlesource.com/chromium/src/+/95d4abb16bc2444b187772d3a4ee461503d4b7a7

[yigu, 2017-02-06 16:28:06]

Description was changed from

==========
Revert "Box shadow should be recorded as main thread scrolling reasons for
non-root layers."

This reverts commit c83ddee4e5dcb0fdce873021c2701c9c3a2c547a.

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

to

==========
Revert "Box shadow should be recorded as main thread scrolling reasons for
non-root layers."

This reverts commit c83ddee4e5dcb0fdce873021c2701c9c3a2c547a.

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

[yigu, 2017-02-06 16:28:07]

Description was changed from

==========
Revert "Box shadow should be recorded as main thread scrolling reasons for
non-root layers."

This reverts commit c83ddee4e5dcb0fdce873021c2701c9c3a2c547a.

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

to

==========
Revert "Box shadow should be recorded as main thread scrolling reasons for
non-root layers."

This reverts commit c83ddee4e5dcb0fdce873021c2701c9c3a2c547a.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[yigu, 2017-02-06 16:28:35]

The CQ bit was checked by yigu@chromium.org

[commit-bot: I haz the power, 2017-02-06 16:29:01]

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-06 16:29:01]

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

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

No L-G-T-M from a valid reviewer yet. 
CQ run can only be started by full committers or once the patch has
received an L-G-T-M from a full committer.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
Committers are members of the group "project-chromium-committers".
Note that this has nothing to do with OWNERS files.

[yigu, 2017-02-06 16:30:20]

Description was changed from

==========
Revert "Box shadow should be recorded as main thread scrolling reasons for
non-root layers."

This reverts commit c83ddee4e5dcb0fdce873021c2701c9c3a2c547a.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Revert "Box shadow should be recorded as main thread scrolling reasons for
non-root layers."

This reverts commit c83ddee4e5dcb0fdce873021c2701c9c3a2c547a.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[yigu, 2017-02-06 16:30:21]

yigu@chromium.org changed reviewers:
+ ajuma@chromium.org, bokan@chromium.org, jwd@chromium.org

[yigu, 2017-02-06 16:34:26]

Description was changed from

==========
Revert "Box shadow should be recorded as main thread scrolling reasons for
non-root layers."

This reverts commit c83ddee4e5dcb0fdce873021c2701c9c3a2c547a.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Bisect job found this CL several times suspecting that it caused regression on
nexus5p(crbug.com/685182). Revert it for now.

This reverts commit c83ddee4e5dcb0fdce873021c2701c9c3a2c547a.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[ajuma, 2017-02-06 16:35:31]

On 2017/02/06 16:29:02, commit-bot: I haz the power wrote:
> No L-G-T-M from a valid reviewer yet. 
> CQ run can only be started by full committers or once the patch has
> received an L-G-T-M from a full committer.
> Even if an L-G-T-M may have been provided, it was from a non-committer,
> _not_ a full super star committer.
> Committers are members of the group "project-chromium-committers".
> Note that this has nothing to do with OWNERS files.

lgtm

[bokan, 2017-02-06 16:38:07]

lgtm

[yigu, 2017-02-06 16:44:27]

The CQ bit was checked by yigu@chromium.org

[yigu, 2017-02-06 16:44:28]

The patchset sent to the CQ was uploaded after l-g-t-m from bokan@chromium.org,
ajuma@chromium.org
Link to the patchset: https://codereview.chromium.org/2676333002/#ps20001
(title: "Keep histograms.xml as is to avoid messing up enum numbers")

[commit-bot: I haz the power, 2017-02-06 16:44:55]

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-06 18:48:07]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1486399467874020,
"parent_rev": "25c670f083721a982eb4b7aa01d06beaf888365e", "commit_rev":
"95d4abb16bc2444b187772d3a4ee461503d4b7a7"}

[commit-bot: I haz the power, 2017-02-06 18:48:39]

Description was changed from

==========
Bisect job found this CL several times suspecting that it caused regression on
nexus5p(crbug.com/685182). Revert it for now.

This reverts commit c83ddee4e5dcb0fdce873021c2701c9c3a2c547a.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Bisect job found this CL several times suspecting that it caused regression on
nexus5p(crbug.com/685182). Revert it for now.

This reverts commit c83ddee4e5dcb0fdce873021c2701c9c3a2c547a.

BUG=
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2676333002
Cr-Commit-Position: refs/heads/master@{#448326}
Committed:
https://chromium.googlesource.com/chromium/src/+/95d4abb16bc2444b187772d3a4ee...
==========

[commit-bot: I haz the power, 2017-02-06 18:48:40]

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

[yigu, 2017-02-07 20:43:09]

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

The reason for reverting is: The regression is not caused by the CL that this
patch tries to revert. .

[bokan, 2017-02-07 20:51:30]

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

The reason for reverting is: Reverted patch wasn't responsible for regression..