Add support for draws in vulkan read and write pixels

Add support for draws in vulkan read and write pixels

GL has a lot more optimizations checks here to make sure we do the most
effecient and correct draw here, but for now as long as the features
are support we just do basic draws for both reads and writes when we
need certain conversions.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2105433002

Committed: https://skia.googlesource.com/skia/+/4583ec51d9ddc830eeb854db068235be96ce59c4

[egdaniel, 2016-06-27 17:23:13]

Description was changed from

==========
Add support for draws in vulkan read and write pixels

GL has a lot more optimizations checks here to make sure we do the most
effecient and correct draw here, but for now as long as the features
are support we just do basic draws for both reads and writes when we
need certain conversions.

BUG=skia:
==========

to

==========
Add support for draws in vulkan read and write pixels

GL has a lot more optimizations checks here to make sure we do the most
effecient and correct draw here, but for now as long as the features
are support we just do basic draws for both reads and writes when we
need certain conversions.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2105433002
==========

[egdaniel, 2016-06-27 17:25:39]

egdaniel@google.com changed reviewers:
+ bsalomon@google.com, jvanverth@google.com

[egdaniel, 2016-06-27 17:25:40]

Next part in getting unit tests working

[bsalomon, 2016-06-27 17:59:26]

lgtm

[jvanverth1, 2016-06-27 18:37:41]

https://codereview.chromium.org/2105433002/diff/20001/src/gpu/vk/GrVkGpu.cpp
File src/gpu/vk/GrVkGpu.cpp (right):

https://codereview.chromium.org/2105433002/diff/20001/src/gpu/vk/GrVkGpu.cpp#...
src/gpu/vk/GrVkGpu.cpp:266: // to srcConfig, though that may be modified if we
decide to do a R/G swap
Should this be an R/B swap?

[egdaniel, 2016-06-27 19:37:53]

The CQ bit was checked by egdaniel@google.com

[egdaniel, 2016-06-27 19:37:54]

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

[commit-bot: I haz the power, 2016-06-27 19:38:04]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[egdaniel, 2016-06-27 19:50:56]

https://codereview.chromium.org/2105433002/diff/20001/src/gpu/vk/GrVkGpu.cpp
File src/gpu/vk/GrVkGpu.cpp (right):

https://codereview.chromium.org/2105433002/diff/20001/src/gpu/vk/GrVkGpu.cpp#...
src/gpu/vk/GrVkGpu.cpp:266: // to srcConfig, though that may be modified if we
decide to do a R/G swap
On 2016/06/27 18:37:41, jvanverth1 wrote:
> Should this be an R/B swap?

Done.

[commit-bot: I haz the power, 2016-06-27 19:57:02]

Description was changed from

==========
Add support for draws in vulkan read and write pixels

GL has a lot more optimizations checks here to make sure we do the most
effecient and correct draw here, but for now as long as the features
are support we just do basic draws for both reads and writes when we
need certain conversions.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2105433002
==========

to

==========
Add support for draws in vulkan read and write pixels

GL has a lot more optimizations checks here to make sure we do the most
effecient and correct draw here, but for now as long as the features
are support we just do basic draws for both reads and writes when we
need certain conversions.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search?issue=2105433002

Committed:
https://skia.googlesource.com/skia/+/4583ec51d9ddc830eeb854db068235be96ce59c4
==========

[commit-bot: I haz the power, 2016-06-27 19:57:03]

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