Tidy up some loops in SMILTimeContainer::updateAnimations

Tidy up some loops in SMILTimeContainer::updateAnimations

Use for-range style loops and get rid of the local 'size' variable.
Also move the sort() call down after the early-out when there are no
animations to apply.

BUG=641437
Committed: https://crrev.com/887b2bd70548734daaa7f1c13ed696612fc095d2
Cr-Commit-Position: refs/heads/master@{#415165}

[fs, 2016-08-28 15:11:25]

The CQ bit was checked by fs@opera.com to run a CQ dry run

[commit-bot: I haz the power, 2016-08-28 15:11:31]

Dry run: 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, 2016-08-28 16:26:30]

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

[commit-bot: I haz the power, 2016-08-28 16:26:31]

Dry run: Try jobs failed on following builders:
  linux_android_rel_ng on master.tryserver.chromium.android (JOB_FAILED,
https://build.chromium.org/p/tryserver.chromium.android/builders/linux_androi...)

[fs, 2016-08-28 17:24:34]

fs@opera.com changed reviewers:
+ pdr@chromium.org

[pdr., 2016-08-29 00:47:03]

LGTM

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

https://codereview.chromium.org/2287973002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/svg/animation/SMILTimeContainer.cpp:522: if
(timedElement->isConnected() && timedElement->isSVGDiscardElement()) {
Forever reminded of the mistakes I made reviewing SVGDiscardElement a long time
ago :(

[fs, 2016-08-29 08:14:16]

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

https://codereview.chromium.org/2287973002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/svg/animation/SMILTimeContainer.cpp:522: if
(timedElement->isConnected() && timedElement->isSVGDiscardElement()) {
On 2016/08/29 at 00:47:03, pdr. wrote:
> Forever reminded of the mistakes I made reviewing SVGDiscardElement a long
time ago :(

Mmm, discard, yum... (I think I know which mistake[s] that would be.)

[fs, 2016-08-29 19:45:00]

The CQ bit was checked by fs@opera.com

[commit-bot: I haz the power, 2016-08-29 20:23:32]

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, 2016-08-30 05:36:25]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 05:38:30]

Description was changed from

==========
Tidy up some loops in SMILTimeContainer::updateAnimations

Use for-range style loops and get rid of the local 'size' variable.
Also move the sort() call down after the early-out when there are no
animations to apply.

BUG=641437
==========

to

==========
Tidy up some loops in SMILTimeContainer::updateAnimations

Use for-range style loops and get rid of the local 'size' variable.
Also move the sort() call down after the early-out when there are no
animations to apply.

BUG=641437

Committed: https://crrev.com/887b2bd70548734daaa7f1c13ed696612fc095d2
Cr-Commit-Position: refs/heads/master@{#415165}
==========

[commit-bot: I haz the power, 2016-08-30 05:38:32]

Patchset 1 (id:??) landed as
https://crrev.com/887b2bd70548734daaa7f1c13ed696612fc095d2
Cr-Commit-Position: refs/heads/master@{#415165}