cc: Rename SwapBuffers on CompositorFrameSink to SubmitCompositorFrame

services/ui/public/cpp/compositor_frame_sink.cc

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "services/ui/public/cpp/compositor_frame_sink.h"
 
 #include "base/bind.h"
 #include "cc/output/compositor_frame.h"
 #include "cc/output/compositor_frame_sink_client.h"
 #include "gpu/ipc/client/gpu_channel_host.h"
@@ skipping 30 matching lines @@
   return true;
 }
 
 void CompositorFrameSink::DetachFromClient() {
   client_->SetBeginFrameSource(nullptr);
   begin_frame_source_.reset();
   surface_.reset();
   cc::CompositorFrameSink::DetachFromClient();
 }
 
-void CompositorFrameSink::SwapBuffers(cc::CompositorFrame frame) {
+void CompositorFrameSink::SubmitCompositorFrame(cc::CompositorFrame frame) {
   // CompositorFrameSink owns WindowSurface, and so if CompositorFrameSink is
   // destroyed then SubmitCompositorFrame's callback will never get called.
   // Thus, base::Unretained is safe here.
   surface_->SubmitCompositorFrame(
-      std::move(frame), base::Bind(&CompositorFrameSink::SwapBuffersComplete,
-                                   base::Unretained(this)));
+      std::move(frame),
+      base::Bind(&CompositorFrameSink::SubmitCompositorFrameAck,

[sky, 2016/10/11 02:43:25]

Any idea why this doesn't supply client_ directly?

[danakj, 2016/10/13 22:02:48]

In similar cases its to use a weakptr but no reason to here.

+                 base::Unretained(this)));
 }
 
 void CompositorFrameSink::OnResourcesReturned(
     ui::WindowSurface* surface,
     mojo::Array<cc::ReturnedResource> resources) {
   client_->ReclaimResources(resources.To<cc::ReturnedResourceArray>());
 }
 
-void CompositorFrameSink::SwapBuffersComplete() {
-  client_->DidSwapBuffersComplete();
+void CompositorFrameSink::SubmitCompositorFrameAck() {
+  client_->DidReceiveCompositorFrameAck();
 }
 
 }  // namespace ui