Add additive animation support for border-radius CSS properties

Add additive animation support for border-radius CSS properties

This patch adds support for composite modes on border-radius
CSS animations via element.animate().

BUG=437696
Committed: https://crrev.com/bd831cabec2a2d24c0b635995df4677cb4b8f46d
Cr-Commit-Position: refs/heads/master@{#376159}

Committed: https://crrev.com/c0106b85472130a7ef9c1c452da53dbecf0b9fb0
Cr-Commit-Position: refs/heads/master@{#377742}

[alancutter (OOO until 2018), 2016-02-10 07:26:42]

alancutter@chromium.org changed reviewers:
+ ericwilligers@chromium.org

[Eric Willigers, 2016-02-10 09:18:24]

lgtm

https://codereview.chromium.org/1682843003/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/animations/composition/border-bottom-left-radius-composition.html
(right):

https://codereview.chromium.org/1682843003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/animations/composition/border-bottom-left-radius-composition.html:7:
underlying: '40px 40px',
I would like some of these tests to have different values for x and y.

[alancutter (OOO until 2018), 2016-02-11 00:24:54]

https://codereview.chromium.org/1682843003/diff/1/third_party/WebKit/LayoutTe...
File
third_party/WebKit/LayoutTests/animations/composition/border-bottom-left-radius-composition.html
(right):

https://codereview.chromium.org/1682843003/diff/1/third_party/WebKit/LayoutTe...
third_party/WebKit/LayoutTests/animations/composition/border-bottom-left-radius-composition.html:7:
underlying: '40px 40px',
On 2016/02/10 at 09:18:24, Eric Willigers wrote:
> I would like some of these tests to have different values for x and y.

Done.

[alancutter (OOO until 2018), 2016-02-17 05:46:36]

alancutter@chromium.org changed reviewers:
+ dstockwell@chromium.org

[alancutter (OOO until 2018), 2016-02-17 05:46:36]

+dstockwell for core.gypi changes.

[dstockwell, 2016-02-18 03:00:44]

lgtm

[alancutter (OOO until 2018), 2016-02-18 04:53:03]

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

[commit-bot: I haz the power, 2016-02-18 04:53:46]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1682843003/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1682843003/40001

[commit-bot: I haz the power, 2016-02-18 04:56:36]

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

[commit-bot: I haz the power, 2016-02-18 04:56:37]

Dry run: Try jobs failed on following builders:
  linux_chromium_compile_dbg_32_ng on tryserver.chromium.linux (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.linux/builders/linux_chromium_...)
  ios_dbg_simulator_ninja on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/ios_dbg_simulator...)
  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...)
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[alancutter (OOO until 2018), 2016-02-18 07:13:00]

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

[commit-bot: I haz the power, 2016-02-18 07:13:39]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1682843003/60001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1682843003/60001

[commit-bot: I haz the power, 2016-02-18 08:44:21]

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

[commit-bot: I haz the power, 2016-02-18 08:44:21]

Dry run: This issue passed the CQ dry run.

[alancutter (OOO until 2018), 2016-02-18 13:22:31]

The CQ bit was checked by alancutter@chromium.org

[alancutter (OOO until 2018), 2016-02-18 13:22:31]

The patchset sent to the CQ was uploaded after l-g-t-m from
ericwilligers@chromium.org, dstockwell@chromium.org
Link to the patchset: https://codereview.chromium.org/1682843003/#ps60001
(title: "Rebased")

[commit-bot: I haz the power, 2016-02-18 13:23:22]

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

[commit-bot: I haz the power, 2016-02-18 13:30:32]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2016-02-18 13:32:44]

Description was changed from

==========
Add additive animation support for border-radius CSS properties

This patch adds support for composite modes on border-radius
CSS animations via element.animate().

BUG=437696
==========

to

==========
Add additive animation support for border-radius CSS properties

This patch adds support for composite modes on border-radius
CSS animations via element.animate().

BUG=437696

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

[commit-bot: I haz the power, 2016-02-18 13:32:45]

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

[Theresa, 2016-02-18 17:09:12]

twellington@chromium.org changed reviewers:
+ twellington@chromium.org

[Theresa, 2016-02-18 17:09:13]

The patchset this is dependent on is causing compilation to fail on Android
arm64, x64 and x86 dbg builds, see
https://code.google.com/p/chromium/issues/detail?id=587875

Sheriff is reverting.

[Theresa, 2016-02-18 17:11:13]

A revert of this CL (patchset #4 id:60001) has been created in
https://codereview.chromium.org/1706273002/ by twellington@chromium.org.

The reason for reverting is: The patchset this is dependent on is causing
compilation to fail on Android
arm64, x64, x86, and MIPS dbg builds, see
https://code.google.com/p/chromium/issues/detail?id=587875

Sheriff is reverting..

[alancutter (OOO until 2018), 2016-02-25 03:26:34]

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

[commit-bot: I haz the power, 2016-02-25 03:29:46]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1682843003/80001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1682843003/80001

[commit-bot: I haz the power, 2016-02-25 05:54:00]

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

[commit-bot: I haz the power, 2016-02-25 05:54:01]

Dry run: This issue passed the CQ dry run.

[alancutter (OOO until 2018), 2016-02-26 00:30:38]

The CQ bit was checked by alancutter@chromium.org

[alancutter (OOO until 2018), 2016-02-26 00:30:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
dstockwell@chromium.org, ericwilligers@chromium.org
Link to the patchset: https://codereview.chromium.org/1682843003/#ps80001
(title: "Rebased")

[commit-bot: I haz the power, 2016-02-26 00:33:29]

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

[commit-bot: I haz the power, 2016-02-26 01:02:13]

Committed patchset #5 (id:80001)

[commit-bot: I haz the power, 2016-02-26 01:03:40]

Description was changed from

==========
Add additive animation support for border-radius CSS properties

This patch adds support for composite modes on border-radius
CSS animations via element.animate().

BUG=437696

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

to

==========
Add additive animation support for border-radius CSS properties

This patch adds support for composite modes on border-radius
CSS animations via element.animate().

BUG=437696

Committed: https://crrev.com/bd831cabec2a2d24c0b635995df4677cb4b8f46d
Cr-Commit-Position: refs/heads/master@{#376159}

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

[commit-bot: I haz the power, 2016-02-26 01:03:41]

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