Bring skydebugger closer to clean-shutdown

mojo/cc/output_surface_mojo.cc

Index: mojo/cc/output_surface_mojo.cc
diff --git a/mojo/cc/output_surface_mojo.cc b/mojo/cc/output_surface_mojo.cc
index 193c7acb78649d606cfcbcc9c954bf111e44141f..e3b87e1ee4e0bd4cbbd7d7fe44788f3cfc247831 100644
--- a/mojo/cc/output_surface_mojo.cc
+++ b/mojo/cc/output_surface_mojo.cc
@@ -25,6 +25,8 @@ OutputSurfaceMojo::OutputSurfaceMojo(
 }
 
 OutputSurfaceMojo::~OutputSurfaceMojo() {
+  if (!surface_id_.is_null())
+    surface_->DestroySurface(SurfaceId::From(surface_id_));

[eseidel, 2014/11/05 18:27:08]

My understanding is that this is necessary based on reading SurfaceService code
(which does not know how to destroy surfaces when you close its pipe!).  We
don't actually hit this code yet because the process errors out earlier in
shutdown than this.

 }
 
 void OutputSurfaceMojo::ReturnResources(Array<ReturnedResourcePtr> resources) {