Use PaintImage in cc QueueImageDecode

Use PaintImage in cc QueueImageDecode

This replaces current SKImage usage in QueueImageDecode and
helps setting up the right PaintImage ID to be used in
decode queue/cache.

BUG=722559, 695108
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel

[sohan, 2017-06-06 17:07:48]

Description was changed from

==========
Plumb PaintImage from Blink to cc ImageController

Plumb PaintImage from Blink to be received by QueueImageDecode.
This replaces current SKImage usage and helps setting up the
right PaintImage ID to be used in decode queue/cache.

BUG=722559,695108
==========

to

==========
Plumb PaintImage from Blink to cc ImageController

Plumb PaintImage from Blink to be received by QueueImageDecode.
This replaces current SKImage usage and helps setting up the
right PaintImage ID to be used in decode queue/cache.

BUG=722559,695108
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[sohanjg_, 2017-06-06 17:12:30]

Description was changed from

==========
Plumb PaintImage from Blink to cc ImageController

Plumb PaintImage from Blink to be received by QueueImageDecode.
This replaces current SKImage usage and helps setting up the
right PaintImage ID to be used in decode queue/cache.

BUG=722559,695108
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Plumb PaintImage from Blink to cc ImageController

Plumb PaintImage from Blink to be received by QueueImageDecode.
This replaces current SKImage usage and helps setting up the
right PaintImage ID to be used in decode queue/cache.

BUG=722559,695108
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[vmpstr, 2017-06-06 18:00:09]

Hmm. This is conflating a few different patches together I think. I actually was
working on the patch to plumb the id from Blink myself over here:
https://codereview.chromium.org/2929493002.

Do you mind rebasing on top of that? The rest of the changes look reasonable,
and I can review after you rebase.

[sohanjg_, 2017-06-06 18:17:28]

On 2017/06/06 18:00:09, vmpstr wrote:
> Hmm. This is conflating a few different patches together I think. I actually
was
> working on the patch to plumb the id from Blink myself over here:
> https://codereview.chromium.org/2929493002.
> 
> Do you mind rebasing on top of that? The rest of the changes look reasonable,
> and I can review after you rebase.

Ahh..sorry for the confusion. I'll rebase the ImageController changes on top of
your patch.

[sohan, 2017-06-07 12:13:53]

Description was changed from

==========
Plumb PaintImage from Blink to cc ImageController

Plumb PaintImage from Blink to be received by QueueImageDecode.
This replaces current SKImage usage and helps setting up the
right PaintImage ID to be used in decode queue/cache.

BUG=722559,695108
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

to

==========
Use PaintImage in cc QueueImageDecode

This replaces current SKImage usage in QueueImageDecode and
helps setting up the right PaintImage ID to be used in
decode queue/cache.

BUG=722559,695108
CQ_INCLUDE_TRYBOTS=master.tryserver.blink:linux_trusty_blink_rel
==========

[sohanjg_, 2017-06-07 12:27:24]

The CQ bit was checked by sohanjg@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-06-07 12:27:30]

Dry run: CQ is trying da patch.

Follow status at:
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[sohan, 2017-06-07 13:35:09]

sohan.jyoti@huawei.com changed reviewers:
+ khushalsagar@chromium.org, vmpstr@chromium.org
- sohanjg@chromium.org

[sohan, 2017-06-07 13:35:09]

Please take a look, thanks.

[commit-bot: I haz the power, 2017-06-07 13:36:09]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-06-07 13:36:09]

Dry run: This issue passed the CQ dry run.

[Khushal, 2017-06-07 18:30:39]

On 2017/06/07 13:35:09, sohan wrote:
> Please take a look, thanks.

Thanks for taking this up Sohan. But I already had to make that interface take a
DrawImage in order to scale it here:
https://codereview.chromium.org/2928433003/. :(

[sohan, 2017-06-08 01:24:48]

On 2017/06/07 18:30:39, Khushal wrote:
> On 2017/06/07 13:35:09, sohan wrote:
> > Please take a look, thanks.
> 
> Thanks for taking this up Sohan. But I already had to make that interface take
a
> DrawImage in order to scale it here:
> https://codereview.chromium.org/2928433003/. :(

Ahh i see..sorry for the confusion :/