Switch GrTextureToYUVPlanes over to newDrawContext calls

Switch GrTextureToYUVPlanes over to newDrawContext calls

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

Committed: https://skia.googlesource.com/skia/+/ebf30e84156358d6841305ecea21c7c979370bad

[robertphillips, 2016-05-11 13:21:14]

Description was changed from

==========
Switch GrTextureToYUVPlanes over to newDrawContext calls
==========

to

==========
Switch GrTextureToYUVPlanes over to newDrawContext calls
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[robertphillips, 2016-05-11 13:21:36]

Description was changed from

==========
Switch GrTextureToYUVPlanes over to newDrawContext calls
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Switch GrTextureToYUVPlanes over to newDrawContext calls

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

[robertphillips, 2016-05-11 13:21:36]

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

[robertphillips, 2016-05-11 13:23:39]

The CQ bit was checked by robertphillips@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-05-11 13:23:44]

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

[commit-bot: I haz the power, 2016-05-11 13:47:39]

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

[commit-bot: I haz the power, 2016-05-11 13:47:40]

Dry run: This issue passed the CQ dry run.

[bsalomon, 2016-05-11 14:29:52]

https://codereview.chromium.org/1970743002/diff/1/src/gpu/GrTextureToYUVPlane...
File src/gpu/GrTextureToYUVPlanes.cpp (right):

https://codereview.chromium.org/1970743002/diff/1/src/gpu/GrTextureToYUVPlane...
src/gpu/GrTextureToYUVPlanes.cpp:144: if (!yuvTex) {
could this be an assert? Can we say that newDrawContext always returns a
GrDrawContext for which asTexture will succeed?

[robertphillips, 2016-05-11 15:06:03]

https://codereview.chromium.org/1970743002/diff/1/src/gpu/GrTextureToYUVPlane...
File src/gpu/GrTextureToYUVPlanes.cpp (right):

https://codereview.chromium.org/1970743002/diff/1/src/gpu/GrTextureToYUVPlane...
src/gpu/GrTextureToYUVPlanes.cpp:144: if (!yuvTex) {
On 2016/05/11 14:29:52, bsalomon wrote:
> could this be an assert? Can we say that newDrawContext always returns a
> GrDrawContext for which asTexture will succeed?

Done.

[robertphillips, 2016-05-11 15:06:13]

The CQ bit was checked by robertphillips@google.com to run a CQ dry run

[commit-bot: I haz the power, 2016-05-11 15:06:21]

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

[bsalomon, 2016-05-11 15:17:16]

lgtm

[commit-bot: I haz the power, 2016-05-11 15:32:25]

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

[commit-bot: I haz the power, 2016-05-11 15:32:26]

Dry run: This issue passed the CQ dry run.

[robertphillips, 2016-05-11 15:33:36]

The CQ bit was checked by robertphillips@google.com

[commit-bot: I haz the power, 2016-05-11 15:33:45]

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

[commit-bot: I haz the power, 2016-05-11 15:34:41]

Description was changed from

==========
Switch GrTextureToYUVPlanes over to newDrawContext calls

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

to

==========
Switch GrTextureToYUVPlanes over to newDrawContext calls

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

Committed:
https://skia.googlesource.com/skia/+/ebf30e84156358d6841305ecea21c7c979370bad
==========

[commit-bot: I haz the power, 2016-05-11 15:34:42]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/ebf30e84156358d6841305ecea21c7c979370bad