[cc] Set BeginFrame sequence numbers on CompositorFrames from Scheduler.

cc/trees/single_thread_proxy.cc

 // 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.
 
 #include "cc/trees/single_thread_proxy.h"
 
 #include "base/auto_reset.h"
 #include "base/memory/ptr_util.h"
 #include "base/profiler/scoped_tracker.h"
 #include "base/trace_event/trace_event.h"
@@ skipping 429 matching lines @@
 
   if (compositor_frame_sink_lost_) {
     RequestNewCompositorFrameSink();
     // RequestNewCompositorFrameSink could have synchronously created an output
     // surface, so check again before returning.
     if (compositor_frame_sink_lost_)
       return;
   }
 
   BeginFrameArgs begin_frame_args(BeginFrameArgs::Create(
-      BEGINFRAME_FROM_HERE, 0, 1, frame_begin_time, base::TimeTicks(),
+      BEGINFRAME_FROM_HERE, UINT32_MAX, 1, frame_begin_time, base::TimeTicks(),

[brianderson, 2017/02/24 01:08:38]

Can you make this a BeginFrameArgs constant with a meaningful name?

[Eric Seckler, 2017/02/27 11:50:32]

Done.

       BeginFrameArgs::DefaultInterval(), BeginFrameArgs::NORMAL));
 
   // Start the impl frame.
   {
     DebugScopedSetImplThread impl(task_runner_provider_);
     WillBeginImplFrame(begin_frame_args);
   }
 
   // Run the "main thread" and get it to commit.
   {
@@ skipping 15 matching lines @@
     layer_tree_host_impl_->ActivateSyncTree();
     DCHECK(
         !layer_tree_host_impl_->active_tree()->needs_update_draw_properties());
     layer_tree_host_impl_->PrepareTiles();
     layer_tree_host_impl_->SynchronouslyInitializeAllTiles();
 
     // TODO(danakj): Don't do this last... we prepared the wrong things. D:
     layer_tree_host_impl_->Animate();
 
     LayerTreeHostImpl::FrameData frame;
+    frame.begin_frame_source_id = begin_frame_args.source_id;
+    frame.begin_frame_sequence_number = begin_frame_args.sequence_number;
+    frame.latest_confirmed_begin_frame_sequence_number =
+        begin_frame_args.sequence_number;
     DoComposite(&frame);
 
     // DoComposite could abort, but because this is a synchronous composite
     // another draw will never be scheduled, so break remaining promises.
     layer_tree_host_impl_->active_tree()->BreakSwapPromises(
         SwapPromise::SWAP_FAILS);
 
     DidFinishImplFrame();
   }
 }
@@ skipping 186 matching lines @@
 
   std::vector<std::unique_ptr<SwapPromise>> empty_swap_promises;
   layer_tree_host_impl_->BeginMainFrameAborted(reason,
                                                std::move(empty_swap_promises));
   scheduler_on_impl_thread_->BeginMainFrameAborted(reason);
 }
 
 DrawResult SingleThreadProxy::ScheduledActionDrawIfPossible() {
   DebugScopedSetImplThread impl(task_runner_provider_);
   LayerTreeHostImpl::FrameData frame;
+  frame.begin_frame_source_id =
+      scheduler_on_impl_thread_->CurrentBeginFrameSourceId();
+  frame.begin_frame_sequence_number =
+      scheduler_on_impl_thread_->CurrentBeginFrameSequenceNumber();
+  frame.latest_confirmed_begin_frame_sequence_number =
+      scheduler_on_impl_thread_
+          ->LatestConfirmedBeginFrameSequenceNumberForActiveTree();
   return DoComposite(&frame);
 }
 
 DrawResult SingleThreadProxy::ScheduledActionDrawForced() {
   NOTREACHED();
   return INVALID_RESULT;
 }
 
 void SingleThreadProxy::ScheduledActionCommit() {
   DebugScopedSetMainThread main(task_runner_provider_);
@@ skipping 39 matching lines @@
 void SingleThreadProxy::DidFinishImplFrame() {
   layer_tree_host_impl_->DidFinishImplFrame();
 #if DCHECK_IS_ON()
   DCHECK(inside_impl_frame_)
       << "DidFinishImplFrame called while not inside an impl frame!";
   inside_impl_frame_ = false;
 #endif
 }
 
 }  // namespace cc