Call DidPostSwapBuffers and SwapBuffersComplete for browser software surface

content/browser/compositor/software_browser_compositor_output_surface.cc

Index: content/browser/compositor/software_browser_compositor_output_surface.cc
diff --git a/content/browser/compositor/software_browser_compositor_output_surface.cc b/content/browser/compositor/software_browser_compositor_output_surface.cc
index 369fe930dbdc31522d0c2139a1fab429a3498945..98c707f66114135ba37121080ae622bc24148c08 100644
--- a/content/browser/compositor/software_browser_compositor_output_surface.cc
+++ b/content/browser/compositor/software_browser_compositor_output_surface.cc
@@ -8,6 +8,7 @@
 #include "base/message_loop/message_loop.h"
 #include "base/time/time.h"
 #include "cc/output/compositor_frame.h"
+#include "cc/output/output_surface_client.h"
 #include "cc/output/software_output_device.h"
 #include "content/browser/compositor/browser_compositor_output_surface_proxy.h"
 #include "content/browser/renderer_host/render_widget_host_impl.h"
@@ -51,6 +52,15 @@ void SoftwareBrowserCompositorOutputSurface::SwapBuffers(
                    output_surface_proxy_,
                    surface_id_));
   }
+  PostSwapBuffersComplete();
+  client_->DidSwapBuffers();

[danakj, 2014/10/08 20:36:53]

wei has confirmed that we already do get to DidSwapBuffers code somehow in
software compositor, so I unchecked the CQ while she figures out what that stack
trace looks like, we don't wanna call it twice!

[danakj, 2014/10/08 20:48:56]

Ahhh ok so right now we don't tell the scheduler DidSwap or DidSwapComplete, but
it somehow just works anyways. This will add both, so that's great.

 }
 
+#if defined(OS_MACOSX)
+void SoftwareBrowserCompositorOutputSurface::OnSurfaceDisplayed() {
+  // See GpuBrowserCompositorOutputSurface for when and how this is used.
+  NOTREACHED() << "Not expected for software surfaces.";
+}
+#endif
+
 }  // namespace content