Add GrGpu support for Vulkan command buffer flush

Add GrGpu support for Vulkan command buffer flush
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1705133002

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

[jvanverth1, 2016-02-17 16:46:17]

Description was changed from

==========
Add GrGpu support for Vulkan command buffer flush
==========

to

==========
Add GrGpu support for Vulkan command buffer flush
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

[jvanverth1, 2016-02-17 16:51:11]

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

[jvanverth1, 2016-02-17 16:51:11]

Part One of building a complete command buffer per flush

[bsalomon, 2016-02-17 17:39:46]

On 2016/02/17 16:51:11, jvanverth1 wrote:
> Part One of building a complete command buffer per flush

Can we roll this and the performFlushWorkaround into one thing?

[egdaniel, 2016-02-17 17:41:55]

lgtm

[jvanverth1, 2016-02-17 18:00:31]

On 2016/02/17 17:39:46, bsalomon wrote:
> On 2016/02/17 16:51:11, jvanverth1 wrote:
> > Part One of building a complete command buffer per flush
> 
> Can we roll this and the performFlushWorkaround into one thing?

What is the performFlushWorkaround? Is that what I'm calling postFlush() here?

[bsalomon, 2016-02-17 18:08:49]

On 2016/02/17 18:00:31, jvanverth1 wrote:
> On 2016/02/17 17:39:46, bsalomon wrote:
> > On 2016/02/17 16:51:11, jvanverth1 wrote:
> > > Part One of building a complete command buffer per flush
> > 
> > Can we roll this and the performFlushWorkaround into one thing?
> 
> What is the performFlushWorkaround? Is that what I'm calling postFlush() here?

Ethan added it to perform a PLS driver bug workaround after each flush. Seems
like it could be done in your postFlush rather than having two virtuals.

[jvanverth1, 2016-02-17 20:39:50]

On 2016/02/17 18:08:49, bsalomon wrote:
> On 2016/02/17 18:00:31, jvanverth1 wrote:
> > On 2016/02/17 17:39:46, bsalomon wrote:
> > > On 2016/02/17 16:51:11, jvanverth1 wrote:
> > > > Part One of building a complete command buffer per flush
> > > 
> > > Can we roll this and the performFlushWorkaround into one thing?
> > 
> > What is the performFlushWorkaround? Is that what I'm calling postFlush()
here?
> 
> Ethan added it to perform a PLS driver bug workaround after each flush. Seems
> like it could be done in your postFlush rather than having two virtuals.

Done.

[bsalomon, 2016-02-17 21:24:29]

lgtm

[jvanverth1, 2016-02-17 21:25:18]

The CQ bit was checked by jvanverth@google.com

[jvanverth1, 2016-02-17 21:25:18]

The patchset sent to the CQ was uploaded after l-g-t-m from egdaniel@google.com
Link to the patchset: https://codereview.chromium.org/1705133002/#ps60001
(title: "Merge postFlush and performFlushWorkaround")

[commit-bot: I haz the power, 2016-02-17 21:25:28]

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

[commit-bot: I haz the power, 2016-02-17 22:04:49]

Description was changed from

==========
Add GrGpu support for Vulkan command buffer flush
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...
==========

to

==========
Add GrGpu support for Vulkan command buffer flush
GOLD_TRYBOT_URL=
https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&is...

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

[commit-bot: I haz the power, 2016-02-17 22:04:49]

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