cc: Introduce BeginFrame sequence numbers and acknowledgements.

cc/scheduler/scheduler_state_machine.h

 // Copyright 2011 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 CC_SCHEDULER_SCHEDULER_STATE_MACHINE_H_
 #define CC_SCHEDULER_SCHEDULER_STATE_MACHINE_H_
 
+#include <stdint.h>
+
 #include <memory>
 #include <string>
 
 #include "base/macros.h"
 #include "cc/base/cc_export.h"
-#include "cc/output/begin_frame_args.h"
 #include "cc/scheduler/commit_earlyout_reason.h"
 #include "cc/scheduler/draw_result.h"
 #include "cc/scheduler/scheduler_settings.h"
 #include "cc/tiles/tile_priority.h"
 
 namespace base {
 namespace trace_event {
 class ConvertableToTraceFormat;
 class TracedValue;
 }
@@ skipping 116 matching lines @@
 
   void AbortDraw();
 
   // Indicates whether the impl thread needs a BeginImplFrame callback in order
   // to make progress.
   bool BeginFrameNeeded() const;
 
   // Indicates that the system has entered and left a BeginImplFrame callback.
   // The scheduler will not draw more than once in a given BeginImplFrame
   // callback nor send more than one BeginMainFrame message.
-  void OnBeginImplFrame();
+  void OnBeginImplFrame(uint64_t source_id, uint64_t sequence_number);
   // Indicates that the scheduler has entered the draw phase. The scheduler
   // will not draw more than once in a single draw phase.
   // TODO(sunnyps): Rename OnBeginImplFrameDeadline to OnDraw or similar.
   void OnBeginImplFrameDeadline();
   void OnBeginImplFrameIdle();
 
   int current_frame_number() const { return current_frame_number_; }
 
   BeginImplFrameState begin_impl_frame_state() const {
     return begin_impl_frame_state_;
@@ skipping 101 matching lines @@
   // True if we need to abort draws to make forward progress.
   bool PendingDrawsShouldBeAborted() const;
 
   bool CouldSendBeginMainFrame() const;
 
   void SetDeferCommits(bool defer_commits);
 
   void SetVideoNeedsBeginFrames(bool video_needs_begin_frames);
   bool video_needs_begin_frames() const { return video_needs_begin_frames_; }
 
+  bool did_submit_in_last_frame() const { return did_submit_in_last_frame_; }
+  uint64_t last_frame_number_compositor_frame_was_fresh() const {
+    return last_frame_number_compositor_frame_was_fresh_;
+  }
+  uint64_t last_frame_number_active_tree_was_fresh() const {
+    return last_frame_number_active_tree_was_fresh_;
+  }
+
  protected:
   bool BeginFrameRequiredForAction() const;
   bool BeginFrameNeededForVideo() const;
   bool ProactiveBeginFrameWanted() const;
 
   bool ShouldTriggerBeginImplFrameDeadlineImmediately() const;
 
   // True if we need to force activations to make forward progress.
   // TODO(sunnyps): Rename this to ShouldAbortCurrentFrame or similar.
   bool PendingActivationsShouldBeForced() const;
 
   bool ShouldBeginCompositorFrameSinkCreation() const;
   bool ShouldDraw() const;
   bool ShouldActivatePendingTree() const;
   bool ShouldSendBeginMainFrame() const;
   bool ShouldCommit() const;
   bool ShouldPrepareTiles() const;
   bool ShouldInvalidateCompositorFrameSink() const;
 
   void WillDrawInternal();
   void DidDrawInternal(DrawResult draw_result);
 
   const SchedulerSettings settings_;
 
   CompositorFrameSinkState compositor_frame_sink_state_;
   BeginImplFrameState begin_impl_frame_state_;
   BeginMainFrameState begin_main_frame_state_;
   ForcedRedrawOnTimeoutState forced_redraw_state_;
 
   // These are used for tracing only.

[Sami, 2016/12/06 12:41:07]

This comment probably isn't accurate anymore?

[Eric Seckler, 2016/12/06 17:34:00]

Done.

   int commit_count_;
-  int current_frame_number_;
-  int last_frame_number_submit_performed_;
-  int last_frame_number_draw_performed_;
-  int last_frame_number_begin_main_frame_sent_;
-  int last_frame_number_invalidate_compositor_frame_sink_performed_;
+  uint64_t begin_frame_source_id_;
+  uint64_t current_frame_number_;
+  uint64_t last_frame_number_submit_performed_;
+  uint64_t last_frame_number_draw_performed_;
+  uint64_t last_frame_number_begin_main_frame_sent_;
+  uint64_t last_frame_number_invalidate_compositor_frame_sink_performed_;
+  uint64_t last_frame_number_pending_tree_was_fresh_;
+  uint64_t last_frame_number_active_tree_was_fresh_;
+  uint64_t last_frame_number_compositor_frame_was_fresh_;
 
   // These are used to ensure that an action only happens once per frame,
   // deadline, etc.
   bool draw_funnel_;
   bool send_begin_main_frame_funnel_;
   bool invalidate_compositor_frame_sink_funnel_;
   // prepare_tiles_funnel_ is "filled" each time PrepareTiles is called
   // and "drained" on each BeginImplFrame. If the funnel gets too full,
   // we start throttling ACTION_PREPARE_TILES such that we average one
   // PrepareTiles per BeginImplFrame.
@@ skipping 26 matching lines @@
   bool did_draw_in_last_frame_;
   bool did_submit_in_last_frame_;
 
  private:
   DISALLOW_COPY_AND_ASSIGN(SchedulerStateMachine);
 };
 
 }  // namespace cc
 
 #endif  // CC_SCHEDULER_SCHEDULER_STATE_MACHINE_H_