Remove NULL ResourceProvider, TileManager early outs

Remove NULL ResourceProvider, TileManager early outs

These early outs were added in r201247 as part of resourceless software
mode in which ResourceProvider and TileManager were not created. This
is no longer the case after r207424, so remove these early outs.

For Android WebView, this does not affect software mode, and does not
make hardware any more broken than it already is.

BUG=245935
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=207673

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

https://codereview.chromium.org/17176012/diff/1/cc/trees/layer_tree_host_impl.cc
File cc/trees/layer_tree_host_impl.cc (left):

https://codereview.chromium.org/17176012/diff/1/cc/trees/layer_tree_host_impl...
cc/trees/layer_tree_host_impl.cc:754: if (renderer_) {
I'm not sure why this was added. Renderer was always created.

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

LGTM

https://codereview.chromium.org/17176012/diff/1/cc/trees/layer_tree_host_impl.cc
File cc/trees/layer_tree_host_impl.cc (right):

https://codereview.chromium.org/17176012/diff/1/cc/trees/layer_tree_host_impl...
cc/trees/layer_tree_host_impl.cc:752: if
(!output_surface_->ForcedDrawToSoftwareDevice())
Should this go away (in another CL)?

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

https://codereview.chromium.org/17176012/diff/1/cc/trees/layer_tree_host_impl.cc
File cc/trees/layer_tree_host_impl.cc (right):

https://codereview.chromium.org/17176012/diff/1/cc/trees/layer_tree_host_impl...
cc/trees/layer_tree_host_impl.cc:752: if
(!output_surface_->ForcedDrawToSoftwareDevice())
On 2013/06/20 18:00:33, danakj wrote:
> Should this go away (in another CL)?

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.

[commit-bot: I haz the power, 2013-06-20 21:02:41]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/boliu@chromium.org/17176012/5001

[commit-bot: I haz the power, 2013-06-21 02:49:30]

Change committed as 207673