Bug 1435022 - Clean up GetLastComposeTime. r?sotaro draft
authorKartikaya Gupta <kgupta@mozilla.com>
Thu, 01 Feb 2018 16:29:06 -0500
changeset 750301 93b3d41228449fec1458f08e1175c82d2db817a5
parent 750300 f4d29c4dd7c5e2e59d012c7eebcedac479ce74eb
child 750302 1302d13f5c44e6e81e003ea0052a6ae41a31e5c6
push id97612
push userkgupta@mozilla.com
push dateThu, 01 Feb 2018 21:30:35 +0000
reviewerssotaro
bugs1435022
milestone60.0a1
Bug 1435022 - Clean up GetLastComposeTime. r?sotaro MozReview-Commit-ID: CEaz01FyjSs
gfx/layers/ipc/CompositorVsyncScheduler.cpp
gfx/layers/ipc/CompositorVsyncScheduler.h
--- a/gfx/layers/ipc/CompositorVsyncScheduler.cpp
+++ b/gfx/layers/ipc/CompositorVsyncScheduler.cpp
@@ -329,10 +329,17 @@ CompositorVsyncScheduler::DispatchVREven
 
 void
 CompositorVsyncScheduler::ScheduleTask(already_AddRefed<CancelableRunnable> aTask)
 {
   MOZ_ASSERT(CompositorThreadHolder::Loop());
   CompositorThreadHolder::Loop()->PostDelayedTask(Move(aTask), 0);
 }
 
+const TimeStamp&
+CompositorVsyncScheduler::GetLastComposeTime() const
+{
+  MOZ_ASSERT(CompositorThreadHolder::IsInCompositorThread());
+  return mLastCompose;
+}
+
 } // namespace layers
 } // namespace mozilla
--- a/gfx/layers/ipc/CompositorVsyncScheduler.h
+++ b/gfx/layers/ipc/CompositorVsyncScheduler.h
@@ -78,21 +78,21 @@ public:
   bool NeedsComposite();
 
   /**
    * Force a composite to happen right away, without waiting for the next vsync.
    * This must be called on the compositor thread.
    */
   void ForceComposeToTarget(gfx::DrawTarget* aTarget, const gfx::IntRect* aRect);
 
-  const TimeStamp& GetLastComposeTime()
-  {
-    return mLastCompose;
-  }
-
+  /**
+   * Return the vsync timestamp of the last or ongoing composite. Must be called
+   * on the compositor thread.
+   */
+  const TimeStamp& GetLastComposeTime() const;
 #ifdef COMPOSITOR_PERFORMANCE_WARNING
   const TimeStamp& GetExpectedComposeStartTime()
   {
     return mExpectedComposeStartTime;
   }
 #endif
 
 private: