Make GrVertexBatch objects hold their own draws during GrDrawTarget flush

Make GrVertexBatch objects hold their own draws during GrDrawTarget flush

NO_MERGE_BUILDS

Committed: https://skia.googlesource.com/skia/+/7539856c1b9cbb1886a6a498cc534b77fc83ddb2

[bsalomon, 2015-08-17 18:28:02]

bsalomon@google.com changed reviewers:
+ joshualitt@google.com

[bsalomon, 2015-08-17 18:29:32]

The onPrepareDraws should be moved to private in the subclasses, but deferring
that to another change.

[joshualitt, 2015-08-17 18:52:42]

nit, lgtm

https://codereview.chromium.org/1286043004/diff/60001/src/gpu/batches/GrVerte...
File src/gpu/batches/GrVertexBatch.cpp (right):

https://codereview.chromium.org/1286043004/diff/60001/src/gpu/batches/GrVerte...
src/gpu/batches/GrVertexBatch.cpp:68: for (SkTLList<DrawArray>::Iter
da(fDrawArrays); da.get(); da.next()) {
const_iter :(

[bsalomon, 2015-08-17 18:59:39]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2015-08-17 18:59:39]

The patchset sent to the CQ was uploaded after l-g-t-m from
joshualitt@google.com
Link to the patchset: https://codereview.chromium.org/1286043004/#ps80001
(title: "rebase")

[commit-bot: I haz the power, 2015-08-17 18:59:56]

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

[bsalomon, 2015-08-17 19:02:18]

The CQ bit was unchecked by bsalomon@google.com

[bsalomon, 2015-08-17 19:02:27]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2015-08-17 19:02:27]

The patchset sent to the CQ was uploaded after l-g-t-m from
joshualitt@google.com
Link to the patchset: https://codereview.chromium.org/1286043004/#ps100001
(title: "Address warning")

[commit-bot: I haz the power, 2015-08-17 19:02:41]

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

[commit-bot: I haz the power, 2015-08-17 19:03:49]

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

[commit-bot: I haz the power, 2015-08-17 19:03:50]

Try jobs failed on following builders:
  Build-Mac10.8-Clang-x86_64-Release-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Mac10.8-Clang-...)
  Build-Ubuntu-GCC-Arm64-Debug-Android-Trybot on client.skia.compile
(JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-Arm...)
  Build-Ubuntu-GCC-x86_64-Release-Trybot on client.skia.compile (JOB_FAILED,
http://build.chromium.org/p/client.skia.compile/builders/Build-Ubuntu-GCC-x86...)

[bsalomon, 2015-08-17 19:48:25]

The CQ bit was checked by bsalomon@google.com

[bsalomon, 2015-08-17 19:48:26]

The patchset sent to the CQ was uploaded after l-g-t-m from
joshualitt@google.com
Link to the patchset: https://codereview.chromium.org/1286043004/#ps140001
(title: "forward decl")

[commit-bot: I haz the power, 2015-08-17 19:48:40]

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

[commit-bot: I haz the power, 2015-08-17 19:55:43]

Committed patchset #8 (id:140001) as
https://skia.googlesource.com/skia/+/7539856c1b9cbb1886a6a498cc534b77fc83ddb2