Remove the clone() method from FilterOperation (and subclasses).
authorsenorblanco@chromium.org <senorblanco@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 31 Jul 2012 16:10:39 +0000 (16:10 +0000)
committersenorblanco@chromium.org <senorblanco@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 31 Jul 2012 16:10:39 +0000 (16:10 +0000)
https://bugs.webkit.org/show_bug.cgi?id=92757

Reviewed by Kentaro Hara.

Since the move to WebFilterOperation in Chromium, this is now dead
code.

Covered by existing tests in css3/filters.

* platform/graphics/filters/CustomFilterOperation.h:
* platform/graphics/filters/FilterOperation.h:

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

Source/WebCore/ChangeLog
Source/WebCore/platform/graphics/filters/CustomFilterOperation.h
Source/WebCore/platform/graphics/filters/FilterOperation.h

index 1333d5e..a217f79 100644 (file)
@@ -1,3 +1,18 @@
+2012-07-31  Stephen White  <senorblanco@chromium.org>
+
+        Remove the clone() method from FilterOperation (and subclasses).
+        https://bugs.webkit.org/show_bug.cgi?id=92757
+
+        Reviewed by Kentaro Hara.
+
+        Since the move to WebFilterOperation in Chromium, this is now dead
+        code.
+
+        Covered by existing tests in css3/filters.
+
+        * platform/graphics/filters/CustomFilterOperation.h:
+        * platform/graphics/filters/FilterOperation.h:
+
 2012-07-31  Thiago Marcos P. Santos  <thiago.santos@intel.com>
 
         Regression(r124135): SVG tests crashing on ports using Cairo
index 63507f5..b4e8b29 100644 (file)
@@ -65,13 +65,6 @@ public:
         return adoptRef(new CustomFilterOperation(program, sortedParameters, meshRows, meshColumns, meshBoxType, meshType));
     }
     
-    virtual PassRefPtr<FilterOperation> clone() const
-    {
-        // Some member vars (e.g., m_program) are not thread-safe, so
-        // we can't be cloned.
-        return 0;
-    }
-    
     CustomFilterProgram* program() const { return m_program.get(); }
     
     const CustomFilterParameterList& parameters() { return m_parameters; }
index b8796e8..2a4f08a 100644 (file)
@@ -82,8 +82,6 @@ public:
     // True if the the value of one pixel can affect the value of another pixel under this operation, such as blur.
     virtual bool movesPixels() const { return false; }
 
-    virtual PassRefPtr<FilterOperation> clone() const = 0;
-
 protected:
     FilterOperation(OperationType type)
         : m_type(type)
@@ -100,11 +98,6 @@ public:
         return adoptRef(new DefaultFilterOperation(type));
     }
 
-    virtual PassRefPtr<FilterOperation> clone() const
-    {
-        return adoptRef(new DefaultFilterOperation(m_type));
-    }
-
 private:
 
     virtual bool operator==(const FilterOperation& o) const
@@ -127,11 +120,6 @@ public:
         return adoptRef(new PassthroughFilterOperation());
     }
 
-    virtual PassRefPtr<FilterOperation> clone() const
-    {
-        return adoptRef(new PassthroughFilterOperation());
-    }
-
 private:
 
     virtual bool operator==(const FilterOperation& o) const
@@ -152,12 +140,6 @@ public:
         return adoptRef(new ReferenceFilterOperation(url, fragment, type));
     }
 
-    virtual PassRefPtr<FilterOperation> clone() const
-    {
-        // Unimplemented
-        return 0;
-    }
-
     virtual bool affectsOpacity() const { return true; }
     virtual bool movesPixels() const { return true; }
 
@@ -199,11 +181,6 @@ public:
         return adoptRef(new BasicColorMatrixFilterOperation(amount, type));
     }
 
-    virtual PassRefPtr<FilterOperation> clone() const
-    {
-        return adoptRef(new BasicColorMatrixFilterOperation(m_amount, m_type));
-    }
-
     double amount() const { return m_amount; }
 
     virtual PassRefPtr<FilterOperation> blend(const FilterOperation* from, double progress, bool blendToPassthrough = false);
@@ -236,11 +213,6 @@ public:
         return adoptRef(new BasicComponentTransferFilterOperation(amount, type));
     }
 
-    virtual PassRefPtr<FilterOperation> clone() const
-    {
-        return adoptRef(new BasicComponentTransferFilterOperation(m_amount, m_type));
-    }
-
     double amount() const { return m_amount; }
 
     virtual bool affectsOpacity() const { return m_type == OPACITY; }
@@ -274,11 +246,6 @@ public:
         return adoptRef(new GammaFilterOperation(amplitude, exponent, offset, type));
     }
 
-    virtual PassRefPtr<FilterOperation> clone() const
-    {
-        return adoptRef(new GammaFilterOperation(m_amplitude, m_exponent, m_offset, m_type));
-    }
-
     double amplitude() const { return m_amplitude; }
     double exponent() const { return m_exponent; }
     double offset() const { return m_offset; }
@@ -314,11 +281,6 @@ public:
         return adoptRef(new BlurFilterOperation(stdDeviation, type));
     }
 
-    virtual PassRefPtr<FilterOperation> clone() const
-    {
-        return adoptRef(new BlurFilterOperation(m_stdDeviation, m_type));
-    }
-
     Length stdDeviation() const { return m_stdDeviation; }
 
     virtual bool affectsOpacity() const { return true; }
@@ -351,11 +313,6 @@ public:
         return adoptRef(new DropShadowFilterOperation(location, stdDeviation, color, type));
     }
 
-    virtual PassRefPtr<FilterOperation> clone() const
-    {
-        return adoptRef(new DropShadowFilterOperation(m_location, m_stdDeviation, m_color, m_type));
-    }
-
     int x() const { return m_location.x(); }
     int y() const { return m_location.y(); }
     IntPoint location() const { return m_location; }