Revert of Revise the code for premultiplyAlpha=false for createImageBitmap(HTMLCanvasElement)

Revert of Revise the code for premultiplyAlpha=false for createImageBitmap(HTMLCanvasElement) (patchset #1 id:1 of https://codereview.chromium.org/1694793002/ )

Reason for revert:
Cause compilation error
https://build.chromium.org/p/chromium.mac/builders/Mac%20Builder/builds/37356/steps/compile/logs/stdio

Original issue's description:
> Revise the code for premultiplyAlpha=false for createImageBitmap(HTMLCanvasElement)
>
> It turns out that HTMLCanvasElement is always premultiplied in Blink, so
> we cannot simply use a decoder to re-decode it to a un-premultiplied format.
> The strategy this CL takes is that after cropping the HTMLCanvasElement
> into a SkImage, we convert the SkImage into a un-premultiplied format if
> that's what the option specifies.

TBR=junov@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

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

[xidachen, 2016-02-12 16:33:42]

Created Revert of Revise the code for premultiplyAlpha=false for
createImageBitmap(HTMLCanvasElement)

[commit-bot: I haz the power, 2016-02-12 16:33:48]

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

[commit-bot: I haz the power, 2016-02-12 16:33:49]

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

[commit-bot: I haz the power, 2016-02-12 16:33:50]

No L-G-T-M from a valid reviewer yet. Only full committers are accepted.
Even if an L-G-T-M may have been provided, it was from a non-committer,
_not_ a full super star committer.
See http://www.chromium.org/getting-involved/become-a-committer
Note that this has nothing to do with OWNERS files.

[jbroman, 2016-02-12 16:33:59]

jbroman@chromium.org changed reviewers:
+ jbroman@chromium.org

[jbroman, 2016-02-12 16:34:00]

lgtm

[jbroman, 2016-02-12 16:34:01]

The CQ bit was checked by jbroman@chromium.org

[commit-bot: I haz the power, 2016-02-12 16:34:20]

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

[commit-bot: I haz the power, 2016-02-12 16:35:17]

Description was changed from

==========
Revert of Revise the code for premultiplyAlpha=false for
createImageBitmap(HTMLCanvasElement) (patchset #1 id:1 of
https://codereview.chromium.org/1694793002/ )

Reason for revert:
Cause compilation error
https://build.chromium.org/p/chromium.mac/builders/Mac%20Builder/builds/37356...

Original issue's description:
> Revise the code for premultiplyAlpha=false for
createImageBitmap(HTMLCanvasElement)
> 
> It turns out that HTMLCanvasElement is always premultiplied in Blink, so
> we cannot simply use a decoder to re-decode it to a un-premultiplied format.
> The strategy this CL takes is that after cropping the HTMLCanvasElement
> into a SkImage, we convert the SkImage into a un-premultiplied format if
> that's what the option specifies.

TBR=junov@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Revert of Revise the code for premultiplyAlpha=false for
createImageBitmap(HTMLCanvasElement) (patchset #1 id:1 of
https://codereview.chromium.org/1694793002/ )

Reason for revert:
Cause compilation error
https://build.chromium.org/p/chromium.mac/builders/Mac%20Builder/builds/37356...

Original issue's description:
> Revise the code for premultiplyAlpha=false for
createImageBitmap(HTMLCanvasElement)
> 
> It turns out that HTMLCanvasElement is always premultiplied in Blink, so
> we cannot simply use a decoder to re-decode it to a un-premultiplied format.
> The strategy this CL takes is that after cropping the HTMLCanvasElement
> into a SkImage, we convert the SkImage into a un-premultiplied format if
> that's what the option specifies.

TBR=junov@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

[commit-bot: I haz the power, 2016-02-12 16:35:18]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-02-16 22:43:29]

Description was changed from

==========
Revert of Revise the code for premultiplyAlpha=false for
createImageBitmap(HTMLCanvasElement) (patchset #1 id:1 of
https://codereview.chromium.org/1694793002/ )

Reason for revert:
Cause compilation error
https://build.chromium.org/p/chromium.mac/builders/Mac%20Builder/builds/37356...

Original issue's description:
> Revise the code for premultiplyAlpha=false for
createImageBitmap(HTMLCanvasElement)
> 
> It turns out that HTMLCanvasElement is always premultiplied in Blink, so
> we cannot simply use a decoder to re-decode it to a un-premultiplied format.
> The strategy this CL takes is that after cropping the HTMLCanvasElement
> into a SkImage, we convert the SkImage into a un-premultiplied format if
> that's what the option specifies.

TBR=junov@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
==========

to

==========
Revert of Revise the code for premultiplyAlpha=false for
createImageBitmap(HTMLCanvasElement) (patchset #1 id:1 of
https://codereview.chromium.org/1694793002/ )

Reason for revert:
Cause compilation error
https://build.chromium.org/p/chromium.mac/builders/Mac%20Builder/builds/37356...

Original issue's description:
> Revise the code for premultiplyAlpha=false for
createImageBitmap(HTMLCanvasElement)
>
> It turns out that HTMLCanvasElement is always premultiplied in Blink, so
> we cannot simply use a decoder to re-decode it to a un-premultiplied format.
> The strategy this CL takes is that after cropping the HTMLCanvasElement
> into a SkImage, we convert the SkImage into a un-premultiplied format if
> that's what the option specifies.

TBR=junov@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true

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

[commit-bot: I haz the power, 2016-02-16 22:43:30]

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