Resolve ready and finished promises asynchronously

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

 /*
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 203 matching lines @@
   void destroyCompositorPlayer();
   void attachCompositorTimeline();
   void detachCompositorTimeline();
   void attachCompositedLayers();
   void detachCompositedLayers();
   // CompositorAnimationDelegate implementation.
   void notifyAnimationStarted(double monotonicTime, int group) override;
   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;
   double m_playbackRate;
   double m_startTime;
   double m_holdTime;
 
   unsigned m_sequenceNumber;
 
-  typedef ScriptPromiseProperty<Member<Animation>,
-                                Member<Animation>,
-                                Member<DOMException>>
-      AnimationPromise;
   Member<AnimationPromise> m_finishedPromise;
   Member<AnimationPromise> m_readyPromise;
 
   Member<AnimationEffectReadOnly> m_content;
   Member<AnimationTimeline> m_timeline;
 
   // Reflects all pausing, including via pauseForTesting().
   bool m_paused;
   bool m_held;
   bool m_isPausedForTesting;
@@ skipping 90 matching lines @@
 
   bool m_currentTimePending;
   bool m_stateIsBeingUpdated;
 
   bool m_effectSuppressed;
 };
 
 }  // namespace blink
 
 #endif  // Animation_h