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

Unified Diff: cc/surfaces/compositor_frame_sink_support.cc

Issue 2779633002: [cc] Remove remaining_frames from BeginFrameAck. (Closed)
Patch Set: Created 3 years, 9 months 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
« no previous file with comments | « cc/scheduler/scheduler_unittest.cc ('k') | cc/surfaces/compositor_frame_sink_support_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/surfaces/compositor_frame_sink_support.cc
diff --git a/cc/surfaces/compositor_frame_sink_support.cc b/cc/surfaces/compositor_frame_sink_support.cc
index 0b229c5ae0b4938c99c698d18e1ba92026325ac6..97c351f2f9acc2d5472b096bf2dcfbbf655ef8d6 100644
--- a/cc/surfaces/compositor_frame_sink_support.cc
+++ b/cc/surfaces/compositor_frame_sink_support.cc
@@ -75,8 +75,6 @@ void CompositorFrameSinkSupport::BeginFrameDidNotSwap(
DCHECK_LE(BeginFrameArgs::kStartingFrameNumber, ack.sequence_number);
// |has_damage| is not transmitted, but false by default.
DCHECK(!ack.has_damage);
- // |remaining_frames| is not transmitted, but 0 by default.
- DCHECK_EQ(0u, ack.remaining_frames);
if (begin_frame_source_)
begin_frame_source_->DidFinishFrame(this, ack);
}
@@ -93,9 +91,8 @@ void CompositorFrameSinkSupport::SubmitCompositorFrame(
frame.metadata.begin_frame_ack.sequence_number =
BeginFrameArgs::kStartingFrameNumber;
}
- // |has_damage| and |remaining_frames| are not transmitted.
+ // |has_damage| is not transmitted.
frame.metadata.begin_frame_ack.has_damage = true;
- frame.metadata.begin_frame_ack.remaining_frames = 0;
surface_factory_.SubmitCompositorFrame(
local_surface_id, std::move(frame),
« no previous file with comments | « cc/scheduler/scheduler_unittest.cc ('k') | cc/surfaces/compositor_frame_sink_support_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698