change factory to method, to parallel pattern in imagefilters. no functionality change.

BUG=skia:
GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1553743002

TBR=

Review URL: https://codereview.chromium.org/1553743002

1 files changed, 10 insertions, 9 deletions

diff --git a/include/core/SkShader.h b/include/core/SkShader.h
index 9b48697d20..97160a4f25 100644
--- a/include/core/SkShader.h
+++ b/include/core/SkShader.h
@@ -346,9 +346,18 @@ public:
 #endif
 
     //////////////////////////////////////////////////////////////////////////
-    //  Factory methods for stock shaders
+    //  Methods to create combinations or variants of shaders
 
     /**
+     *  Return a shader that will apply the specified localMatrix to this shader.
+     *  The specified matrix will be applied before any matrix associated with this shader.
+     */
+    SkShader* newWithLocalMatrix(const SkMatrix&) const;
+    
+    //////////////////////////////////////////////////////////////////////////
+    //  Factory methods for stock shaders
+    
+    /**
      *  Call this to create a new "empty" shader, that will not draw anything.
      */
     static SkShader* CreateEmptyShader();
@@ -399,14 +408,6 @@ public:
                                          const SkRect* tile);
 
     /**
-     *  Return a shader that will apply the specified localMatrix to the proxy shader.
-     *  The specified matrix will be applied before any matrix associated with the proxy.
-     *
-     *  Note: ownership of the proxy is not transferred (though a ref is taken).
-     */
-    static SkShader* CreateLocalMatrixShader(SkShader* proxy, const SkMatrix& localMatrix);
-
-    /**
      *  If this shader can be represented by another shader + a localMatrix, return that shader
      *  and, if not NULL, the localMatrix. If not, return NULL and ignore the localMatrix parameter.
      *