Remove NULL ResourceProvider, TileManager early outs

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 ffd48bffcd3dd098d7fd757450d0f119cb0c5075..94653f77165bda80a1dd5eb557174db39b31c494 100644
--- a/cc/trees/layer_tree_host_impl.cc
+++ b/cc/trees/layer_tree_host_impl.cc
@@ -751,10 +751,8 @@ bool LayerTreeHostImpl::CalculateRenderPasses(FrameData* frame) {
   RemoveRenderPasses(CullRenderPassesWithNoQuads(), frame);
   if (!output_surface_->ForcedDrawToSoftwareDevice())

[danakj, 2013/06/20 18:00:33]

Should this go away (in another CL)?

[boliu, 2013/06/20 21:01:43]

I think so after the WillDraw change that skips drawing most layers in software
mode.

One bit of concern is that if this is a software draw after hardware has been
initialized, we might be freeing textures here. However we are already doing
this all over the place anyway and one more won't hurt for now. Planned fix is
to queue up commands and pump commands only in a hardware draw or this other
"process mode" callback that webview can request at any time. In this process
mode callback, the context current, but we are not allowed to draw.

     renderer_->DecideRenderPassAllocationsForFrame(frame->render_passes);
-  if (renderer_) {

[boliu, 2013/06/20 17:13:53]

I'm not sure why this was added. Renderer was always created.

-    RemoveRenderPasses(CullRenderPassesWithCachedTextures(renderer_.get()),
-                       frame);
-  }
+  RemoveRenderPasses(CullRenderPassesWithCachedTextures(renderer_.get()),
+                     frame);
 
   // If we're making a frame to draw, it better have at least one render pass.
   DCHECK(!frame->render_passes.empty());
@@ -1218,8 +1216,7 @@ void LayerTreeHostImpl::DidDrawAllLayers(const FrameData& frame) {
 
   // Once all layers have been drawn, pending texture uploads should no
   // longer block future uploads.
-  if (resource_provider_)
-    resource_provider_->MarkPendingUploadsAsNonBlocking();
+  resource_provider_->MarkPendingUploadsAsNonBlocking();
 }
 
 void LayerTreeHostImpl::FinishAllRendering() {