Make Gradient::gradientSpaceTransform(), Pattern::patternSpaceTransform() methods...
authorzandobersek@gmail.com <zandobersek@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 Jan 2019 10:41:18 +0000 (10:41 +0000)
committerzandobersek@gmail.com <zandobersek@gmail.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 7 Jan 2019 10:41:18 +0000 (10:41 +0000)
https://bugs.webkit.org/show_bug.cgi?id=193189

Reviewed by Philippe Normand.

The transform getter methods on the Gradient and Pattern classes both
return const references to the transform objects, and don't modify any
internal state. They should be marked const accordingly, allowing
invocations of these two methods through const references to Gradient
and Pattern objects.

* platform/graphics/Gradient.h:
(WebCore::Gradient::gradientSpaceTransform): Now const.
* platform/graphics/Pattern.h:
(WebCore::Pattern::patternSpaceTransform): Now const.

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@239669 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/Gradient.h
Source/WebCore/platform/graphics/Pattern.h

index a2c7461..6d70e99 100644 (file)
@@ -1,5 +1,23 @@
 2019-01-07  Zan Dobersek  <zdobersek@igalia.com>
 
+        Make Gradient::gradientSpaceTransform(), Pattern::patternSpaceTransform() methods const
+        https://bugs.webkit.org/show_bug.cgi?id=193189
+
+        Reviewed by Philippe Normand.
+
+        The transform getter methods on the Gradient and Pattern classes both
+        return const references to the transform objects, and don't modify any
+        internal state. They should be marked const accordingly, allowing
+        invocations of these two methods through const references to Gradient
+        and Pattern objects.
+
+        * platform/graphics/Gradient.h:
+        (WebCore::Gradient::gradientSpaceTransform): Now const.
+        * platform/graphics/Pattern.h:
+        (WebCore::Pattern::patternSpaceTransform): Now const.
+
+2019-01-07  Zan Dobersek  <zdobersek@igalia.com>
+
         REGRESSION(r239636): ImageDecoder::setEncodedDataStatusChangeCallback() can be called on a null decoder
         https://bugs.webkit.org/show_bug.cgi?id=193187
 
index 4c9577e..5a869b9 100644 (file)
@@ -119,7 +119,7 @@ public:
 
     // CG needs to transform the gradient at draw time.
     void setGradientSpaceTransform(const AffineTransform& gradientSpaceTransformation);
-    const AffineTransform& gradientSpaceTransform() { return m_gradientSpaceTransformation; }
+    const AffineTransform& gradientSpaceTransform() const { return m_gradientSpaceTransformation; }
 
     void fill(GraphicsContext&, const FloatRect&);
     void adjustParametersForTiledDrawing(FloatSize&, FloatRect&, const FloatSize& spacing);
index f467a0f..a267ea8 100644 (file)
@@ -65,7 +65,7 @@ public:
     PlatformPatternPtr createPlatformPattern(const GraphicsContext&, float alpha, const AffineTransform& userSpaceTransformation) const;
 #endif
     void setPatternSpaceTransform(const AffineTransform& patternSpaceTransformation);
-    const AffineTransform& patternSpaceTransform() { return m_patternSpaceTransformation; };
+    const AffineTransform& patternSpaceTransform() const { return m_patternSpaceTransformation; };
     bool repeatX() const { return m_repeatX; }
     bool repeatY() const { return m_repeatY; }