Call DidPostSwapBuffers and SwapBuffersComplete for browser software surface

content/browser/compositor/browser_compositor_output_surface.cc

Index: content/browser/compositor/browser_compositor_output_surface.cc
diff --git a/content/browser/compositor/browser_compositor_output_surface.cc b/content/browser/compositor/browser_compositor_output_surface.cc
index b1e550162f026e98b0046e7698517ba47579e7e8..8607cfde94e65dd68b3e1d3a48b7afcda6a01c19 100644
--- a/content/browser/compositor/browser_compositor_output_surface.cc
+++ b/content/browser/compositor/browser_compositor_output_surface.cc
@@ -68,14 +68,6 @@ bool BrowserCompositorOutputSurface::BindToClient(
   return true;
 }
 
-void BrowserCompositorOutputSurface::OnSwapBuffersComplete() {
-  // On Mac, delay acknowledging the swap to the output surface client until
-  // it has been drawn.
-#if !defined(OS_MACOSX)
-  cc::OutputSurface::OnSwapBuffersComplete();
-#endif
-}
-
 void BrowserCompositorOutputSurface::OnUpdateVSyncParameters(
     base::TimeTicks timebase,
     base::TimeDelta interval) {
@@ -98,7 +90,7 @@ void BrowserCompositorOutputSurface::SetReflector(ReflectorImpl* reflector) {
 
 #if defined(OS_MACOSX)
 void BrowserCompositorOutputSurface::OnSurfaceDisplayed() {
-  cc::OutputSurface::OnSwapBuffersComplete();
+  NOTREACHED();

[danakj, 2014/10/08 19:32:29]

how about making it pure virtual and NOTREACHED in the software implementation
with a comment about why?

[no sievers, 2014/10/08 19:59:00]

Done.

 }
 #endif