{ClipPathOperation, FilterOperation}::getOperationType() should not include 'get'
authortimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Oct 2013 22:33:57 +0000 (22:33 +0000)
committertimothy_horton@apple.com <timothy_horton@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Oct 2013 22:33:57 +0000 (22:33 +0000)
https://bugs.webkit.org/show_bug.cgi?id=123172

Reviewed by Simon Fraser.

No new tests, just a rename.

"get" in WebCore tends to mean that the function has out arguments; these have no arguments.
Rename FilterOperation::getOperationType() to FilterOperation::type().
I noticed that ClipPathOperation had the same mistake, so I fixed it there too.

Removed long and useless list of WebCore files.

* Shared/WebCoreArgumentCoders.cpp:
(CoreIPC::encodeFilterOperation):
* Shared/mac/RemoteLayerTreeTransaction.mm:
(WebKit::RemoteLayerTreeTextStream::operator<<):
Adopt the new name.

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

28 files changed:
Source/WebCore/ChangeLog
Source/WebCore/css/CSSComputedStyleDeclaration.cpp
Source/WebCore/css/StyleResolver.cpp
Source/WebCore/page/animation/CSSPropertyAnimation.cpp
Source/WebCore/platform/graphics/GraphicsLayer.cpp
Source/WebCore/platform/graphics/ca/GraphicsLayerCA.cpp
Source/WebCore/platform/graphics/ca/mac/PlatformCAFiltersMac.mm
Source/WebCore/platform/graphics/ca/mac/PlatformCALayerMac.mm
Source/WebCore/platform/graphics/filters/FilterOperation.h
Source/WebCore/platform/graphics/filters/FilterOperations.cpp
Source/WebCore/platform/graphics/transforms/IdentityTransformOperation.h
Source/WebCore/platform/graphics/transforms/Matrix3DTransformOperation.h
Source/WebCore/platform/graphics/transforms/MatrixTransformOperation.h
Source/WebCore/platform/graphics/transforms/PerspectiveTransformOperation.h
Source/WebCore/platform/graphics/transforms/RotateTransformOperation.h
Source/WebCore/platform/graphics/transforms/ScaleTransformOperation.h
Source/WebCore/platform/graphics/transforms/SkewTransformOperation.h
Source/WebCore/platform/graphics/transforms/TransformOperation.h
Source/WebCore/platform/graphics/transforms/TranslateTransformOperation.h
Source/WebCore/rendering/ClipPathOperation.h
Source/WebCore/rendering/FilterEffectRenderer.cpp
Source/WebCore/rendering/RenderLayer.cpp
Source/WebCore/rendering/RenderLayerFilterInfo.cpp
Source/WebCore/rendering/style/RenderStyle.cpp
Source/WebCore/rendering/svg/SVGRenderingContext.cpp
Source/WebKit2/ChangeLog
Source/WebKit2/Shared/WebCoreArgumentCoders.cpp
Source/WebKit2/Shared/mac/RemoteLayerTreeTransaction.mm

index d40a7f4..f26990a 100644 (file)
@@ -1,3 +1,18 @@
+2013-10-22  Tim Horton  <timothy_horton@apple.com>
+
+        {ClipPathOperation, FilterOperation}::getOperationType() should not include 'get'
+        https://bugs.webkit.org/show_bug.cgi?id=123172
+
+        Reviewed by Simon Fraser.
+
+        No new tests, just a rename.
+
+        "get" in WebCore tends to mean that the function has out arguments; these have no arguments.
+        Rename FilterOperation::getOperationType() to FilterOperation::type().
+        I noticed that ClipPathOperation had the same mistake, so I fixed it there too.
+
+        Removed long and useless list of files.
+
 2013-10-22  Samuel White  <samuel_white@apple.com>
 
         AX: Add paramAttrs to fetch start and end text markers in a given rect.
