cc: Get rid of PostSwapBuffersComplete.

content/browser/compositor/vulkan_browser_compositor_output_surface.cc

 // Copyright (c) 2016 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 "content/browser/compositor/vulkan_browser_compositor_output_surface.h"
 
 #include "cc/output/output_surface_client.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
 #include "gpu/vulkan/vulkan_surface.h"
 
@@ skipping 30 matching lines @@
     surface_->Destroy();
     surface_.reset();
   }
 }
 
 void VulkanBrowserCompositorOutputSurface::OnGpuSwapBuffersCompleted(
     const std::vector<ui::LatencyInfo>& latency_info,
     gfx::SwapResult result,
     const gpu::GpuProcessHostedCALayerTreeParamsMac* params_mac) {
   RenderWidgetHostImpl::CompositorFrameDrawn(latency_info);
-  OnSwapBuffersComplete();
+  client_->DidSwapBuffersComplete();
 }
 
 void VulkanBrowserCompositorOutputSurface::SwapBuffers(
     cc::CompositorFrame* frame) {
   surface_->SwapBuffers();
-  PostSwapBuffersComplete();

[boliu, 2016/10/09 22:55:08]

was the previous thing calling DidSwapBuffersComplete twice per frame? by
mistake?

[danakj, 2016/10/10 20:28:19]

I think so but this was never seriously run yet, it may be that
OnGpuSwapBuffersCompleted isn't hooked up, but we'll find that out quickly
if/when we use this class again.

 }
 
 }  // namespace content