Reland recording box shadow as main thread scrolling reason with conflict fixed

Reland with conflict fixed

BUG=660907
TEST=StyleRelatedMainThreadScrollingReasonTest.BoxShadowTest
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2700613002
Cr-Commit-Position: refs/heads/master@{#450958}
Committed: https://chromium.googlesource.com/chromium/src/+/3d1a8d1c963cae739551639bcec3e4b2e15b9b10

[yigu, 2017-02-15 19:56:04]

Description was changed from

==========
Reland with conflict fixed

BUG=660907
TEST=StyleRelatedMainThreadScrollingReasonTest.BoxShadowTest
==========

to

==========
Reland with conflict fixed

BUG=660907
TEST=StyleRelatedMainThreadScrollingReasonTest.BoxShadowTest
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

[yigu, 2017-02-15 19:56:04]

Description was changed from

==========
Reland with conflict fixed

BUG=660907
TEST=StyleRelatedMainThreadScrollingReasonTest.BoxShadowTest
CQ_INCLUDE_TRYBOTS=master.tryserver.chromium.linux:linux_layout_tests_slimming_paint_v2
==========

to

==========
Reland with conflict fixed

BUG=660907
TEST=StyleRelatedMainThreadScrollingReasonTest.BoxShadowTest
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[yigu, 2017-02-15 19:56:13]

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

[commit-bot: I haz the power, 2017-02-15 19:57:07]

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

[yigu, 2017-02-15 19:57:14]

Description was changed from

==========
Reland with conflict fixed

BUG=660907
TEST=StyleRelatedMainThreadScrollingReasonTest.BoxShadowTest
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Reland with conflict fixed

BUG=660907
TEST=StyleRelatedMainThreadScrollingReasonTest.BoxShadowTest
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[yigu, 2017-02-15 20:53:57]

yigu@chromium.org changed reviewers:
+ ajuma@chromium.org, pdr@chromium.org

[yigu, 2017-02-15 20:53:58]

Hi Philip, this patch is a reland cl of a previously reverted CL and it was
supposed to land 1 week ago. bokan@ (OOO until Feb. 21) submitted it but for
some reason it was not successfully landed.
(https://codereview.chromium.org/2685653003/)

PaintLayer.cpp has changed since then with some conflicts so we cannot land
bokan@'s CL. The only difference from his CL is in PaintLayer.cpp. PTAL.
Thanks!

[commit-bot: I haz the power, 2017-02-15 21:09:07]

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

[commit-bot: I haz the power, 2017-02-15 21:09:08]

Dry run: Try jobs failed on following builders:
  mac_chromium_rel_ng on master.tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[yigu, 2017-02-15 21:19:34]

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

[commit-bot: I haz the power, 2017-02-15 21:21:24]

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

[ajuma, 2017-02-15 22:36:15]

On 2017/02/15 20:53:58, yigu wrote:
> Hi Philip, this patch is a reland cl of a previously reverted CL and it was
> supposed to land 1 week ago. bokan@ (OOO until Feb. 21) submitted it but for
> some reason it was not successfully landed.
> (https://codereview.chromium.org/2685653003/)
> 
> PaintLayer.cpp has changed since then with some conflicts so we cannot land
> bokan@'s CL. The only difference from his CL is in PaintLayer.cpp. PTAL.
> Thanks!

cc lgtm

[commit-bot: I haz the power, 2017-02-16 00:43:48]

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

[commit-bot: I haz the power, 2017-02-16 00:43:49]

Dry run: Try jobs failed on following builders:
  linux_chromium_chromeos_ozone_rel_ng on master.tryserver.chromium.linux
(JOB_TIMED_OUT, no build URL)

[pdr., 2017-02-16 04:36:55]

On 2017/02/15 at 22:36:15, ajuma wrote:
> On 2017/02/15 20:53:58, yigu wrote:
> > Hi Philip, this patch is a reland cl of a previously reverted CL and it was
> > supposed to land 1 week ago. bokan@ (OOO until Feb. 21) submitted it but for
> > some reason it was not successfully landed.
> > (https://codereview.chromium.org/2685653003/)
> > 
> > PaintLayer.cpp has changed since then with some conflicts so we cannot land
> > bokan@'s CL. The only difference from his CL is in PaintLayer.cpp. PTAL.
> > Thanks!
> 
> cc lgtm

LGTM 2

[yigu, 2017-02-16 14:03:38]

The CQ bit was checked by yigu@chromium.org

[commit-bot: I haz the power, 2017-02-16 14:03:51]

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 14:40:43]

CQ is committing da patch.

Bot data: {"patchset_id": 20001, "attempt_start_ts": 1487253818610770,
"parent_rev": "cda054bfe94bdcbdb275b563d52dc7f1d443f94b", "commit_rev":
"3d1a8d1c963cae739551639bcec3e4b2e15b9b10"}

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

Description was changed from

==========
Reland with conflict fixed

BUG=660907
TEST=StyleRelatedMainThreadScrollingReasonTest.BoxShadowTest
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Reland with conflict fixed

BUG=660907
TEST=StyleRelatedMainThreadScrollingReasonTest.BoxShadowTest
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

Review-Url: https://codereview.chromium.org/2700613002
Cr-Commit-Position: refs/heads/master@{#450958}
Committed:
https://chromium.googlesource.com/chromium/src/+/3d1a8d1c963cae739551639bcec3...
==========

[commit-bot: I haz the power, 2017-02-16 14:41:31]

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