content: Use DidCommitAndDraw to ack resize
Once the renderer generates a CompositorFrame after resize,
there is really nothing more it can do and it should start the next
resize. Waiting for DidReceiveCompositorFrameAck wastes renderer's time
(2 IPC messages + the time it takes for Viz to ack a frame) and doesn't
provide any value.
Change-Id: Iba1d9c7862b6540fc2133e787c8c531d53e0c845
Reviewed-on: https://chromium-review.googlesource.com/882612
Reviewed-by: Fady Samuel <[email protected]>
Reviewed-by: Antoine Labour <[email protected]>
Commit-Queue: Saman Sami <[email protected]>
Cr-Commit-Position: refs/heads/master@{#531774}diff --git a/content/renderer/render_widget.cc b/content/renderer/render_widget.cc
index 12c267b..b318bd3 100644
--- a/content/renderer/render_widget.cc
+++ b/content/renderer/render_widget.cc
@@ -1016,6 +1016,8 @@
// tab_capture_performancetest.cc.
TRACE_EVENT0("gpu", "RenderWidget::DidCommitAndDrawCompositorFrame");
+ DidResizeOrRepaintAck();
+
for (auto& observer : render_frames_)
observer.DidCommitAndDrawCompositorFrame();
@@ -1028,8 +1030,6 @@
void RenderWidget::DidCompletePageScaleAnimation() {}
void RenderWidget::DidReceiveCompositorFrameAck() {
- TRACE_EVENT0("renderer", "RenderWidget::DidReceiveCompositorFrameAck");
- DidResizeOrRepaintAck();
}
bool RenderWidget::IsClosing() const {
