Wait for all device work to finish before destroy VkDevice

Wait for all device work to finish before destroy VkDevice

Currently before destroying the GrVkGpu, we do wait on all queue work to finish before destroying objects there.
Since we only have one queue, the vkDeviceWaitIdle added here is probably not necessary, but the spec does suggest
having it just to make sure all work is fully done on the device.

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

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

[egdaniel, 2016-04-11 20:21:07]

Description was changed from

==========
Wait for all device work to finish before destroy VkDevice

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

to

==========
Wait for all device work to finish before destroy VkDevice

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

[egdaniel, 2016-04-11 20:24:18]

Description was changed from

==========
Wait for all device work to finish before destroy VkDevice

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

to

==========
Wait for all device work to finish before destroy VkDevice

Currently before destroying the GrVkGpu, we do wait on all queue work to finish
before destroying objects there. Since we only have one queue, the
vkDeviceWaitIdle added here is probably not necessary, but the spec does suggest
having it just to make sure all work is fully done on the device.

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

[egdaniel, 2016-04-11 20:24:39]

Description was changed from

==========
Wait for all device work to finish before destroy VkDevice

Currently before destroying the GrVkGpu, we do wait on all queue work to finish
before destroying objects there. Since we only have one queue, the
vkDeviceWaitIdle added here is probably not necessary, but the spec does suggest
having it just to make sure all work is fully done on the device.

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

to

==========
Wait for all device work to finish before destroy VkDevice

Currently before destroying the GrVkGpu, we do wait on all queue work to finish
before destroying objects there.
Since we only have one queue, the vkDeviceWaitIdle added here is probably not
necessary, but the spec does suggest
having it just to make sure all work is fully done on the device.

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

[egdaniel, 2016-04-11 20:26:11]

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

[egdaniel, 2016-04-11 20:26:12]

Sadly this change does not seem to help the bug we've run into where we hang
trying to destroy the vkInstance

[jvanverth1, 2016-04-11 20:28:15]

lgtm

[egdaniel, 2016-04-11 20:30:34]

The CQ bit was checked by egdaniel@google.com

[commit-bot: I haz the power, 2016-04-11 20:30:39]

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

[commit-bot: I haz the power, 2016-04-11 20:41:58]

Description was changed from

==========
Wait for all device work to finish before destroy VkDevice

Currently before destroying the GrVkGpu, we do wait on all queue work to finish
before destroying objects there.
Since we only have one queue, the vkDeviceWaitIdle added here is probably not
necessary, but the spec does suggest
having it just to make sure all work is fully done on the device.

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

to

==========
Wait for all device work to finish before destroy VkDevice

Currently before destroying the GrVkGpu, we do wait on all queue work to finish
before destroying objects there.
Since we only have one queue, the vkDeviceWaitIdle added here is probably not
necessary, but the spec does suggest
having it just to make sure all work is fully done on the device.

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/+/f4ace41dd9590944c9d4f9e3d1c576914db7af18
==========

[commit-bot: I haz the power, 2016-04-11 20:41:59]

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