Remove SkDevice::accessRenderTarget virtual

include/core/SkDevice.h

Index: include/core/SkDevice.h
diff --git a/include/core/SkDevice.h b/include/core/SkDevice.h
index 827897c20e45209d63246c8b153e71ab7094a1a3..76d464445c67de35c2f27e9c9150df312649d445 100644
--- a/include/core/SkDevice.h
+++ b/include/core/SkDevice.h
@@ -105,16 +105,6 @@ public:
     bool peekPixels(SkPixmap*);
 
     /**
-     * Return the device's associated gpu render target, or NULL.
-     */
-    virtual GrRenderTarget* accessRenderTarget() { return nullptr; }
-
-    /**
-     * Don't call this!
-     */
-    virtual GrDrawContext* accessDrawContext() { return nullptr; }
-
-    /**
      *  Return the device's origin: its offset in device coordinates from
      *  the default origin in its canvas' matrix/clip
      */
@@ -385,6 +375,11 @@ private:
 
     virtual bool forceConservativeRasterClip() const { return false; }
 
+    /**
+     * Don't call this!
+     */
+    virtual GrDrawContext* accessDrawContext() { return nullptr; }

[bsalomon, 2016/07/20 14:34:13]

Should we allow SkSurface_Gpu (and others) to downcast when they know it's a GPU
surface/canvas and make this non-virtual?

[robertphillips, 2016/07/20 14:49:13]

Yeah - that would be better. I'll do that in another CL.

+
     // just called by SkCanvas when built as a layer
     void setOrigin(int x, int y) { fOrigin.set(x, y); }