RESOLVED FIXED 121587
[SVG] Handle endEvent for svg animations
https://bugs.webkit.org/show_bug.cgi?id=121587
Summary [SVG] Handle endEvent for svg animations
Ryosuke Niwa
Reported 2013-09-18 20:38:19 PDT
Attachments
Simple testcase (475 bytes, image/svg+xml)
2015-10-12 03:32 PDT, Antoine Quint
no flags
Patch (11.13 KB, patch)
2015-10-12 06:11 PDT, Antoine Quint
no flags
Patch for landing (11.02 KB, patch)
2015-10-12 13:57 PDT, Antoine Quint
no flags
Radar WebKit Bug Importer
Comment 1 2015-10-12 01:42:54 PDT
Antoine Quint
Comment 2 2015-10-12 03:32:07 PDT
Created attachment 262882 [details] Simple testcase
Antoine Quint
Comment 3 2015-10-12 06:11:59 PDT
Dean Jackson
Comment 4 2015-10-12 11:30:37 PDT
Comment on attachment 262886 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=262886&action=review > Source/WebCore/svg/animation/SMILTimeContainer.cpp:272 > AnimationsVector* scheduled = it.value.get(); > + unsigned size = scheduled->size(); > + for (unsigned n = 0; n < size; n++) { > + SVGSMILElement* animation = scheduled->at(n); > + if (!animation->hasConditionsConnected()) > + animation->connectConditions(); > + } I think you can use a modern for loop here. for (auto* animation : scheduled) { if (!animation->hasConditionsConnected()) animation->connectConditions(); } While here, we can come up with a better variable name than "it". > Source/WebCore/svg/animation/SMILTimeContainer.cpp:276 > + for (auto& it : m_scheduledAnimations) { > + AnimationsVector* scheduled = it.value.get(); Same here about the variable name.
Antoine Quint
Comment 5 2015-10-12 13:57:35 PDT
Created attachment 262918 [details] Patch for landing
WebKit Commit Bot
Comment 6 2015-10-12 14:49:08 PDT
Comment on attachment 262918 [details] Patch for landing Clearing flags on attachment: 262918 Committed r190890: <http://trac.webkit.org/changeset/190890>
WebKit Commit Bot
Comment 7 2015-10-12 14:49:12 PDT
All reviewed patches have been landed. Closing bug.
Note You need to log in before you can comment on or make changes to this bug.