cc: Disable image hijack canvas when we don't have images in the map.

cc: Disable image hijack canvas when we don't have images in the map.

This patch ensures that we don't use the hijack canvas when there are
no images in the discardable map, since this is the way we "disable"
things on android.

Note we already do this in PlaybackToSharedCanvas, so this is just
ensuring that we also do that in PlaybackToCanvas.

BUG=591246
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/afb46c11bdde02fe8cd4a76ae5b08e30543d6255
Cr-Commit-Position: refs/heads/master@{#378913}

[vmpstr, 2016-03-02 23:48:00]

Description was changed from

==========
cc: Disable image hijack canvas when we don't have images in the map.

This patch ensures that we don't use the hijack canvas when there are
no images in the discardable map, since this is the way we "disable"
things on android.

Note we already do this in PlaybackToSharedCanvas, so this is just
ensuring that we also do that in PlaybackToCanvas.

BUG=591246
==========

to

==========
cc: Disable image hijack canvas when we don't have images in the map.

This patch ensures that we don't use the hijack canvas when there are
no images in the discardable map, since this is the way we "disable"
things on android.

Note we already do this in PlaybackToSharedCanvas, so this is just
ensuring that we also do that in PlaybackToCanvas.

BUG=591246
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

[vmpstr, 2016-03-02 23:48:35]

vmpstr@chromium.org changed reviewers:
+ enne@chromium.org, ericrk@chromium.org

[vmpstr, 2016-03-02 23:48:35]

Please take a look.

[ericrk, 2016-03-03 00:27:39]

lgtm

[vmpstr, 2016-03-03 00:30:36]

The CQ bit was checked by vmpstr@chromium.org

[commit-bot: I haz the power, 2016-03-03 00:30:57]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1756413002/1
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1756413002/1

[enne (OOO), 2016-03-03 01:19:40]

lgtm2, but wow this is worrisome for how expensive the canvas is.

[vmpstr, 2016-03-03 01:21:17]

On 2016/03/03 01:19:40, enne wrote:
> lgtm2, but wow this is worrisome for how expensive the canvas is.

It's not so much the canvas as the fact that now that we're handling low filter
quality, we're doing at raster decodes. I've filed crbug.com/591571 to improve
the performance.

[commit-bot: I haz the power, 2016-03-03 01:48:39]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-03-03 01:49:45]

Description was changed from

==========
cc: Disable image hijack canvas when we don't have images in the map.

This patch ensures that we don't use the hijack canvas when there are
no images in the discardable map, since this is the way we "disable"
things on android.

Note we already do this in PlaybackToSharedCanvas, so this is just
ensuring that we also do that in PlaybackToCanvas.

BUG=591246
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel
==========

to

==========
cc: Disable image hijack canvas when we don't have images in the map.

This patch ensures that we don't use the hijack canvas when there are
no images in the discardable map, since this is the way we "disable"
things on android.

Note we already do this in PlaybackToSharedCanvas, so this is just
ensuring that we also do that in PlaybackToCanvas.

BUG=591246
CQ_INCLUDE_TRYBOTS=tryserver.blink:linux_blink_rel

Committed: https://crrev.com/afb46c11bdde02fe8cd4a76ae5b08e30543d6255
Cr-Commit-Position: refs/heads/master@{#378913}
==========

[commit-bot: I haz the power, 2016-03-03 01:49:46]

Patchset 1 (id:??) landed as
https://crrev.com/afb46c11bdde02fe8cd4a76ae5b08e30543d6255
Cr-Commit-Position: refs/heads/master@{#378913}