Fix skiaserve crash when displaying batch bounds

Fix skiaserve crash when displaying batch bounds

In my test case, the first actual batch would combine with the clear
issued before auditing had been enabled. After the successful combine,
we would try to inform the audit manager, but it didn't know about the
batch, triggering an assert (and subsequent crash). Fix is ijust to
ensure that no prior batches are pending when we turn on auditing.

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

Committed: https://skia.googlesource.com/skia/+/1c9f9228e5dae355cbcdf6c4e6a10dc92ad6fe47

[Brian Osman, 2016-04-15 17:36:28]

Description was changed from

==========
Fix skiaserve crash when displaying batch bounds

In my test case, the first actual batch would combine with the clear
issued before auditing had been enabled. After the successful combine,
we would try to inform the audit manager, but it didn't know about the
batch, triggering an assert (and subsequent crash). Fix is ijust to
ensure that no prior batches are pending when we turn on auditing.

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

to

==========
Fix skiaserve crash when displaying batch bounds

In my test case, the first actual batch would combine with the clear
issued before auditing had been enabled. After the successful combine,
we would try to inform the audit manager, but it didn't know about the
batch, triggering an assert (and subsequent crash). Fix is ijust to
ensure that no prior batches are pending when we turn on auditing.

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

[Brian Osman, 2016-04-15 17:37:32]

brianosman@google.com changed reviewers:
+ bsalomon@google.com, ethannicholas@google.com, jcgregorio@google.com

[Brian Osman, 2016-04-15 17:41:43]

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

[commit-bot: I haz the power, 2016-04-15 17:41:46]

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

[jcgregorio, 2016-04-15 17:49:28]

lgtm

[Brian Osman, 2016-04-15 17:52:12]

The CQ bit was unchecked by brianosman@google.com

[Brian Osman, 2016-04-15 17:52:14]

The CQ bit was checked by brianosman@google.com

[commit-bot: I haz the power, 2016-04-15 17:52:26]

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

[commit-bot: I haz the power, 2016-04-15 18:00:53]

Description was changed from

==========
Fix skiaserve crash when displaying batch bounds

In my test case, the first actual batch would combine with the clear
issued before auditing had been enabled. After the successful combine,
we would try to inform the audit manager, but it didn't know about the
batch, triggering an assert (and subsequent crash). Fix is ijust to
ensure that no prior batches are pending when we turn on auditing.

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

to

==========
Fix skiaserve crash when displaying batch bounds

In my test case, the first actual batch would combine with the clear
issued before auditing had been enabled. After the successful combine,
we would try to inform the audit manager, but it didn't know about the
batch, triggering an assert (and subsequent crash). Fix is ijust to
ensure that no prior batches are pending when we turn on auditing.

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/+/1c9f9228e5dae355cbcdf6c4e6a10dc92ad6fe47
==========

[commit-bot: I haz the power, 2016-04-15 18:00:54]

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