Resolve ready and finished promises asynchronously

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

Index: third_party/WebKit/Source/core/animation/Animation.h
diff --git a/third_party/WebKit/Source/core/animation/Animation.h b/third_party/WebKit/Source/core/animation/Animation.h
index 339209ee65f3a4012595097b29c7f009e9cdfe12..8553cb142f660333d31fc07695e6651dd24d6444 100644
--- a/third_party/WebKit/Source/core/animation/Animation.h
+++ b/third_party/WebKit/Source/core/animation/Animation.h
@@ -221,6 +221,13 @@ class CORE_EXPORT Animation final : public EventTargetWithInlineData,
   void notifyAnimationFinished(double monotonicTime, int group) override {}
   void notifyAnimationAborted(double monotonicTime, int group) override {}
 
+  typedef ScriptPromiseProperty<Member<Animation>,
+                                Member<Animation>,
+                                Member<DOMException>>
+      AnimationPromise;

[alancutter (OOO until 2018), 2017/01/09 03:25:07]

Nit: "using =" might be easier to read here with the line wrapping.

[adithyas, 2017/01/09 15:22:58]

Done.

+
+  void resolvePromiseAsync(Member<AnimationPromise>);

[jbroman, 2017/01/07 04:06:36]

nit: For Oilpan objects, we usually pass T* rather than Member<T> as arguments.
I'd make this AnimationPromise*, and move the ".get()" to the call sites.

[adithyas, 2017/01/09 15:22:58]

Done.

+
   String m_id;
 
   AnimationPlayState m_playState;
@@ -230,10 +237,6 @@ class CORE_EXPORT Animation final : public EventTargetWithInlineData,
 
   unsigned m_sequenceNumber;
 
-  typedef ScriptPromiseProperty<Member<Animation>,
-                                Member<Animation>,
-                                Member<DOMException>>
-      AnimationPromise;
   Member<AnimationPromise> m_finishedPromise;
   Member<AnimationPromise> m_readyPromise;