Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(190)

Unified Diff: third_party/WebKit/Source/core/svg/SVGDocumentExtensions.cpp

Issue 1412423004: Web Animations: Move property handle filtering into AnimationStack building (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@_svgAdditiveApplication
Patch Set: Rebased Created 5 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/Source/core/css/resolver/StyleResolver.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/core/svg/SVGDocumentExtensions.cpp
diff --git a/third_party/WebKit/Source/core/svg/SVGDocumentExtensions.cpp b/third_party/WebKit/Source/core/svg/SVGDocumentExtensions.cpp
index 043d4495f497854528802ddd2a8a5967bf36e21e..b5dd1e51f79997016c64190304a979eaccf6d237 100644
--- a/third_party/WebKit/Source/core/svg/SVGDocumentExtensions.cpp
+++ b/third_party/WebKit/Source/core/svg/SVGDocumentExtensions.cpp
@@ -99,6 +99,11 @@ void SVGDocumentExtensions::serviceOnAnimationFrame(Document& document, double m
document.accessSVGExtensions().serviceAnimations(monotonicAnimationStartTime);
}
+static bool isSVGAttributeHandle(const PropertyHandle& propertyHandle)
+{
+ return propertyHandle.isSVGAttribute();
+}
+
void SVGDocumentExtensions::serviceAnimations(double monotonicAnimationStartTime)
{
if (RuntimeEnabledFeatures::smilEnabled()) {
@@ -114,10 +119,8 @@ void SVGDocumentExtensions::serviceAnimations(double monotonicAnimationStartTime
// TODO(alancutter): Make SVG animation effect application a separate document lifecycle phase from servicing animations to be responsive to Javascript manipulation of exposed animation objects.
for (auto& svgElement : webAnimationsPendingSVGElements) {
ActiveInterpolationsMap activeInterpolationsMap = AnimationStack::activeInterpolations(
- &svgElement->elementAnimations()->animationStack(), nullptr, nullptr, KeyframeEffect::DefaultPriority);
+ &svgElement->elementAnimations()->animationStack(), nullptr, nullptr, KeyframeEffect::DefaultPriority, isSVGAttributeHandle);
for (auto& entry : activeInterpolationsMap) {
- if (!entry.key.isSVGAttribute())
- continue;
const QualifiedName& attribute = entry.key.svgAttribute();
const Interpolation& interpolation = *entry.value.first();
if (interpolation.isInvalidatableInterpolation()) {
« no previous file with comments | « third_party/WebKit/Source/core/css/resolver/StyleResolver.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698