simple cleanup in GrTargetCommands

simple cleanup in GrTargetCommands

BUG=skia:

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

[joshualitt, 2015-04-30 14:31:44]

joshualitt@google.com changed reviewers:
+ joshualitt@google.com, robertphillips@google.com

[joshualitt, 2015-04-30 14:31:45]

ptal

[robertphillips, 2015-04-30 15:51:08]

lgtm + questions

https://codereview.chromium.org/1116923003/diff/1/src/gpu/GrTargetCommands.cpp
File src/gpu/GrTargetCommands.cpp (right):

https://codereview.chromium.org/1116923003/diff/1/src/gpu/GrTargetCommands.cp...
src/gpu/GrTargetCommands.cpp:37: // Check if there is a Batch Draw we can batch
with
Why don't we need a "fCmdBuffer.count() && ..." in this if-test?

https://codereview.chromium.org/1116923003/diff/1/src/gpu/GrTargetCommands.cp...
src/gpu/GrTargetCommands.cpp:40: if
(!previous->fBatch->combineIfPossible(batch)) {
Why not return NULL here and just fall through otherwise?

[joshualitt, 2015-04-30 16:01:08]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2015-04-30 16:01:09]

The patchset sent to the CQ was uploaded after l-g-t-m from
robertphillips@google.com
Link to the patchset: https://codereview.chromium.org/1116923003/#ps20001
(title: "feedback inc")

[commit-bot: I haz the power, 2015-04-30 16:01:20]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1116923003/20001

[commit-bot: I haz the power, 2015-04-30 16:06:01]

Committed patchset #2 (id:20001) as
https://skia.googlesource.com/skia/+/a30009bec5d495b4524b746142a2afa4452908de