Add ability to query read pixels support without a render target.

Add ability to query read pixels support without a render target.

Add more checks to onGetReadPixelsInfo.
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1615023003

Committed: https://skia.googlesource.com/skia/+/1aa202935ff698f1f35c5435455073fd9f1d08de

[bsalomon, 2016-01-21 21:07:11]

Description was changed from

==========
Add ability to query read pixels support without a render target.

Add more checks to onGetReadPixelsInfo.
==========

to

==========
Add ability to query read pixels support without a render target.

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

[bsalomon, 2016-01-21 21:10:51]

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

[bsalomon, 2016-01-21 21:10:52]

Adds more checks to onGetReadPixelsInfo that it's temp draw manipulations will
actually succeed.

[robertphillips, 2016-01-22 14:56:11]

lgtm

https://codereview.chromium.org/1615023003/diff/20001/src/gpu/gl/GrGLGpu.h
File src/gpu/gl/GrGLGpu.h (right):

https://codereview.chromium.org/1615023003/diff/20001/src/gpu/gl/GrGLGpu.h#ne...
src/gpu/gl/GrGLGpu.h:166: 
pixels -> pixel ?
Here and in other two comments too

[bsalomon, 2016-01-22 15:50:19]

Squashed 1615143002 into this

https://codereview.chromium.org/1615023003/diff/20001/src/gpu/gl/GrGLGpu.h
File src/gpu/gl/GrGLGpu.h (right):

https://codereview.chromium.org/1615023003/diff/20001/src/gpu/gl/GrGLGpu.h#ne...
src/gpu/gl/GrGLGpu.h:166: 
On 2016/01/22 14:56:10, robertphillips wrote:
> pixels -> pixel ?
> Here and in other two comments too

Done.

[bsalomon, 2016-01-22 15:52:46]

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

[commit-bot: I haz the power, 2016-01-22 15:52:59]

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

[commit-bot: I haz the power, 2016-01-22 16:09:16]

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

[commit-bot: I haz the power, 2016-01-22 16:09:16]

Dry run: This issue passed the CQ dry run.

[bsalomon, 2016-01-22 16:15:17]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2016-01-22 16:15:17]

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/1615023003/#ps40001
(title: "squash with prev cl in series")

[commit-bot: I haz the power, 2016-01-22 16:15:25]

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

[commit-bot: I haz the power, 2016-01-22 16:16:11]

Description was changed from

==========
Add ability to query read pixels support without a render target.

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

to

==========
Add ability to query read pixels support without a render target.

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

Committed:
https://skia.googlesource.com/skia/+/1aa202935ff698f1f35c5435455073fd9f1d08de
==========

[commit-bot: I haz the power, 2016-01-22 16:16:12]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/1aa202935ff698f1f35c5435455073fd9f1d08de