Fix bufferIndex check in VulkanWindowContext

Fix bufferIndex check in VulkanWindowContext

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

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

[egdaniel, 2016-09-26 18:53:19]

Description was changed from

==========
Fix bufferIndex check in VulkanWindowContext

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

to

==========
Fix bufferIndex check in VulkanWindowContext

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

[egdaniel, 2016-09-26 18:55:29]

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

[egdaniel, 2016-09-26 18:55:30]

This does not fix the VK_TIMEOUT I was seeing which is a bug introduced in the
newest Adreno drivers it seems like.

[egdaniel, 2016-09-26 18:55:55]

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

[commit-bot: I haz the power, 2016-09-26 18:56:03]

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-09-26 19:13:31]

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

[commit-bot: I haz the power, 2016-09-26 19:13:32]

Dry run: This issue passed the CQ dry run.

[jvanverth1, 2016-09-26 19:18:17]

lgtm

How did it ever work?

[egdaniel, 2016-09-26 19:19:31]

On 2016/09/26 19:18:17, jvanverth1 wrote:
> lgtm
> 
> How did it ever work?

beats me

[egdaniel, 2016-09-26 19:19:34]

The CQ bit was checked by egdaniel@google.com

[commit-bot: I haz the power, 2016-09-26 19:19:41]

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

[jvanverth1, 2016-09-26 19:19:59]

On 2016/09/26 19:18:17, jvanverth1 wrote:
> lgtm
> 
> How did it ever work?

not lgtm

[jvanverth1, 2016-09-26 19:20:34]

On 2016/09/26 19:19:59, jvanverth1 wrote:
> On 2016/09/26 19:18:17, jvanverth1 wrote:
> > lgtm
> > 
> > How did it ever work?
> 
> not lgtm

There's one more backbuffer than images. See line 316.

[jvanverth1, 2016-09-26 19:20:54]

The CQ bit was unchecked by jvanverth@google.com

[commit-bot: I haz the power, 2016-09-26 19:20:58]

Description was changed from

==========
Fix bufferIndex check in VulkanWindowContext

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

to

==========
Fix bufferIndex check in VulkanWindowContext

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

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

[commit-bot: I haz the power, 2016-09-26 19:21:00]

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

[egdaniel, 2016-09-26 19:30:38]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2370953002/ by egdaniel@google.com.

The reason for reverting is: Fix not correct..

[jvanverth1, 2016-09-26 19:31:27]

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/2370963002/ by jvanverth@google.com.

The reason for reverting is: The bufferIndex check was correct, this is wrong..