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())

[danakj, 2013/01/08 01:06:50]

Ya, I was thinking about something like this also. I'm fine with this, as it
shouldn't introduce races. You know that no commit is possibly sitting in a
message loop queue, so that sounds reasonable to me.

+        request->commitPending = m_schedulerOnImplThread->commitPending();
     request->completion.signal();
 }