Change createImageBitmap(Blob) to use ImageDecoder instead of ImageSource

Change createImageBitmap(Blob) to use ImageDecoder instead of ImageSource

At this moment, createImageBitmap(Blob) uses ImageSource for image decoding,
which internally uses DeferredImageDecoder. However, the problem is that
we cannot set the AlphaPremultiplied and the GammaAndColorProfileApplied
parameters in this case.

This CL changes the ImageSource to ImageDecoder where we could explicitly
set the premultiplyAlpha and colorProfile parameters.

This change will makes it easy later on when we handle premultiplyAlpha
=false in the case of createImageBitmap(Blob). Note that this CL
doesn't change any behavior, so as long as all layout tests pass, it
should be fine.

BUG=589132
Committed: https://crrev.com/13834522ca6d7969f80743285dff0e09050b8227
Cr-Commit-Position: refs/heads/master@{#378376}

[xidachen, 2016-02-12 15:56:00]

Description was changed from

==========
Change createImageBitmap(Blob) to use ImageDecoder instead of ImageSource

At this moment, createImageBitmap(Blob) uses ImageSource for image decoding,
which internally uses DeferredImageDecoder. However, the problem is that
we cannot set the AlphaPremultiplied and the GammaAndColorProfileApplied
parameters in this case.

This CL changes the ImageSource to ImageDecoder where we could explicitly
set the premultiplyAlpha and colorProfile parameters.
==========

to

==========
Change createImageBitmap(Blob) to use ImageDecoder instead of ImageSource

At this moment, createImageBitmap(Blob) uses ImageSource for image decoding,
which internally uses DeferredImageDecoder. However, the problem is that
we cannot set the AlphaPremultiplied and the GammaAndColorProfileApplied
parameters in this case.

This CL changes the ImageSource to ImageDecoder where we could explicitly
set the premultiplyAlpha and colorProfile parameters.

This change will makes it easy later on when we handle premultiplyAlpha
=false in the case of createImageBitmap(Blob).
==========

[xidachen, 2016-02-12 15:56:00]

xidachen@chromium.org changed reviewers:
+ junov@chromium.org

[xidachen, 2016-02-12 15:56:32]

PTAL

[xidachen, 2016-02-29 17:59:09]

Description was changed from

==========
Change createImageBitmap(Blob) to use ImageDecoder instead of ImageSource

At this moment, createImageBitmap(Blob) uses ImageSource for image decoding,
which internally uses DeferredImageDecoder. However, the problem is that
we cannot set the AlphaPremultiplied and the GammaAndColorProfileApplied
parameters in this case.

This CL changes the ImageSource to ImageDecoder where we could explicitly
set the premultiplyAlpha and colorProfile parameters.

This change will makes it easy later on when we handle premultiplyAlpha
=false in the case of createImageBitmap(Blob).
==========

to

==========
Change createImageBitmap(Blob) to use ImageDecoder instead of ImageSource

At this moment, createImageBitmap(Blob) uses ImageSource for image decoding,
which internally uses DeferredImageDecoder. However, the problem is that
we cannot set the AlphaPremultiplied and the GammaAndColorProfileApplied
parameters in this case.

This CL changes the ImageSource to ImageDecoder where we could explicitly
set the premultiplyAlpha and colorProfile parameters.

This change will makes it easy later on when we handle premultiplyAlpha
=false in the case of createImageBitmap(Blob).

BUG=589132
==========

[xidachen, 2016-02-29 20:09:39]

Description was changed from

==========
Change createImageBitmap(Blob) to use ImageDecoder instead of ImageSource

At this moment, createImageBitmap(Blob) uses ImageSource for image decoding,
which internally uses DeferredImageDecoder. However, the problem is that
we cannot set the AlphaPremultiplied and the GammaAndColorProfileApplied
parameters in this case.

This CL changes the ImageSource to ImageDecoder where we could explicitly
set the premultiplyAlpha and colorProfile parameters.

This change will makes it easy later on when we handle premultiplyAlpha
=false in the case of createImageBitmap(Blob).

BUG=589132
==========

to

==========
Change createImageBitmap(Blob) to use ImageDecoder instead of ImageSource

At this moment, createImageBitmap(Blob) uses ImageSource for image decoding,
which internally uses DeferredImageDecoder. However, the problem is that
we cannot set the AlphaPremultiplied and the GammaAndColorProfileApplied
parameters in this case.

This CL changes the ImageSource to ImageDecoder where we could explicitly
set the premultiplyAlpha and colorProfile parameters.

This change will makes it easy later on when we handle premultiplyAlpha
=false in the case of createImageBitmap(Blob). Note that this CL
doesn't change any behavior, so as long as all layout tests pass, it
should be fine.

BUG=589132
==========

[xidachen, 2016-02-29 20:12:26]

PTAL.

[Justin Novosad, 2016-02-29 21:14:46]

lgtm

[xidachen, 2016-02-29 21:29:36]

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

[commit-bot: I haz the power, 2016-02-29 21:30:08]

Dry run: CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1696753002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1696753002/20001

[commit-bot: I haz the power, 2016-02-29 22:50:31]

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

[commit-bot: I haz the power, 2016-02-29 22:50:32]

Dry run: This issue passed the CQ dry run.

[xidachen, 2016-03-01 02:29:54]

The CQ bit was checked by xidachen@chromium.org

[commit-bot: I haz the power, 2016-03-01 02:30:30]

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

[commit-bot: I haz the power, 2016-03-01 02:37:25]

Description was changed from

==========
Change createImageBitmap(Blob) to use ImageDecoder instead of ImageSource

At this moment, createImageBitmap(Blob) uses ImageSource for image decoding,
which internally uses DeferredImageDecoder. However, the problem is that
we cannot set the AlphaPremultiplied and the GammaAndColorProfileApplied
parameters in this case.

This CL changes the ImageSource to ImageDecoder where we could explicitly
set the premultiplyAlpha and colorProfile parameters.

This change will makes it easy later on when we handle premultiplyAlpha
=false in the case of createImageBitmap(Blob). Note that this CL
doesn't change any behavior, so as long as all layout tests pass, it
should be fine.

BUG=589132
==========

to

==========
Change createImageBitmap(Blob) to use ImageDecoder instead of ImageSource

At this moment, createImageBitmap(Blob) uses ImageSource for image decoding,
which internally uses DeferredImageDecoder. However, the problem is that
we cannot set the AlphaPremultiplied and the GammaAndColorProfileApplied
parameters in this case.

This CL changes the ImageSource to ImageDecoder where we could explicitly
set the premultiplyAlpha and colorProfile parameters.

This change will makes it easy later on when we handle premultiplyAlpha
=false in the case of createImageBitmap(Blob). Note that this CL
doesn't change any behavior, so as long as all layout tests pass, it
should be fine.

BUG=589132
==========

[commit-bot: I haz the power, 2016-03-01 02:37:26]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2016-03-01 02:38:24]

