Changeset 173506 in webkit

Timestamp:

Sep 10, 2014 9:01:53 PM (10 years ago)

Author:

cavalcantii@gmail.com

Message:

[SVG] Cleanup iterator code in SMILTimeContainer using range-based loop
​https://bugs.webkit.org/show_bug.cgi?id=136720

Reviewed by Filip Pizlo.

No new tests, no change on behavior.

• svg/animation/SMILTimeContainer.cpp:

(WebCore::SMILTimeContainer::setElapsed):
(WebCore::SMILTimeContainer::updateAnimations):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• svg/animation/SMILTimeContainer.cpp (modified) (2 diffs)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2014-09-10  Adenilson Cavalcanti  <cavalcantii@gmail.com>
+
+        [SVG] Cleanup iterator code in SMILTimeContainer using range-based loop
+        https://bugs.webkit.org/show_bug.cgi?id=136720
+
+        Reviewed by Filip Pizlo.
+
+        No new tests, no change on behavior.
+
+        * svg/animation/SMILTimeContainer.cpp:
+        (WebCore::SMILTimeContainer::setElapsed):
+        (WebCore::SMILTimeContainer::updateAnimations):
+
 2014-09-10  Timothy Horton  <timothy_horton@apple.com>
 

trunk/Source/WebCore/svg/animation/SMILTimeContainer.cpp

@@ -189 +189 @@
     m_preventScheduledAnimationsChanges = true;
 #endif
-    GroupedAnimationsMap::iterator end = m_scheduledAnimations.end();
-    for (GroupedAnimationsMap::iterator it = m_scheduledAnimations.begin(); it != end; ++it) {
-        AnimationsVector* scheduled = it->value.get();
+    for (auto& it : m_scheduledAnimations) {
+        AnimationsVector* scheduled = it.value.get();
         unsigned size = scheduled->size();
         for (unsigned n = 0; n < size; n++)
@@ -267 +266 @@
 
     AnimationsVector animationsToApply;
-    GroupedAnimationsMap::iterator end = m_scheduledAnimations.end();
-    for (GroupedAnimationsMap::iterator it = m_scheduledAnimations.begin(); it != end; ++it) {
-        AnimationsVector* scheduled = it->value.get();
+    for (auto& it : m_scheduledAnimations) {
+        AnimationsVector* scheduled = it.value.get();
 
         // Sort according to priority. Elements with later begin time have higher priority.