2011-02-28 Renata Hodovan <reni@webkit.org>
authorreni@webkit.org <reni@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 28 Feb 2011 16:16:39 +0000 (16:16 +0000)
committerreni@webkit.org <reni@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 28 Feb 2011 16:16:39 +0000 (16:16 +0000)
        Reviewed by Andreas Kling.

        Optimize parameter transmissions in FEConvolveMatrix.
        https://bugs.webkit.org/show_bug.cgi?id=55381

        Parameter transmission via reference is more efficient than with copy. So they are substituted.

        No new tests are needed since this is a refactoring.

        * platform/graphics/filters/FEConvolveMatrix.cpp:
        (WebCore::FEConvolveMatrix::setKernelSize):
        (WebCore::FEConvolveMatrix::setTargetOffset):
        (WebCore::FEConvolveMatrix::setKernelUnitLength):
        * platform/graphics/filters/FEConvolveMatrix.h:

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

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/filters/FEConvolveMatrix.cpp
Source/WebCore/platform/graphics/filters/FEConvolveMatrix.h

index a6c1191..d76c69c 100644 (file)
@@ -1,3 +1,20 @@
+2011-02-28  Renata Hodovan  <reni@webkit.org>
+
+        Reviewed by Andreas Kling.
+
+        Optimize parameter transmissions in FEConvolveMatrix.
+        https://bugs.webkit.org/show_bug.cgi?id=55381
+
+        Parameter transmission via reference is more efficient than with copy. So they are substituted.
+
+        No new tests are needed since this is a refactoring.
+
+        * platform/graphics/filters/FEConvolveMatrix.cpp:
+        (WebCore::FEConvolveMatrix::setKernelSize):
+        (WebCore::FEConvolveMatrix::setTargetOffset):
+        (WebCore::FEConvolveMatrix::setKernelUnitLength):
+        * platform/graphics/filters/FEConvolveMatrix.h:
+
 2011-02-28  Yury Semikhatsky  <yurys@chromium.org>
 
         Unreviewed. Fix Chromium tests failures due to r79858.
index f558a15..a1f8dfd 100644 (file)
@@ -63,7 +63,7 @@ IntSize FEConvolveMatrix::kernelSize() const
     return m_kernelSize;
 }
 
-void FEConvolveMatrix::setKernelSize(IntSize kernelSize)
+void FEConvolveMatrix::setKernelSize(const IntSize& kernelSize)
 {
     m_kernelSize = kernelSize; 
 }
@@ -109,7 +109,7 @@ IntPoint FEConvolveMatrix::targetOffset() const
     return m_targetOffset; 
 }
 
-bool FEConvolveMatrix::setTargetOffset(IntPoint targetOffset)
+bool FEConvolveMatrix::setTargetOffset(const IntPoint& targetOffset)
 {
     if (m_targetOffset == targetOffset)
         return false;
@@ -135,7 +135,7 @@ FloatPoint FEConvolveMatrix::kernelUnitLength() const
     return m_kernelUnitLength; 
 }
 
-bool FEConvolveMatrix::setKernelUnitLength(FloatPoint kernelUnitLength)
+bool FEConvolveMatrix::setKernelUnitLength(const FloatPoint& kernelUnitLength)
 {
     if (m_kernelUnitLength == kernelUnitLength)
         return false;
index e1d871c..5dc8873 100644 (file)
@@ -49,7 +49,7 @@ public:
             bool, const Vector<float>&);
 
     IntSize kernelSize() const;
-    void setKernelSize(IntSize);
+    void setKernelSize(const IntSize&);
 
     const Vector<float>& kernel() const;
     void setKernel(const Vector<float>&);
@@ -61,13 +61,13 @@ public:
     bool setBias(float);
 
     IntPoint targetOffset() const;
-    bool setTargetOffset(IntPoint);
+    bool setTargetOffset(const IntPoint&);
 
     EdgeModeType edgeMode() const;
     bool setEdgeMode(EdgeModeType);
 
     FloatPoint kernelUnitLength() const;
-    bool setKernelUnitLength(FloatPoint);
+    bool setKernelUnitLength(const FloatPoint&);
 
     bool preserveAlpha() const;
     bool setPreserveAlpha(bool);