cc: Remove "SwapBuffers" naming from LayerTreeHostSingleThreadClient

content/renderer/render_widget.cc

Index: content/renderer/render_widget.cc
diff --git a/content/renderer/render_widget.cc b/content/renderer/render_widget.cc
index ad847c86a3ddd8b1054489ea31f3c3c236bdabe6..9bccbfc603193a9c0e6dac8cfa7eaa3aedec2754 100644
--- a/content/renderer/render_widget.cc
+++ b/content/renderer/render_widget.cc
@@ -782,8 +782,8 @@ void RenderWidget::DidCommitCompositorFrame() {
 
 void RenderWidget::DidCompletePageScaleAnimation() {}
 
-void RenderWidget::DidCompleteSwapBuffers() {
-  TRACE_EVENT0("renderer", "RenderWidget::DidCompleteSwapBuffers");
+void RenderWidget::DidReceiveCompositorFrameAck() {
+  TRACE_EVENT0("renderer", "RenderWidget::DidReceiveCompositorFrameAck");
 
   // Notify subclasses threaded composited rendering was flushed to the screen.
   DidFlushPaint();
@@ -809,23 +809,6 @@ bool RenderWidget::IsClosing() const {
   return host_closing_;
 }
 
-void RenderWidget::OnSwapBuffersAborted() {
-  TRACE_EVENT0("renderer", "RenderWidget::OnSwapBuffersAborted");
-  // Schedule another frame so the compositor learns about it.
-  ScheduleComposite();

[danakj, 2016/10/21 23:36:52]

This would possibly prevent context recovery because we have to composite to get
a context, but layout tests don't lose context and I NOTREACHED() that this is
the case and bots will inform me otherwise.

Oh, and ScheduleComposite does nothing unless
GetCompositorImplThreadTaskRunner() is true which it isnt in single thread
anyway so I think this is all no-ops.

[piman, 2016/10/21 23:44:22]

I think most uses of ScheduleComposite are wrong (either redundant or not
sufficient if it was needed), so I'm pretty ok with that

-}
-
-void RenderWidget::OnSwapBuffersComplete() {
-  TRACE_EVENT0("renderer", "RenderWidget::OnSwapBuffersComplete");
-
-  // Notify subclasses that composited rendering was flushed to the screen.
-  DidFlushPaint();

[danakj, 2016/10/21 23:36:52]

This looks scary but RWC calls DidCompleteSwapBuffers right before calling
OnSwapBuffersAborted, and it does the same thing, so we were doing this twice.

-}
-
-void RenderWidget::OnSwapBuffersPosted() {
-  TRACE_EVENT0("renderer", "RenderWidget::OnSwapBuffersPosted");
-}
-
 void RenderWidget::RequestScheduleAnimation() {
   scheduleAnimation();
 }