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

Unified Diff: content/browser/renderer_host/delegated_frame_host.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: content/browser/renderer_host/delegated_frame_host.cc
diff --git a/content/browser/renderer_host/delegated_frame_host.cc b/content/browser/renderer_host/delegated_frame_host.cc
index 937512feb8f87e119ecece2d10ceb85dfcd81dad..a2b10ebf7c9552a5d449c0613091e6f60f52a868 100644
--- a/content/browser/renderer_host/delegated_frame_host.cc
+++ b/content/browser/renderer_host/delegated_frame_host.cc
@@ -219,7 +219,8 @@ void DelegatedFrameHost::SetNeedsBeginFrames(bool needs_begin_frames) {
support_->SetNeedsBeginFrame(needs_begin_frames);
}
-void DelegatedFrameHost::BeginFrameDidNotSwap(const cc::BeginFrameAck& ack) {
+void DelegatedFrameHost::BeginFrameDidNotProduceFrame(
+ const cc::BeginFrameAck& ack) {
DidFinishFrame(ack);
cc::BeginFrameAck modified_ack = ack;
@@ -232,7 +233,7 @@ void DelegatedFrameHost::BeginFrameDidNotSwap(const cc::BeginFrameAck& ack) {
latest_confirmed_begin_frame_sequence_number_;
}
- support_->BeginFrameDidNotSwap(modified_ack);
+ support_->BeginFrameDidNotProduceFrame(modified_ack);
}
bool DelegatedFrameHost::ShouldSkipFrame(const gfx::Size& size_in_dip) {
@@ -421,7 +422,7 @@ void DelegatedFrameHost::SubmitCompositorFrame(
renderer_compositor_frame_sink_->DidReceiveCompositorFrameAck(resources);
skipped_frames_ = true;
- BeginFrameDidNotSwap(ack);
+ BeginFrameDidNotProduceFrame(ack);
return;
}

Powered by Google App Engine
This is Rietveld 408576698