Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1756)

Unified Diff: cc/blimp/layer_tree_host_remote.cc

Issue 2583483002: [cc] Adds source_id and sequence_number to BeginFrameArgs. (Closed)
Patch Set: Created 4 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: cc/blimp/layer_tree_host_remote.cc
diff --git a/cc/blimp/layer_tree_host_remote.cc b/cc/blimp/layer_tree_host_remote.cc
index 466a4209370460a429f6a3f943a2e735f5de5b43..d1584d7bace2d401106adaf3f9f04a19a8134522 100644
--- a/cc/blimp/layer_tree_host_remote.cc
+++ b/cc/blimp/layer_tree_host_remote.cc
@@ -11,7 +11,6 @@
#include "cc/blimp/engine_picture_cache.h"
#include "cc/blimp/picture_data_conversions.h"
#include "cc/blimp/remote_compositor_bridge.h"
-#include "cc/output/begin_frame_args.h"
#include "cc/output/compositor_frame_sink.h"
#include "cc/proto/compositor_message.pb.h"
#include "cc/proto/gfx_conversions.h"
@@ -371,7 +370,8 @@ void LayerTreeHostRemote::BeginMainFrame() {
current_pipeline_stage_ = FramePipelineStage::ANIMATE;
base::TimeTicks now = base::TimeTicks::Now();
client_->BeginMainFrame(BeginFrameArgs::Create(
- BEGINFRAME_FROM_HERE, now, now + kDefaultFrameInterval,
+ BEGINFRAME_FROM_HERE, begin_frame_source_.source_id(),
+ begin_frame_number_++, now, now + kDefaultFrameInterval,
brianderson 2016/12/15 22:10:33 Increment after the call.
Eric Seckler 2016/12/16 17:25:57 Done.
kDefaultFrameInterval, BeginFrameArgs::NORMAL));
// We don't run any animations on the layer because threaded animations are
// disabled.

Powered by Google App Engine
This is Rietveld 408576698