Use HashSet instead of Dynamic hash for tracking vulkan resources

Use HashSet instead of Dynamic hash for tracking vulkan resources

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

Committed: https://skia.googlesource.com/skia/+/00db3fd7a790ecfbb4ce3c65572e1bee2a0078db

[egdaniel, 2016-07-29 13:51:12]

Description was changed from

==========
Use HashSet instead of Dynamic hash for tracking vulkan resources

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

to

==========
Use HashSet instead of Dynamic hash for tracking vulkan resources

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

[egdaniel, 2016-07-29 13:59:57]

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

[mtklein, 2016-07-29 14:33:20]

lgtm

https://codereview.chromium.org/2194823003/diff/40001/src/gpu/vk/GrVkResource.h
File src/gpu/vk/GrVkResource.h (right):

https://codereview.chromium.org/2194823003/diff/40001/src/gpu/vk/GrVkResource...
src/gpu/vk/GrVkResource.h:51: if (fHashSet.count()) {
No real need for this if?

[egdaniel, 2016-07-29 15:05:41]

https://codereview.chromium.org/2194823003/diff/40001/src/gpu/vk/GrVkResource.h
File src/gpu/vk/GrVkResource.h (right):

https://codereview.chromium.org/2194823003/diff/40001/src/gpu/vk/GrVkResource...
src/gpu/vk/GrVkResource.h:51: if (fHashSet.count()) {
On 2016/07/29 14:33:19, mtklein wrote:
> No real need for this if?

Ah right good catch. Gone.

[egdaniel, 2016-07-29 15:05:44]

The CQ bit was checked by egdaniel@google.com

[egdaniel, 2016-07-29 15:05:45]

The patchset sent to the CQ was uploaded after l-g-t-m from mtklein@google.com
Link to the patchset: https://codereview.chromium.org/2194823003/#ps60001
(title: "review cleanup")

[commit-bot: I haz the power, 2016-07-29 15:05:56]

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-07-29 15:55:55]

Description was changed from

==========
Use HashSet instead of Dynamic hash for tracking vulkan resources

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

to

==========
Use HashSet instead of Dynamic hash for tracking vulkan resources

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

Committed:
https://skia.googlesource.com/skia/+/00db3fd7a790ecfbb4ce3c65572e1bee2a0078db
==========

[commit-bot: I haz the power, 2016-07-29 15:55:56]

Committed patchset #4 (id:60001) as
https://skia.googlesource.com/skia/+/00db3fd7a790ecfbb4ce3c65572e1bee2a0078db