Switch uniforms to be descriptor set 0 in Vulkan

Switch uniforms to be descriptor set 0 in Vulkan

This (plus some cleanup) gives us some minor perf gains and as a side effect
the perf debug layer no longer complains when we bind certain descriptor sets.

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

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

[egdaniel, 2016-04-06 20:20:37]

Description was changed from

==========
Switch uniforms to be descriptor set 0 in Vulkan

This (plus some cleanup) gives us some minor perf gains and as a side effect
the perf debug layer no longer complains when we bind certain descriptor sets.

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

to

==========
Switch uniforms to be descriptor set 0 in Vulkan

This (plus some cleanup) gives us some minor perf gains and as a side effect
the perf debug layer no longer complains when we bind certain descriptor sets.

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

[egdaniel, 2016-04-06 20:22:48]

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

[egdaniel, 2016-04-06 20:22:48]

https://codereview.chromium.org/1864303002/diff/1/src/gpu/vk/GrVkPipelineStat...
File src/gpu/vk/GrVkPipelineStateBuilder.cpp (right):

https://codereview.chromium.org/1864303002/diff/1/src/gpu/vk/GrVkPipelineStat...
src/gpu/vk/GrVkPipelineStateBuilder.cpp:206: dsUniBindings[0].descriptorCount =
1;
This is needed so that the layout for all our uniforms stays the same (and we
don't spend time invaliding unnecessary descriptor sets. We don't actually end
up making the extra uniform buffers, just saying the layout allows them.

[jvanverth1, 2016-04-06 20:32:02]

lgtm

[egdaniel, 2016-04-06 20:34:08]

The CQ bit was checked by egdaniel@google.com

[commit-bot: I haz the power, 2016-04-06 20:34:14]

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

[commit-bot: I haz the power, 2016-04-06 20:47:10]

Description was changed from

==========
Switch uniforms to be descriptor set 0 in Vulkan

This (plus some cleanup) gives us some minor perf gains and as a side effect
the perf debug layer no longer complains when we bind certain descriptor sets.

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

to

==========
Switch uniforms to be descriptor set 0 in Vulkan

This (plus some cleanup) gives us some minor perf gains and as a side effect
the perf debug layer no longer complains when we bind certain descriptor sets.

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

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

[commit-bot: I haz the power, 2016-04-06 20:47:11]

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