Fall back to older inorder buffer if we have NVPR

Fall back to older inorder buffer if we have NVPR

BUG=skia:

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

Committed: https://skia.googlesource.com/skia/+/24a366a03756d7a8755f940d3160698cf9122cc0

[joshualitt, 2015-05-27 17:07:44]

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

[bsalomon, 2015-05-27 17:16:06]

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

https://codereview.chromium.org/1143153008/diff/1/src/gpu/GrInOrderDrawBuffer...
src/gpu/GrInOrderDrawBuffer.cpp:14: , fCommands(GrCommandBuilder::Create(
seems like a lot of logic in the init list.

static inline bool allow_reodering(const GrCaps& caps) {
    return caps->pathRenderingSupport();
}

GrInOrderDrawBuffer::GrInOrderDrawBuffer(GrContext* context)
    : INHERITED(context)
    ,
fCommands(GrCommandBuilder::Create(allow_reordering(context->getGpu()->caps()))

?

lgtm

[joshualitt, 2015-05-27 17:24:24]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2015-05-27 17:24:25]

The patchset sent to the CQ was uploaded after l-g-t-m from bsalomon@google.com
Link to the patchset: https://codereview.chromium.org/1143153008/#ps40001
(title: "tweaks")

[commit-bot: I haz the power, 2015-05-27 17:24:35]

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

[bsalomon, 2015-05-27 17:26:10]

https://codereview.chromium.org/1143153008/diff/40001/src/gpu/GrInOrderDrawBu...
File src/gpu/GrInOrderDrawBuffer.cpp (right):

https://codereview.chromium.org/1143153008/diff/40001/src/gpu/GrInOrderDrawBu...
src/gpu/GrInOrderDrawBuffer.cpp:18: ,
fCommands(GrCommandBuilder::Create(context->getGpu(),
We can't get here without a GPU object.

[joshualitt, 2015-05-27 17:28:48]

The CQ bit was unchecked by joshualitt@google.com

[joshualitt, 2015-05-27 17:30:23]

The CQ bit was checked by joshualitt@google.com

[commit-bot: I haz the power, 2015-05-27 17:30:39]

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

[commit-bot: I haz the power, 2015-05-27 17:31:15]

Committed patchset #3 (id:40001) as
https://skia.googlesource.com/skia/+/b72ec98b2090e5728354dac66eb1886a28e4da6b

[joshualitt, 2015-05-27 17:39:33]

A revert of this CL (patchset #3 id:40001) has been created in
https://codereview.chromium.org/1160663006/ by joshualitt@google.com.

The reason for reverting is: breaking bots.

[joshualitt, 2015-05-27 17:45:24]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2015-05-27 17:45:25]

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

[commit-bot: I haz the power, 2015-05-27 17:45:43]

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

[commit-bot: I haz the power, 2015-05-27 17:56:47]

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