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

Unified Diff: content/browser/renderer_host/render_widget_host_impl.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
Index: content/browser/renderer_host/render_widget_host_impl.cc
diff --git a/content/browser/renderer_host/render_widget_host_impl.cc b/content/browser/renderer_host/render_widget_host_impl.cc
index 3eed18a686a989d488de4d6c367a5d9b7dc0eced..45aa1a69b468658390a50e3569a002ad0896db34 100644
--- a/content/browser/renderer_host/render_widget_host_impl.cc
+++ b/content/browser/renderer_host/render_widget_host_impl.cc
@@ -1878,9 +1878,8 @@ bool RenderWidgetHostImpl::OnSwapCompositorFrame(
bad_message::RWH_INVALID_BEGIN_FRAME_ACK_COMPOSITOR_FRAME);
return false;
}
- // |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;
if (!ui::LatencyInfo::Verify(frame.metadata.latency_info,
"RenderWidgetHostImpl::OnSwapCompositorFrame")) {
@@ -1942,10 +1941,9 @@ void RenderWidgetHostImpl::OnBeginFrameDidNotSwap(
return;
}
- // |has_damage| and |remaining_frames| are not transmitted.
+ // |has_damage| is not transmitted.
cc::BeginFrameAck modified_ack = ack;
modified_ack.has_damage = false;
- modified_ack.remaining_frames = 0;
if (view_)
view_->OnBeginFrameDidNotSwap(modified_ack);

Powered by Google App Engine
This is Rietveld 408576698