Synchronously drop tile textures in android webview

Synchronously drop tile textures in android webview

Merely applying zero memory is not enough to synchronously drop all tile
textures. Instead destroy and recreate TileManager instead.

BUG=496057, 499004
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/3d442d362ddec4594dc12983d88bdbe24f1c361c
Cr-Commit-Position: refs/heads/master@{#334020}

[boliu, 2015-06-10 17:07:37]

boliu@chromium.org changed reviewers:
+ vmpstr@chromium.org

[boliu, 2015-06-10 21:44:44]

ptal

Note intending to merge this back to m44.

[boliu, 2015-06-10 22:06:02]

boliu@chromium.org changed reviewers:
+ danakj@chromium.org

[boliu, 2015-06-10 22:06:03]

+danakj

[vmpstr, 2015-06-10 23:15:54]

Overall this looks fine to me, can you add a TODO(boliu) or TODO(vmpstr) to the
comment as well, so that there's some type of ownership :)

https://codereview.chromium.org/1176713002/diff/40001/cc/trees/layer_tree_hos...
File cc/trees/layer_tree_host_impl.cc (right):

https://codereview.chromium.org/1176713002/diff/40001/cc/trees/layer_tree_hos...
cc/trees/layer_tree_host_impl.cc:1337: DestroyTileManager();
Can you make a comment that explains that destruction of tile manager will
synchronously wait for all tiles to be canceled/completed.

Also, this doesn't do any flushes or anything like that, do we need that as
well?

[boliu, 2015-06-10 23:20:56]

https://codereview.chromium.org/1176713002/diff/40001/cc/trees/layer_tree_hos...
File cc/trees/layer_tree_host_impl.cc (right):

https://codereview.chromium.org/1176713002/diff/40001/cc/trees/layer_tree_hos...
cc/trees/layer_tree_host_impl.cc:1337: DestroyTileManager();
On 2015/06/10 23:15:54, vmpstr wrote:
> Can you make a comment that explains that destruction of tile manager will
> synchronously wait for all tiles to be canceled/completed.

Done

> 
> Also, this doesn't do any flushes or anything like that, do we need that as
> well?
>  

That was added last week in https://codereview.chromium.org/1162793005/ :)

[vmpstr, 2015-06-11 17:35:26]

lgtm

[boliu, 2015-06-11 18:59:35]

The CQ bit was checked by boliu@chromium.org

[commit-bot: I haz the power, 2015-06-11 18:59:48]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1176713002/60001

[commit-bot: I haz the power, 2015-06-11 20:01:39]

Committed patchset #4 (id:60001)

[commit-bot: I haz the power, 2015-06-11 20:03:39]

Patchset 4 (id:??) landed as
https://crrev.com/3d442d362ddec4594dc12983d88bdbe24f1c361c
Cr-Commit-Position: refs/heads/master@{#334020}