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..306137e8520c160ea21d380637d5cdb1daf6ecfd 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,8 @@ void SoftwareBrowserCompositorOutputSurface::SwapBuffers(
                    output_surface_proxy_,
                    surface_id_));
   }
+  client_->DidSwapBuffers();

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

Ooh this seems really important if we're going to use the scheduler. In the
software case we'd hit max pending swaps and just stop swapping?

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

Currently in compositor.cc without DidSwapBuffers() we never change the
swap_state_ i.e. don't think we ever swapped so we also don't end up limiting
ourselves.

I think it worked because both calls were missing. If I add only either one it
will trigger DCHECKs().

+  PostSwapBuffersComplete();

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

can you post first before DidSwap like we do in other places?

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

Done.

 }
 
 }  // namespace content