Alternative syntax for element.animate list of keyframes

third_party/WebKit/Source/core/animation/EffectInput.h

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef EffectInput_h
 #define EffectInput_h
 
 #include "core/CoreExport.h"
 #include "core/animation/EffectModel.h"
 #include "wtf/Allocator.h"
 #include "wtf/Vector.h"
 
 namespace blink {
 
 class EffectModel;
 class EffectModelOrDictionarySequenceOrDictionary;
 class Dictionary;
 class Element;
 class ExceptionState;
+class StringKeyframe;
 
 class CORE_EXPORT EffectInput {
     STATIC_ONLY(EffectInput);
 public:
     // TODO(alancutter): Replace Element* parameter with Document&.
     static EffectModel* convert(Element*, const EffectModelOrDictionarySequenceOrDictionary&, ExceptionState&);
 
 private:
-    static EffectModel* convert(Element*, const Vector<Dictionary>& keyframeDictionaryVector, ExceptionState&);
+    // keyframesSpecifiedAsList should be
+    // - true if the keyframeDictionaryVector was specified as a list originally (we entered this function from the DictionarySequence case of
+    //       EffectModelOrDictionarySequenceOrDictionary above)
+    // - false if we were in the single Dictionary case (handled by the function below) and needed to fall back to the sequence case instead
+    static EffectModel* convert(Element*, const Vector<Dictionary>& keyframeDictionaryVector, ExceptionState&, bool keyframesSpecifiedAsList);
+
+    static EffectModel* convert(Element*, const Dictionary& keyframeDictionary, ExceptionState&);
+
+    // Returns true if the property passed in is a compositable property.
+    static bool setKeyframeValue(Element*, StringKeyframe&, const String& property, const String& value);
+
+    static EffectModel* createEffectModelFromKeyframes(Element*, const Vector<RefPtr<StringKeyframe>>& keyframes, bool encounteredCompositableProperty, ExceptionState&);

[alancutter (OOO until 2018), 2016/02/25 06:52:27]

No need for private static methods, these can live in an anonymous namespace in
the cpp file. This will probably reduce the number of includes and forward
declarations in the header file.

[suzyh_UTC10 (ex-contributor), 2016/02/26 04:35:31]

Done.

 };
 
 } // namespace blink
 
 #endif