Initial CL to create Reorder command builder behind a flag

Initial CL to create Reorder command builder behind a flag

BUG=skia:

Committed: https://skia.googlesource.com/skia/+/3b58d75170508567436cce259384d336949407d4

[joshualitt, 2015-05-06 21:08:02]

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

[joshualitt, 2015-05-06 21:08:03]

This is obviously very preliminary, but I wanted to get something landed we can
all experiment with.

[bsalomon, 2015-05-07 17:29:27]

Amazing that this is so few lines of code.

lgtm

[joshualitt, 2015-05-07 17:30:53]

The CQ bit was checked by joshualitt@google.com

[commit-bot: I haz the power, 2015-05-07 17:31:09]

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

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

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

[commit-bot: I haz the power, 2015-05-07 17:31:16]

Failed to apply patch for src/gpu/GrReorderCommandBuilder.h:
While running git apply --index -3 -p1;
  error: patch failed: src/gpu/GrInOrderCommandBuilder.h:5
  Falling back to three-way merge...
  Applied patch to 'src/gpu/GrReorderCommandBuilder.h' with conflicts.
  U src/gpu/GrReorderCommandBuilder.h

Patch:   NR 
src/gpu/GrInOrderCommandBuilder.h->src/gpu/GrReorderCommandBuilder.h
Index: src/gpu/GrReorderCommandBuilder.h
diff --git a/src/gpu/GrInOrderCommandBuilder.h
b/src/gpu/GrReorderCommandBuilder.h
similarity index 74%
copy from src/gpu/GrInOrderCommandBuilder.h
copy to src/gpu/GrReorderCommandBuilder.h
index
4fc7cc74dd12abdb20d7d3d020bfb0b00cb30615..9ad0ad3951fb5657b1f295e651b7116f9cd81793
100644
--- a/src/gpu/GrInOrderCommandBuilder.h
+++ b/src/gpu/GrReorderCommandBuilder.h
@@ -5,17 +5,17 @@
  * found in the LICENSE file.
  */
 
-#ifndef GrInOrderCommandBuilder_DEFINED
-#define GrInOrderCommandBuilder_DEFINED
+#ifndef GrReorderCommandBuilder_DEFINED
+#define GrReorderCommandBuilder_DEFINED
 
 #include "GrCommandBuilder.h"
 
-class GrInOrderCommandBuilder : public GrCommandBuilder {
+class GrReorderCommandBuilder : public GrCommandBuilder {
 public:
     typedef GrCommandBuilder::Cmd Cmd;
     typedef GrCommandBuilder::State State;
 
-    GrInOrderCommandBuilder(GrGpu* gpu,
+    GrReorderCommandBuilder(GrGpu* gpu,
                             GrVertexBufferAllocPool* vertexPool,
                             GrIndexBufferAllocPool* indexPool)
         : INHERITED(gpu, vertexPool, indexPool) {
@@ -26,11 +26,19 @@ public:
                            const GrPathProcessor*,
                            const GrPath*,
                            const GrScissorState&,
-                           const GrStencilSettings&) override;
+                           const GrStencilSettings&) override {
+        SkFAIL("Unsupported\n");
+        return NULL;
+    }
+
     Cmd* recordDrawPath(State*,
                         const GrPathProcessor*,
                         const GrPath*,
-                        const GrStencilSettings&) override;
+                        const GrStencilSettings&) override {
+        SkFAIL("Unsupported\n");
+        return NULL;
+    }
+
     Cmd* recordDrawPaths(State*,
                          GrInOrderDrawBuffer*,
                          const GrPathProcessor*,
@@ -41,7 +49,10 @@ public:
                          GrDrawTarget::PathTransformType ,
                          int,
                          const GrStencilSettings&,
-                         const GrDrawTarget::PipelineInfo&) override;
+                         const GrDrawTarget::PipelineInfo&) override {
+        SkFAIL("Unsupported\n");
+        return NULL;
+    }
 
 private:
     typedef GrCommandBuilder INHERITED;

[joshualitt, 2015-05-07 17:52:06]

The CQ bit was checked by joshualitt@google.com

[joshualitt, 2015-05-07 17:52:07]

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/1129943004/#ps40001
(title: "more tidying")

[commit-bot: I haz the power, 2015-05-07 17:52:24]

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

[commit-bot: I haz the power, 2015-05-07 18:14:39]

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