Move buildFilterOperations to FilterEffectBuilder

third_party/WebKit/Source/core/animation/CompositorAnimations.cpp

Index: third_party/WebKit/Source/core/animation/CompositorAnimations.cpp
diff --git a/third_party/WebKit/Source/core/animation/CompositorAnimations.cpp b/third_party/WebKit/Source/core/animation/CompositorAnimations.cpp
index cffa41ac1f0995190ba92a63e18654a474a40d77..5d4bce1cdabaf26ff512e42ebf30dba3bf81260b 100644
--- a/third_party/WebKit/Source/core/animation/CompositorAnimations.cpp
+++ b/third_party/WebKit/Source/core/animation/CompositorAnimations.cpp
@@ -41,6 +41,7 @@
 #include "core/layout/LayoutBoxModelObject.h"
 #include "core/layout/LayoutObject.h"
 #include "core/layout/compositing/CompositedLayerMapping.h"
+#include "core/paint/FilterEffectBuilder.h"
 #include "core/paint/PaintLayer.h"
 #include "platform/animation/AnimationTranslationUtil.h"
 #include "platform/animation/CompositorAnimation.h"
@@ -53,7 +54,6 @@
 #include "platform/animation/CompositorTransformKeyframe.h"
 #include "platform/geometry/FloatBox.h"
 #include "public/platform/Platform.h"
-#include "public/platform/WebCompositorSupport.h"
 #include "wtf/PtrUtil.h"
 #include <algorithm>
 #include <cmath>
@@ -416,6 +416,12 @@ bool CompositorAnimations::convertTimingForCompositor(const Timing& timing, doub
 
 namespace {
 
+CompositorFilterOperations toCompositorFilterOperations(const FilterOperations& inOperations)

[loyso (OOO), 2016/09/22 04:55:51]

There is no need to establish this function - there is just one call site in
this file.

+{
+    FilterEffectBuilder builder(nullptr, FloatRect(), 1);

[loyso (OOO), 2016/09/22 04:55:51]

Can we have a dedicated ctor with no args for this case? Now we have all those
forwarding C++11 ctors.

+    return builder.buildFilterOperations(inOperations);
+}
+
 void addKeyframeToCurve(CompositorFilterAnimationCurve& curve, Keyframe::PropertySpecificKeyframe* keyframe,
     const AnimatableValue* value, const TimingFunction& keyframeTimingFunction)
 {