index 88c0d7c..d1bea04 100644 (file)
@@ -952,7 +952,7 @@ PassRefPtr<CSSValue> ComputedStyleExtractor::valueForFilter(const RenderObject*
     Vector<RefPtr<FilterOperation>>::const_iterator end = filterOperations.operations().end();
     for (Vector<RefPtr<FilterOperation>>::const_iterator it = filterOperations.operations().begin(); it != end; ++it) {
         FilterOperation* filterOperation = (*it).get();
-        switch (filterOperation->getOperationType()) {
+        switch (filterOperation->type()) {
         case FilterOperation::REFERENCE: {
             ReferenceFilterOperation* referenceOperation = static_cast<ReferenceFilterOperation*>(filterOperation);
             filterValue = WebKitCSSFilterValue::create(WebKitCSSFilterValue::ReferenceFilterOperation);
@@ -2828,10 +2828,10 @@ PassRefPtr<CSSValue> ComputedStyleExtractor::propertyValue(CSSPropertyID propert
             return counterToCSSValue(style.get(), propertyID);
         case CSSPropertyWebkitClipPath:
             if (ClipPathOperation* operation = style->clipPath()) {
-                if (operation->getOperationType() == ClipPathOperation::SHAPE)
+                if (operation->type() == ClipPathOperation::SHAPE)
                     return valueForBasicShape(style.get(), static_cast<ShapeClipPathOperation*>(operation)->basicShape());
 #if ENABLE(SVG)
-                else if (operation->getOperationType() == ClipPathOperation::REFERENCE) {
+                if (operation->type() == ClipPathOperation::REFERENCE) {
                     ReferenceClipPathOperation* referenceOperation = static_cast<ReferenceClipPathOperation*>(operation);
                     return CSSPrimitiveValue::create(referenceOperation->url(), CSSPrimitiveValue::CSS_URI);
                 }
index caf560b..3c8e781 100644 (file)
@@ -3401,7 +3401,7 @@ void StyleResolver::loadPendingSVGDocuments()
     Vector<RefPtr<FilterOperation>>& filterOperations = state.style()->mutableFilter().operations();
     for (unsigned i = 0; i < filterOperations.size(); ++i) {
         RefPtr<FilterOperation> filterOperation = filterOperations.at(i);
-        if (filterOperation->getOperationType() == FilterOperation::REFERENCE) {
+        if (filterOperation->type() == FilterOperation::REFERENCE) {
             ReferenceFilterOperation* referenceFilter = static_cast<ReferenceFilterOperation*>(filterOperation.get());
 
             WebKitCSSSVGDocumentValue* value = state.pendingSVGDocuments().get(referenceFilter);
@@ -3464,7 +3464,7 @@ void StyleResolver::loadPendingShaders()
     Vector<RefPtr<FilterOperation>>& filterOperations = m_state.style()->mutableFilter().operations();
     for (unsigned i = 0; i < filterOperations.size(); ++i) {
         RefPtr<FilterOperation> filterOperation = filterOperations.at(i);
-        if (filterOperation->getOperationType() == FilterOperation::CUSTOM) {
+        if (filterOperation->type() == FilterOperation::CUSTOM) {
             CustomFilterOperation* customFilter = static_cast<CustomFilterOperation*>(filterOperation.get());
             ASSERT(customFilter->program());
             StyleCustomFilterProgram* program = static_cast<StyleCustomFilterProgram*>(customFilter->program());
index ed35ec3..7ad05d5 100644 (file)
@@ -131,7 +131,7 @@ static inline PassRefPtr<ClipPathOperation> blendFunc(const AnimationBase*, Clip
         return to;
 
     // Other clip-path operations than BasicShapes can not be animated.
-    if (from->getOperationType() != ClipPathOperation::SHAPE || to->getOperationType() != ClipPathOperation::SHAPE)
+    if (from->type() != ClipPathOperation::SHAPE || to->type() != ClipPathOperation::SHAPE)
         return to;
 
     const BasicShape* fromShape = static_cast<ShapeClipPathOperation*>(from)->basicShape();
index 93d9c43..2bab702 100644 (file)
@@ -512,7 +512,7 @@ int GraphicsLayer::validateTransformOperations(const KeyframeValueList& valueLis
     double maxRotAngle = -1.0;
         
     for (size_t j = 0; j < firstVal.operations().size(); ++j) {
-        TransformOperation::OperationType type = firstVal.operations().at(j)->getOperationType();
+        TransformOperation::OperationType type = firstVal.operations().at(j)->type();
         
         // if this is a rotation entry, we need to see if any angle differences are >= 180 deg
         if (type == TransformOperation::ROTATE_X ||
index 14f538b..ac7ae2e 100644 (file)
@@ -2247,7 +2247,7 @@ bool GraphicsLayerCA::createAnimationFromKeyframes(const KeyframeValueList& valu
 
 bool GraphicsLayerCA::appendToUncommittedAnimations(const KeyframeValueList& valueList, const TransformOperations* operations, const Animation* animation, const String& animationName, const IntSize& boxSize, int animationIndex, double timeOffset, bool isMatrixAnimation)
 {
-    TransformOperation::OperationType transformOp = isMatrixAnimation ? TransformOperation::MATRIX_3D : operations->operations().at(animationIndex)->getOperationType();
+    TransformOperation::OperationType transformOp = isMatrixAnimation ? TransformOperation::MATRIX_3D : operations->operations().at(animationIndex)->type();
     bool additive = animationIndex > 0;
     bool isKeyframe = valueList.size() > 2;
 
@@ -2359,7 +2359,7 @@ bool GraphicsLayerCA::appendToUncommittedAnimations(const KeyframeValueList& val
 {
     bool isKeyframe = valueList.size() > 2;
     
-    FilterOperation::OperationType filterOp = operation->getOperationType();
+    FilterOperation::OperationType filterOp = operation->type();
     int numAnimatedProperties = PlatformCAFilters::numAnimatedFilterProperties(filterOp);
     
     // Each filter might need to animate multiple properties, each with their own keyPath. The keyPath is always of the form:
@@ -2407,7 +2407,7 @@ bool GraphicsLayerCA::createFilterAnimationsFromKeyframes(const KeyframeValueLis
 
     // FIXME: We can't currently hardware animate shadows.
     for (int i = 0; i < numAnimations; ++i) {
-        if (operations.at(i)->getOperationType() == FilterOperation::DROP_SHADOW)
+        if (operations.at(i)->type() == FilterOperation::DROP_SHADOW)
             return false;
     }
 
@@ -2698,12 +2698,12 @@ bool GraphicsLayerCA::setFilterAnimationEndpoints(const KeyframeValueList& value
     ASSERT(fromOperation || toOperation);
 
     if (!fromOperation) {
-        defaultFromOperation = DefaultFilterOperation::create(toOperation->getOperationType());
+        defaultFromOperation = DefaultFilterOperation::create(toOperation->type());
         fromOperation = defaultFromOperation.get();
     }
 
     if (!toOperation) {
-        defaultToOperation = DefaultFilterOperation::create(fromOperation->getOperationType());
+        defaultToOperation = DefaultFilterOperation::create(fromOperation->type());
         toOperation = defaultToOperation.get();
     }
 
index a9172d2..d8d775b 100644 (file)
@@ -105,7 +105,7 @@ void PlatformCAFilters::setFiltersOnLayer(PlatformLayer* layer, const FilterOper
     for (unsigned i = 0; i < filters.size(); ++i) {
         String filterName = String::format("filter_%d", i);
         const FilterOperation* filterOperation = filters.at(i);
-        switch (filterOperation->getOperationType()) {
+        switch (filterOperation->type()) {
         case FilterOperation::DROP_SHADOW: {
             // FIXME: For now assume drop shadow is the last filter, put it on the layer.
             // <rdar://problem/10959969> Handle case where drop-shadow is not the last filter.
@@ -334,7 +334,7 @@ RetainPtr<NSValue> PlatformCAFilters::filterValueForOperation(const FilterOperat
 #if USE_CA_FILTERS
     UNUSED_PARAM(internalFilterPropertyIndex);
 #endif
-    FilterOperation::OperationType type = operation->getOperationType();
+    FilterOperation::OperationType type = operation->type();
     RetainPtr<id> value;
     
     switch (type) {
@@ -513,7 +513,7 @@ RetainPtr<NSValue> PlatformCAFilters::filterValueForOperation(const FilterOperat
 #if USE_CA_FILTERS
 RetainPtr<NSValue> PlatformCAFilters::colorMatrixValueForFilter(const FilterOperation& filterOperation)
 {
-    switch (filterOperation.getOperationType()) {
+    switch (filterOperation.type()) {
     case FilterOperation::SEPIA: {
         const BasicColorMatrixFilterOperation& op = static_cast<const BasicColorMatrixFilterOperation&>(filterOperation);
         double t = op.amount();
index 05cdb12..b734746 100644 (file)
@@ -651,7 +651,7 @@ bool PlatformCALayerMac::filtersCanBeComposited(const FilterOperations& filters)
     
     for (unsigned i = 0; i < filters.size(); ++i) {
         const FilterOperation* filterOperation = filters.at(i);
-        switch (filterOperation->getOperationType()) {
+        switch (filterOperation->type()) {
             case FilterOperation::REFERENCE:
 #if ENABLE(CSS_SHADERS)
             case FilterOperation::CUSTOM:
index 19afb55..b2c4632 100644 (file)
@@ -97,8 +97,8 @@ public:
         return 0; 
     }
 
-    virtual OperationType getOperationType() const { return m_type; }
-    virtual bool isSameType(const FilterOperation& o) const { return o.getOperationType() == m_type; }
+    virtual OperationType type() const { return m_type; }
+    virtual bool isSameType(const FilterOperation& o) const { return o.type() == m_type; }
     
     virtual bool isDefault() const { return false; }
 
index db65943..a28538a 100644 (file)
@@ -91,7 +91,7 @@ bool FilterOperations::operationsMatch(const FilterOperations& other) const
 bool FilterOperations::hasCustomFilter() const
 {
     for (size_t i = 0; i < m_operations.size(); ++i) {
-        FilterOperation::OperationType type = m_operations.at(i)->getOperationType();
+        FilterOperation::OperationType type = m_operations.at(i)->type();
         if (type == FilterOperation::CUSTOM || type == FilterOperation::VALIDATED_CUSTOM)
             return true;
     }
@@ -102,7 +102,7 @@ bool FilterOperations::hasCustomFilter() const
 bool FilterOperations::hasReferenceFilter() const
 {
     for (size_t i = 0; i < m_operations.size(); ++i) {
-        if (m_operations.at(i)->getOperationType() == FilterOperation::REFERENCE)
+        if (m_operations.at(i)->type() == FilterOperation::REFERENCE)
             return true;
     }
     return false;
@@ -111,7 +111,7 @@ bool FilterOperations::hasReferenceFilter() const
 bool FilterOperations::hasOutsets() const
 {
     for (size_t i = 0; i < m_operations.size(); ++i) {
-        FilterOperation::OperationType operationType = m_operations.at(i).get()->getOperationType();
+        FilterOperation::OperationType operationType = m_operations.at(i).get()->type();
         if (operationType == FilterOperation::BLUR || operationType == FilterOperation::DROP_SHADOW)
             return true;
     }
@@ -123,7 +123,7 @@ FilterOutsets FilterOperations::outsets() const
     FilterOutsets totalOutsets;
     for (size_t i = 0; i < m_operations.size(); ++i) {
         FilterOperation* filterOperation = m_operations.at(i).get();
-        switch (filterOperation->getOperationType()) {
+        switch (filterOperation->type()) {
         case FilterOperation::BLUR: {
             BlurFilterOperation* blurOperation = static_cast<BlurFilterOperation*>(filterOperation);
             float stdDeviation = floatValueForLength(blurOperation->stdDeviation(), 0);
index e3eccc1..c1f7f45 100644 (file)
@@ -38,8 +38,8 @@ public:
         
 private:
     virtual bool isIdentity() const { return true; }
-    virtual OperationType getOperationType() const { return IDENTITY; }
-    virtual bool isSameType(const TransformOperation& o) const { return o.getOperationType() == IDENTITY; }
+    virtual OperationType type() const { return IDENTITY; }
+    virtual bool isSameType(const TransformOperation& o) const { return o.type() == IDENTITY; }
 
     virtual bool operator==(const TransformOperation& o) const
     {
index b0577a6..889c2b7 100644 (file)
@@ -42,8 +42,8 @@ public:
 private:    
     virtual bool isIdentity() const { return m_matrix.isIdentity(); }
 
-    virtual OperationType getOperationType() const { return MATRIX_3D; }
-    virtual bool isSameType(const TransformOperation& o) const { return o.getOperationType() == MATRIX_3D; }
+    virtual OperationType type() const { return MATRIX_3D; }
+    virtual bool isSameType(const TransformOperation& o) const { return o.type() == MATRIX_3D; }
 
     virtual bool operator==(const TransformOperation& o) const
     {
index 45f9ea7..76c5dff 100644 (file)
@@ -47,8 +47,8 @@ public:
 private:
     virtual bool isIdentity() const { return m_a == 1 && m_b == 0 && m_c == 0 && m_d == 1 && m_e == 0 && m_f == 0; }
 
-    virtual OperationType getOperationType() const { return MATRIX; }
-    virtual bool isSameType(const TransformOperation& o) const { return o.getOperationType() == MATRIX; }
+    virtual OperationType type() const { return MATRIX; }
+    virtual bool isSameType(const TransformOperation& o) const { return o.type() == MATRIX; }
 
     virtual bool operator==(const TransformOperation& o) const
     {
index 3730d42..73aa680 100644 (file)
@@ -43,8 +43,8 @@ public:
     
 private:
     virtual bool isIdentity() const { return !floatValueForLength(m_p, 1); }
-    virtual OperationType getOperationType() const { return PERSPECTIVE; }
-    virtual bool isSameType(const TransformOperation& o) const { return o.getOperationType() == PERSPECTIVE; }
+    virtual OperationType type() const { return PERSPECTIVE; }
+    virtual bool isSameType(const TransformOperation& o) const { return o.type() == PERSPECTIVE; }
 
     virtual bool operator==(const TransformOperation& o) const
     {
index 0d495c5..f47a51e 100644 (file)
@@ -49,8 +49,8 @@ public:
 private:
     virtual bool isIdentity() const { return m_angle == 0; }
 
-    virtual OperationType getOperationType() const { return m_type; }
-    virtual bool isSameType(const TransformOperation& o) const { return o.getOperationType() == m_type; }
+    virtual OperationType type() const { return m_type; }
+    virtual bool isSameType(const TransformOperation& o) const { return o.type() == m_type; }
 
     virtual bool operator==(const TransformOperation& o) const
     {
index 7343d09..62beb23 100644 (file)
@@ -48,8 +48,8 @@ public:
 private:
     virtual bool isIdentity() const { return m_x == 1 &&  m_y == 1 &&  m_z == 1; }
 
-    virtual OperationType getOperationType() const { return m_type; }
-    virtual bool isSameType(const TransformOperation& o) const { return o.getOperationType() == m_type; }
+    virtual OperationType type() const { return m_type; }
+    virtual bool isSameType(const TransformOperation& o) const { return o.type() == m_type; }
 
     virtual bool operator==(const TransformOperation& o) const
     {
index ddde986..e9f0c81 100644 (file)
@@ -41,8 +41,8 @@ public:
 
 private:
     virtual bool isIdentity() const { return m_angleX == 0 && m_angleY == 0; }
-    virtual OperationType getOperationType() const { return m_type; }
-    virtual bool isSameType(const TransformOperation& o) const { return o.getOperationType() == m_type; }
+    virtual OperationType type() const { return m_type; }
+    virtual bool isSameType(const TransformOperation& o) const { return o.type() == m_type; }
 
     virtual bool operator==(const TransformOperation& o) const
     {
index 1e7ebb0..f9f975f 100644 (file)
@@ -63,12 +63,12 @@ public:
 
     virtual PassRefPtr<TransformOperation> blend(const TransformOperation* from, double progress, bool blendToIdentity = false) = 0;
 
-    virtual OperationType getOperationType() const = 0;
+    virtual OperationType type() const = 0;
     virtual bool isSameType(const TransformOperation&) const { return false; }
     
     bool is3DOperation() const
     {
-        OperationType opType = getOperationType();
+        OperationType opType = type();
         return opType == SCALE_Z ||
                opType == SCALE_3D ||
                opType == TRANSLATE_Z ||
index fc2bc1f..63cf587 100644 (file)
@@ -54,8 +54,8 @@ public:
 private:
     virtual bool isIdentity() const { return !floatValueForLength(m_x, 1) && !floatValueForLength(m_y, 1) && !floatValueForLength(m_z, 1); }
 
-    virtual OperationType getOperationType() const { return m_type; }
-    virtual bool isSameType(const TransformOperation& o) const { return o.getOperationType() == m_type; }
+    virtual OperationType type() const { return m_type; }
+    virtual bool isSameType(const TransformOperation& o) const { return o.type() == m_type; }
 
     virtual bool operator==(const TransformOperation& o) const
     {
index d0d4211..6ab8852 100644 (file)
@@ -51,8 +51,8 @@ public:
     virtual bool operator==(const ClipPathOperation&) const = 0;
     bool operator!=(const ClipPathOperation& o) const { return !(*this == o); }
 
-    virtual OperationType getOperationType() const { return m_type; }
-    virtual bool isSameType(const ClipPathOperation& o) const { return o.getOperationType() == m_type; }
+    virtual OperationType type() const { return m_type; }
+    virtual bool isSameType(const ClipPathOperation& o) const { return o.type() == m_type; }
 
 protected:
     explicit ClipPathOperation(OperationType type)
index 1da7f04..d5a47f5 100644 (file)
@@ -197,7 +197,7 @@ bool FilterEffectRenderer::build(RenderElement* renderer, const FilterOperations
     for (size_t i = 0; i < operations.operations().size(); ++i) {
         RefPtr<FilterEffect> effect;
         FilterOperation* filterOperation = operations.operations().at(i).get();
-        switch (filterOperation->getOperationType()) {
+        switch (filterOperation->type()) {
         case FilterOperation::REFERENCE: {
             ReferenceFilterOperation* referenceOperation = static_cast<ReferenceFilterOperation*>(filterOperation);
             effect = buildReferenceFilter(renderer, previousEffect, referenceOperation);
@@ -360,7 +360,7 @@ bool FilterEffectRenderer::build(RenderElement* renderer, const FilterOperations
             effect->setClipsToBounds(consumer == FilterFunction);
             effect->setOperatingColorSpace(ColorSpaceDeviceRGB);
             
-            if (filterOperation->getOperationType() != FilterOperation::REFERENCE) {
+            if (filterOperation->type() != FilterOperation::REFERENCE) {
                 effect->inputEffects().append(previousEffect);
                 m_effects.append(effect);
             }
index cfc1b8c..d1c2ba2 100644 (file)
@@ -3705,7 +3705,7 @@ bool RenderLayer::setupClipPath(GraphicsContext* context, const LayerPaintingInf
     RenderStyle* style = renderer().style();
 
     ASSERT(style->clipPath());
-    if (style->clipPath()->getOperationType() == ClipPathOperation::SHAPE) {
+    if (style->clipPath()->type() == ClipPathOperation::SHAPE) {
         ShapeClipPathOperation* clipPath = static_cast<ShapeClipPathOperation*>(style->clipPath());
 
         if (!rootRelativeBoundsComputed) {
@@ -3719,7 +3719,7 @@ bool RenderLayer::setupClipPath(GraphicsContext* context, const LayerPaintingInf
     }
 
 #if ENABLE(SVG)
-    if (style->clipPath()->getOperationType() == ClipPathOperation::REFERENCE) {
+    if (style->clipPath()->type() == ClipPathOperation::REFERENCE) {
         ReferenceClipPathOperation* referenceClipPathOperation = static_cast<ReferenceClipPathOperation*>(style->clipPath());
         Element* element = renderer().document().getElementById(referenceClipPathOperation->fragment());
         if (element && element->hasTagName(SVGNames::clipPathTag) && element->renderer()) {
@@ -6413,7 +6413,7 @@ FilterOperations RenderLayer::computeFilterOperations(const RenderStyle* style)
     FilterOperations outputFilters;
     for (size_t i = 0; i < filters.size(); ++i) {
         RefPtr<FilterOperation> filterOperation = filters.operations().at(i);
-        if (filterOperation->getOperationType() == FilterOperation::CUSTOM) {
+        if (filterOperation->type() == FilterOperation::CUSTOM) {
             // We have to wait until the program of CSS Shaders is loaded before setting it on the layer.
             // Note that we will handle the loading of the shaders and repainting of the layer in updateOrRemoveFilterClients.
             const CustomFilterOperation* customOperation = static_cast<const CustomFilterOperation*>(filterOperation.get());
index c075ecf..90a1e51 100644 (file)
@@ -111,7 +111,7 @@ void RenderLayer::FilterInfo::updateReferenceFilterClients(const FilterOperation
     removeReferenceFilterClients();
     for (size_t i = 0, size = operations.size(); i < size; ++i) {
         FilterOperation* filterOperation = operations.operations()[i].get();
-        if (filterOperation->getOperationType() != FilterOperation::REFERENCE)
+        if (filterOperation->type() != FilterOperation::REFERENCE)
             continue;
         ReferenceFilterOperation* referenceFilterOperation = static_cast<ReferenceFilterOperation*>(filterOperation);
         CachedSVGDocumentReference* documentReference = referenceFilterOperation->cachedSVGDocumentReference();
@@ -166,7 +166,7 @@ void RenderLayer::FilterInfo::updateCustomFilterClients(const FilterOperations&
     Vector<RefPtr<CustomFilterProgram>> cachedCustomFilterPrograms;
     for (size_t i = 0, size = operations.size(); i < size; ++i) {
         const FilterOperation* filterOperation = operations.operations()[i].get();
-        if (filterOperation->getOperationType() != FilterOperation::CUSTOM)
+        if (filterOperation->type() != FilterOperation::CUSTOM)
             continue;
         const CustomFilterOperation* customFilterOperation = static_cast<const CustomFilterOperation*>(filterOperation);
         CustomFilterProgram* program = customFilterOperation->program();
index 56cbf40..31674de 100644 (file)
@@ -993,7 +993,7 @@ inline bool requireTransformOrigin(const Vector<RefPtr<TransformOperation>>& tra
 
     unsigned size = transformOperations.size();
     for (unsigned i = 0; i < size; ++i) {
-        TransformOperation::OperationType type = transformOperations[i]->getOperationType();
+        TransformOperation::OperationType type = transformOperations[i]->type();
         if (type != TransformOperation::TRANSLATE_X
             && type != TransformOperation::TRANSLATE_Y
             && type != TransformOperation::TRANSLATE 
index 96adf47..be5b76e 100644 (file)
@@ -127,7 +127,7 @@ void SVGRenderingContext::prepareToRenderSVGContent(RenderObject* object, PaintI
     }
 
     ClipPathOperation* clipPathOperation = style->clipPath();
-    if (clipPathOperation && clipPathOperation->getOperationType() == ClipPathOperation::SHAPE) {
+    if (clipPathOperation && clipPathOperation->type() == ClipPathOperation::SHAPE) {
         ShapeClipPathOperation* clipPath = static_cast<ShapeClipPathOperation*>(clipPathOperation);
         m_paintInfo->context->clipPath(clipPath->path(object->objectBoundingBox()), clipPath->windRule());
     }
index 6667fe4..d85f59b 100644 (file)
@@ -1,3 +1,16 @@
+2013-10-22  Tim Horton  <timothy_horton@apple.com>
+
+        {ClipPathOperation, FilterOperation}::getOperationType() should not include 'get'
+        https://bugs.webkit.org/show_bug.cgi?id=123172
+
+        Reviewed by Simon Fraser.
+
+        * Shared/WebCoreArgumentCoders.cpp:
+        (CoreIPC::encodeFilterOperation):
+        * Shared/mac/RemoteLayerTreeTransaction.mm:
+        (WebKit::RemoteLayerTreeTextStream::operator<<):
+        Adopt the new name.
+
 2013-10-22  Csaba Osztrogon√°c  <ossy@webkit.org>
 
         URTBF after r157803 for !ENABLE(CSS_FILTERS) platforms.
index a1865fe..6bb97f0 100644 (file)
@@ -1184,9 +1184,9 @@ bool ArgumentCoder<WebCore::UserScript>::decode(ArgumentDecoder& decoder, WebCor
 #if ENABLE(CSS_FILTERS)
 static void encodeFilterOperation(ArgumentEncoder& encoder, const FilterOperation& filter)
 {
-    encoder.encodeEnum(filter.getOperationType());
+    encoder.encodeEnum(filter.type());
 
-    switch (filter.getOperationType()) {
+    switch (filter.type()) {
     case FilterOperation::REFERENCE: {
         const auto& referenceFilter = static_cast<const ReferenceFilterOperation&>(filter);
         encoder << referenceFilter.url();
index 625b3ba..32132a5 100644 (file)
@@ -416,7 +416,7 @@ RemoteLayerTreeTextStream& RemoteLayerTreeTextStream::operator<<(const FilterOpe
     RemoteLayerTreeTextStream& ts = *this;
     for (size_t i = 0; i < filters.size(); ++i) {
         const auto filter = filters.at(i);
-        switch (filter->getOperationType()) {
+        switch (filter->type()) {
         case FilterOperation::REFERENCE:
             ts << "reference";
             break;