Web Animations: Move property handle filtering into AnimationStack building

Web Animations: Move property handle filtering into AnimationStack building

Previously we were filtering out PropertyHandle types while iterating over
an AnimationStack.
This change pushes the filter into the construction of the AnimationStack
to avoid having undesirable entries in the first place.

Committed: https://crrev.com/2fb3f8fa8fda5c1322aa26d760e04b4397850f9a
Cr-Commit-Position: refs/heads/master@{#357045}

[alancutter (OOO until 2018), 2015-10-21 00:16:06]

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

[dstockwell, 2015-10-21 01:00:39]

https://codereview.chromium.org/1412423004/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/animation/AnimationStack.cpp (right):

https://codereview.chromium.org/1412423004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/animation/AnimationStack.cpp:98: // We don't
exactly know when new animations will start, but timelineCurrentTime is a good
estimate.
Needs a rebase. Took me a while to figure out what was going on with this
comment.

[alancutter (OOO until 2018), 2015-10-21 04:22:16]

https://codereview.chromium.org/1412423004/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/animation/AnimationStack.cpp (right):

https://codereview.chromium.org/1412423004/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/animation/AnimationStack.cpp:98: // We don't
exactly know when new animations will start, but timelineCurrentTime is a good
estimate.
On 2015/10/21 at 01:00:39, dstockwell wrote:
> Needs a rebase. Took me a while to figure out what was going on with this
comment.

Done.

[dstockwell, 2015-10-27 06:56:26]

lgtm

[alancutter (OOO until 2018), 2015-10-30 03:13:16]

The CQ bit was checked by alancutter@chromium.org

[commit-bot: I haz the power, 2015-10-30 03:13:31]

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

[commit-bot: I haz the power, 2015-10-30 03:15:13]

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

[commit-bot: I haz the power, 2015-10-30 03:15:14]

Try jobs failed on following builders:
  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_gn_rel on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_gn_r...)

[alancutter (OOO until 2018), 2015-10-30 03:20:01]

The CQ bit was checked by alancutter@chromium.org

[alancutter (OOO until 2018), 2015-10-30 03:20:02]

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

[commit-bot: I haz the power, 2015-10-30 03:20:08]

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

[commit-bot: I haz the power, 2015-10-30 04:32:26]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-10-30 04:33:28]

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