ThreadedTest that a non-null OutputSurface that fails initialization immediately doesn't crash

cc/thread_proxy.cc

Index: cc/thread_proxy.cc
diff --git a/cc/thread_proxy.cc b/cc/thread_proxy.cc
index 13339ab5cd4863f3bfd32e1db7f88385bb6b744a..275bb374fddd97d87af7f4c30a20e9ab1c4d2dcc 100644
--- a/cc/thread_proxy.cc
+++ b/cc/thread_proxy.cc
@@ -1012,7 +1012,8 @@ bool ThreadProxy::commitPendingForTesting()
 void ThreadProxy::commitPendingOnImplThreadForTesting(CommitPendingRequest* request)
 {
     DCHECK(isImplThread());
-    request->commitPending = m_schedulerOnImplThread->commitPending();
+    if (m_layerTreeHostImpl->outputSurface())
+        request->commitPending = m_schedulerOnImplThread->commitPending();

[danakj, 2013/01/09 01:39:59]

else commitPending = false;

I suspect you're leaving tests hanging because this variable is left
uninitialized.

[jamesr, 2013/01/09 19:35:41]

D'oh!  PTAL

     request->completion.signal();
 }