Another trivial cleanup

include/gpu/GrStagedProcessor.h

+/*
+ * Copyright 2015 Google Inc.
+ *
+ * Use of this source code is governed by a BSD-style license that can be
+ * found in the LICENSE file.
+ */
+
+#ifndef GrStagedProcessorStage_DEFINED
+#define GrStagedProcessorStage_DEFINED
+
+#include "GrFragmentProcessor.h"
+#include "SkRefCnt.h"
+
+/**

[robertphillips, 2015/07/13 19:12:29]

Update comment ?
// templatized based on which container is desired for the GrFragmentProcessor
?

+ * Wraps a GrFragmentProcessor, basically a copyable SkAutoTUnref
+ */
+template<template<typename> class T>
+class GrStagedProcessor {
+public:
+    explicit GrStagedProcessor(const GrFragmentProcessor* proc) : fProc(SkRef(proc)) {}
+
+    GrStagedProcessor(const GrStagedProcessor& other) { fProc.reset(SkRef(other.fProc.get())); }
+
+    const GrFragmentProcessor* processor() const { return fProc.get(); }
+
+    bool operator==(const GrStagedProcessor& that) const {
+        return this->processor() == that.processor();
+    }
+
+    bool operator!=(const GrStagedProcessor& that) const { return !(*this == that); }
+
+    const char* name() const { return fProc->name(); }
+
+protected:
+    T<const GrFragmentProcessor> fProc;
+};
+

[robertphillips, 2015/07/13 19:12:29]

Can we not do:
typedef GrStagedProcessor<GrPendingProgramElement> GrPendingFragmentStage;
and get rid of GrPendingFragmentStage.h

+typedef GrStagedProcessor<SkAutoTUnref> GrFragmentStage;
+
+#endif