Add AnimationTimingProperties dictionary

Source/bindings/core/v8/ToV8.h

Index: Source/bindings/core/v8/ToV8.h
diff --git a/Source/bindings/core/v8/ToV8.h b/Source/bindings/core/v8/ToV8.h
index 514ee5257ec5c0ea97b1d3266d2f25c5fd7e6f0b..e06dcd323a1ad8eccd8fff17325ed32c64cf9df7 100644
--- a/Source/bindings/core/v8/ToV8.h
+++ b/Source/bindings/core/v8/ToV8.h
@@ -16,6 +16,7 @@
 namespace blink {
 
 class DOMWindow;
+class Dictionary;
 class EventTarget;
 class WorkerGlobalScope;
 
@@ -182,6 +183,14 @@ inline v8::Handle<v8::Value> toV8(const ScriptValue& value, v8::Handle<v8::Objec
     return value.v8Value();
 }
 
+// Dictionary
+
+inline v8::Handle<v8::Value> toV8(const Dictionary& value, v8::Handle<v8::Object> creationContext, v8::Isolate*)
+{
+    ASSERT_NOT_REACHED();

[Jens Widell, 2015/02/23 12:07:00]

Adding this was necessary due to the use of sequence<Dictionary> in a union
type, since this unconditionally generated code to convert a union value to a V8
value. The union is used as a method argument only, so the conversion code is
not actually used.

Properly converting a Dictionary to a V8 value is of course possible, but we
should probably do it by cloning the V8 object wrapped by the Dictionary rather
than by just returning it. Or maybe not. Hopefully we never actually need this.

Do we think it's okay to add this implementation, just to make the generated
conversion code compile?

[haraken, 2015/02/23 13:41:17]

I'm fine in this CL, but it would be great if we could remove it in a follow-up
:)

+    return v8::Handle<v8::Value>();
+}
+
 // Array
 
 template<typename Sequence>