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

Unified Diff: cc/surfaces/direct_compositor_frame_sink.cc

Issue 2888043004: [cc] Add and plumb CFS::DidNotProduceFrame. (Closed)
Patch Set: remove ExternalBFS::OnDidFinishFrame and related ack tracking. Created 3 years, 7 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
Index: cc/surfaces/direct_compositor_frame_sink.cc
diff --git a/cc/surfaces/direct_compositor_frame_sink.cc b/cc/surfaces/direct_compositor_frame_sink.cc
index 505d38ad0914acc436e7641685d1489688b9f16d..196c813bfeb424fc190a1e792ad46c1b2b9eb582 100644
--- a/cc/surfaces/direct_compositor_frame_sink.cc
+++ b/cc/surfaces/direct_compositor_frame_sink.cc
@@ -110,6 +110,13 @@ void DirectCompositorFrameSink::SubmitCompositorFrame(CompositorFrame frame) {
std::move(frame));
}
+void DirectCompositorFrameSink::BeginFrameDidNotProduceFrame(
+ const BeginFrameAck& ack) {
+ DCHECK(!ack.has_damage);
+ DCHECK_LE(BeginFrameArgs::kStartingFrameNumber, ack.sequence_number);
+ support_->BeginFrameDidNotProduceFrame(ack);
+}
+
void DirectCompositorFrameSink::DisplayOutputSurfaceLost() {
is_lost_ = true;
client_->DidLoseCompositorFrameSink();
@@ -152,10 +159,4 @@ void DirectCompositorFrameSink::OnNeedsBeginFrames(bool needs_begin_frame) {
support_->SetNeedsBeginFrame(needs_begin_frame);
}
-void DirectCompositorFrameSink::OnDidFinishFrame(const BeginFrameAck& ack) {
- // If there was damage, SubmitCompositorFrame includes the ack.
- if (!ack.has_damage)
- support_->BeginFrameDidNotSwap(ack);
-}
-
} // namespace cc

Powered by Google App Engine
This is Rietveld 408576698