Allow vulkan to upload data to testing textures with optimal layouts

Allow vulkan to upload data to testing textures with optimal layouts

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

Committed: https://skia.googlesource.com/skia/+/3602d4f16a01da860d16eb36fb52eb62487495cc

[egdaniel, 2016-08-11 19:38:53]

Description was changed from

==========
Allow vulkan to upload data to testing textures with optimal layouts

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

to

==========
Allow vulkan to upload data to testing textures with optimal layouts

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

[egdaniel, 2016-08-11 19:40:09]

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

[egdaniel, 2016-08-11 19:40:09]

enjoy :)

https://codereview.chromium.org/2242553002/diff/1/gm/imagefromyuvtextures.cpp
File gm/imagefromyuvtextures.cpp (right):

https://codereview.chromium.org/2242553002/diff/1/gm/imagefromyuvtextures.cpp...
gm/imagefromyuvtextures.cpp:34: return SkISize::Make(50, 175);
we we're cutting off the last row the gm.

[jvanverth1, 2016-08-11 20:42:29]

Ugly, but functional. lgtm

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

https://codereview.chromium.org/2242553002/diff/1/src/gpu/vk/GrVkGpu.cpp#newc...
src/gpu/vk/GrVkGpu.cpp:1190: // Create Fence for queue
Ugh

[egdaniel, 2016-08-11 20:47:53]

The CQ bit was checked by egdaniel@google.com

[commit-bot: I haz the power, 2016-08-11 20:48:00]

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

[commit-bot: I haz the power, 2016-08-11 21:20:14]

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

[commit-bot: I haz the power, 2016-08-11 21:20:15]

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

[egdaniel, 2016-08-12 18:11:48]

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

[commit-bot: I haz the power, 2016-08-12 18:11:56]

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

[commit-bot: I haz the power, 2016-08-12 18:56:42]

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

[commit-bot: I haz the power, 2016-08-12 18:56:43]

Dry run: This issue passed the CQ dry run.

[egdaniel, 2016-08-12 18:57:48]

The CQ bit was checked by egdaniel@google.com

[egdaniel, 2016-08-12 18:57:49]

The patchset sent to the CQ was uploaded after l-g-t-m from jvanverth@google.com
Link to the patchset: https://codereview.chromium.org/2242553002/#ps20001
(title: "fix cast")

[commit-bot: I haz the power, 2016-08-12 18:57:53]

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

[commit-bot: I haz the power, 2016-08-12 18:58:55]

Description was changed from

==========
Allow vulkan to upload data to testing textures with optimal layouts

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

to

==========
Allow vulkan to upload data to testing textures with optimal layouts

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

Committed:
https://skia.googlesource.com/skia/+/3602d4f16a01da860d16eb36fb52eb62487495cc
==========

[commit-bot: I haz the power, 2016-08-12 18:58:56]

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

[robertphillips, 2016-08-15 13:02:21]

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

[robertphillips, 2016-08-15 13:02:22]

This seems to have broken the X1 and Windows10 Vulkan bots with:

FAILURE: ../../../tests/SurfaceTest.cpp:854	origColorPM == pixels[x + y * kW]