Move context-related callbacks into OutputSurface

cc/trees/layer_tree_host_impl.cc

Index: cc/trees/layer_tree_host_impl.cc
diff --git a/cc/trees/layer_tree_host_impl.cc b/cc/trees/layer_tree_host_impl.cc
index f2fb51cc61e356176a112e9826a8bc687942c146..d17e4ac1c64075ac148cf6a9ab26b5b263d0d29d 100644
--- a/cc/trees/layer_tree_host_impl.cc
+++ b/cc/trees/layer_tree_host_impl.cc
@@ -1052,7 +1052,8 @@ const LayerTreeSettings& LayerTreeHostImpl::Settings() const {
 }
 
 void LayerTreeHostImpl::DidLoseOutputSurface() {
-  client_->DidLoseOutputSurfaceOnImplThread();
+  if (renderer_)

[enne (OOO), 2013/03/25 18:29:19]

Why do you need to not send this notification if there's no renderer?

[danakj, 2013/03/25 18:40:43]

The unit tests were built in a world where losing the context when there's no
renderer_ would not call this, since the callback came through the renderer
before.

Should we TODO() to adjust the tests to expect this instead?

+    client_->DidLoseOutputSurfaceOnImplThread();
 }
 
 void LayerTreeHostImpl::OnSwapBuffersComplete() {