Add SkImage::NewTextureFromPixmap

Add SkImage::NewTextureFromPixmap
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1776913003

Committed: https://skia.googlesource.com/skia/+/045afea97902936869d71389e656a9c6917fa32f

[bsalomon, 2016-03-09 16:34:54]

Description was changed from

==========
Add SkImage::NewTextureFromPixmap
==========

to

==========
Add SkImage::NewTextureFromPixmap
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[bsalomon, 2016-03-09 16:41:43]

bsalomon@google.com changed reviewers:
+ reed@google.com; robertphillip

[bsalomon, 2016-03-09 16:41:44]

The color table variation doesn't work because SkImage::peekPixels(SkPixmap*)
doesn't populate the color table of the pixmap. Mike says he will fix this.

[robertphillips, 2016-03-09 16:51:28]

robertphillips@google.com changed reviewers:
+ robertphillips@google.com

[robertphillips, 2016-03-09 16:51:28]

lgtm

https://codereview.chromium.org/1776913003/diff/1/tests/ImageTest.cpp
File tests/ImageTest.cpp (right):

https://codereview.chromium.org/1776913003/diff/1/tests/ImageTest.cpp#newcode75
tests/ImageTest.cpp:75: }
// Add a comment about why this is broken?

[bsalomon, 2016-03-09 16:55:58]

https://codereview.chromium.org/1776913003/diff/1/tests/ImageTest.cpp
File tests/ImageTest.cpp (right):

https://codereview.chromium.org/1776913003/diff/1/tests/ImageTest.cpp#newcode75
tests/ImageTest.cpp:75: }
On 2016/03/09 16:51:28, robertphillips wrote:
> // Add a comment about why this is broken?

Added a comment to the test below. This function actually works, it's the peek
in the test that has the bug.

[bsalomon, 2016-03-09 18:35:47]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-03-09 18:35:47]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/1776913003/#ps40001
(title: "add comment")

[commit-bot: I haz the power, 2016-03-09 18:35:53]

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

[commit-bot: I haz the power, 2016-03-09 18:40:03]

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

[commit-bot: I haz the power, 2016-03-09 18:40:04]

Try jobs failed on following builders:
  Build-Win-MSVC-x86-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86-D...)
  Build-Win-MSVC-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86_6...)

[bsalomon, 2016-03-09 19:04:03]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-03-09 19:04:03]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/1776913003/#ps60001
(title: "include initializer_list")

[commit-bot: I haz the power, 2016-03-09 19:04:09]

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

[commit-bot: I haz the power, 2016-03-09 19:08:20]

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

[commit-bot: I haz the power, 2016-03-09 19:08:21]

Try jobs failed on following builders:
  Build-Win-MSVC-x86_64-Debug-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Win-MSVC-x86_6...)

[bsalomon, 2016-03-09 19:20:35]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-03-09 19:20:35]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/1776913003/#ps80001
(title: "try again")

[commit-bot: I haz the power, 2016-03-09 19:20:40]

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

[commit-bot: I haz the power, 2016-03-09 19:31:20]

Description was changed from

==========
Add SkImage::NewTextureFromPixmap
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Add SkImage::NewTextureFromPixmap
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

Committed:
https://skia.googlesource.com/skia/+/045afea97902936869d71389e656a9c6917fa32f
==========

[commit-bot: I haz the power, 2016-03-09 19:31:21]

Committed patchset #5 (id:80001) as
https://skia.googlesource.com/skia/+/045afea97902936869d71389e656a9c6917fa32f