Description was changed from

==========
Change createImageBitmap(Blob) to use ImageDecoder instead of ImageSource

At this moment, createImageBitmap(Blob) uses ImageSource for image decoding,
which internally uses DeferredImageDecoder. However, the problem is that
we cannot set the AlphaPremultiplied and the GammaAndColorProfileApplied
parameters in this case.

This CL changes the ImageSource to ImageDecoder where we could explicitly
set the premultiplyAlpha and colorProfile parameters.

This change will makes it easy later on when we handle premultiplyAlpha
=false in the case of createImageBitmap(Blob). Note that this CL
doesn't change any behavior, so as long as all layout tests pass, it
should be fine.

BUG=589132
==========

to

==========
Change createImageBitmap(Blob) to use ImageDecoder instead of ImageSource

At this moment, createImageBitmap(Blob) uses ImageSource for image decoding,
which internally uses DeferredImageDecoder. However, the problem is that
we cannot set the AlphaPremultiplied and the GammaAndColorProfileApplied
parameters in this case.

This CL changes the ImageSource to ImageDecoder where we could explicitly
set the premultiplyAlpha and colorProfile parameters.

This change will makes it easy later on when we handle premultiplyAlpha
=false in the case of createImageBitmap(Blob). Note that this CL
doesn't change any behavior, so as long as all layout tests pass, it
should be fine.

BUG=589132

Committed: https://crrev.com/13834522ca6d7969f80743285dff0e09050b8227
Cr-Commit-Position: refs/heads/master@{#378376}
==========

[commit-bot: I haz the power, 2016-03-01 02:38:24]

Patchset 2 (id:??) landed as
https://crrev.com/13834522ca6d7969f80743285dff0e09050b8227
Cr-Commit-Position: refs/heads/master@{#378376}