2011-04-10 Luke Macpherson <macpherson@chromium.org>
authorcommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 11 Apr 2011 03:49:38 +0000 (03:49 +0000)
committercommit-queue@webkit.org <commit-queue@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Mon, 11 Apr 2011 03:49:38 +0000 (03:49 +0000)
        Reviewed by Simon Fraser.

        Rename CSSStyleApplyProperty::inherit,initial,value applyInheritValue,applyInitialValue,applyValue
        https://bugs.webkit.org/show_bug.cgi?id=58212

        No new tests - function renaming only / no functionality changes.

        * css/CSSStyleApplyProperty.cpp:
        Rename all instances of inherit, initial and value.
        * css/CSSStyleApplyProperty.h:
        Rename all instances of inherit, initial and value.
        * css/CSSStyleSelector.cpp:
        Rename all instances of inherit, initial and value.

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

Source/WebCore/ChangeLog
Source/WebCore/css/CSSStyleApplyProperty.cpp
Source/WebCore/css/CSSStyleApplyProperty.h
Source/WebCore/css/CSSStyleSelector.cpp

index 9d2074f..dd1f9ba 100644 (file)
@@ -1,3 +1,19 @@
+2011-04-10  Luke Macpherson   <macpherson@chromium.org>
+
+        Reviewed by Simon Fraser.
+
+        Rename CSSStyleApplyProperty::inherit,initial,value applyInheritValue,applyInitialValue,applyValue
+        https://bugs.webkit.org/show_bug.cgi?id=58212
+
+        No new tests - function renaming only / no functionality changes.
+
+        * css/CSSStyleApplyProperty.cpp:
+        Rename all instances of inherit, initial and value.
+        * css/CSSStyleApplyProperty.h:
+        Rename all instances of inherit, initial and value.
+        * css/CSSStyleSelector.cpp:
+        Rename all instances of inherit, initial and value.
+
 2011-04-10  Mike Lawther  <mikelawther@chromium.org>
 
         Reviewed by Ojan Vafai.
index e111bed..d11b618 100644 (file)
@@ -40,9 +40,9 @@ namespace WebCore {
 
 class ApplyPropertyNull : public ApplyPropertyBase {
 public:
-    virtual void inherit(CSSStyleSelector*) const {}
-    virtual void initial(CSSStyleSelector*) const {}
-    virtual void value(CSSStyleSelector*, CSSValue*) const {}
+    virtual void applyInheritValue(CSSStyleSelector*) const {}
+    virtual void applyInitialValue(CSSStyleSelector*) const {}
+    virtual void applyValue(CSSStyleSelector*, CSSValue*) const {}
 };
 
 template <typename T>
@@ -55,17 +55,17 @@ public:
     {
     }
 
-    virtual void inherit(CSSStyleSelector* selector) const
+    virtual void applyInheritValue(CSSStyleSelector* selector) const
     {
         (selector->style()->*m_setter)((selector->parentStyle()->*m_getter)());
     }
 
-    virtual void initial(CSSStyleSelector* selector) const
+    virtual void applyInitialValue(CSSStyleSelector* selector) const
     {
         (selector->style()->*m_setter)((*m_initial)());
     }
 
-    virtual void value(CSSStyleSelector* selector, CSSValue* value) const
+    virtual void applyValue(CSSStyleSelector* selector, CSSValue* value) const
     {
         if (value->isPrimitiveValue())
             (selector->style()->*m_setter)(*(static_cast<CSSPrimitiveValue*>(value)));
@@ -86,7 +86,7 @@ public:
         , m_setter(setter)
     {
     }
-    virtual void inherit(CSSStyleSelector* selector) const
+    virtual void applyInheritValue(CSSStyleSelector* selector) const
     {
         const Color& color = (selector->parentStyle()->*m_getter)();
         if (m_defaultValue && !color.isValid())
@@ -94,12 +94,12 @@ public:
         else
             (selector->style()->*m_setter)(color);
     }
-    virtual void initial(CSSStyleSelector* selector) const
+    virtual void applyInitialValue(CSSStyleSelector* selector) const
     {
         Color color;
         (selector->style()->*m_setter)(color);
     }
-    virtual void value(CSSStyleSelector* selector, CSSValue* value) const
+    virtual void applyValue(CSSStyleSelector* selector, CSSValue* value) const
     {
         if (value->isPrimitiveValue())
             (selector->style()->*m_setter)(selector->getColorFromPrimitiveValue(static_cast<CSSPrimitiveValue*>(value)));
@@ -118,20 +118,20 @@ public:
     {
     }
 
-    virtual void initial(CSSStyleSelector* selector) const
+    virtual void applyInitialValue(CSSStyleSelector* selector) const
     {
         (selector->style()->*m_setter)(m_initialValue());
     }
 
-    virtual void value(CSSStyleSelector* selector, CSSValue* value) const
+    virtual void applyValue(CSSStyleSelector* selector, CSSValue* value) const
     {
         if (!value->isPrimitiveValue())
             return;
 
         if ((static_cast<CSSPrimitiveValue*>(value))->getIdent() == CSSValueCurrentcolor)
-            inherit(selector);
+            applyInheritValue(selector);
         else
-            ApplyPropertyColorBase::value(selector, value);
+            ApplyPropertyColorBase::applyValue(selector, value);
     }
 protected:
     Color (*m_initialValue)();
@@ -145,9 +145,9 @@ public:
     {
     }
 
-    virtual void value(CSSStyleSelector* selector, CSSValue* value) const
+    virtual void applyValue(CSSStyleSelector* selector, CSSValue* value) const
     {
-        ApplyPropertyDefault<TextDirection>::value(selector, value);
+        ApplyPropertyDefault<TextDirection>::applyValue(selector, value);
         Element* element = selector->element();
         if (element && selector->element() == element->document()->documentElement())
             element->document()->setDirectionSetOnDocumentElement(true);
@@ -174,7 +174,7 @@ public:
     {
     }
 
-    virtual void inherit(CSSStyleSelector* selector) const
+    virtual void applyInheritValue(CSSStyleSelector* selector) const
     {
         FillLayer* currChild = (selector->style()->*m_accessLayers)();
         FillLayer* prevChild = 0;
@@ -198,7 +198,7 @@ public:
         }
     }
 
-    virtual void initial(CSSStyleSelector* selector) const
+    virtual void applyInitialValue(CSSStyleSelector* selector) const
     {
         FillLayer* currChild = (selector->style()->*m_accessLayers)();
         (currChild->*m_set)((*m_initial)(m_fillLayerType));
@@ -206,7 +206,7 @@ public:
             (currChild->*m_clear)();
     }
 
-    virtual void value(CSSStyleSelector* selector, CSSValue* value) const
+    virtual void applyValue(CSSStyleSelector* selector, CSSValue* value) const
     {
         FillLayer* currChild = (selector->style()->*m_accessLayers)();
         FillLayer* prevChild = 0;
index c31259b..860a4f8 100644 (file)
@@ -39,9 +39,9 @@ class ApplyPropertyBase {
 public:
     ApplyPropertyBase() { }
     virtual ~ApplyPropertyBase() { }
-    virtual void inherit(CSSStyleSelector*) const = 0;
-    virtual void initial(CSSStyleSelector*) const = 0;
-    virtual void value(CSSStyleSelector*, CSSValue*) const = 0;
+    virtual void applyInheritValue(CSSStyleSelector*) const = 0;
+    virtual void applyInitialValue(CSSStyleSelector*) const = 0;
+    virtual void applyValue(CSSStyleSelector*, CSSValue*) const = 0;
 };
 
 class CSSStyleApplyProperty {
@@ -49,22 +49,22 @@ class CSSStyleApplyProperty {
 public:
     static const CSSStyleApplyProperty& sharedCSSStyleApplyProperty();
 
-    void inherit(CSSPropertyID property, CSSStyleSelector* selector) const
+    void applyInheritValue(CSSPropertyID property, CSSStyleSelector* selector) const
     {
         ASSERT(implements(property));
-        propertyValue(property)->inherit(selector);
+        propertyValue(property)->applyInheritValue(selector);
     }
 
-    void initial(CSSPropertyID property, CSSStyleSelector* selector) const
+    void applyInitialValue(CSSPropertyID property, CSSStyleSelector* selector) const
     {
         ASSERT(implements(property));
-        propertyValue(property)->initial(selector);
+        propertyValue(property)->applyInitialValue(selector);
     }
 
-    void value(CSSPropertyID property, CSSStyleSelector* selector, CSSValue* value) const
+    void applyValue(CSSPropertyID property, CSSStyleSelector* selector, CSSValue* value) const
     {
         ASSERT(implements(property));
-        propertyValue(property)->value(selector, value);
+        propertyValue(property)->applyValue(selector, value);
     }
 
     bool implements(CSSPropertyID property) const
index 4e29113..cab48ce 100644 (file)
@@ -3642,11 +3642,11 @@ void CSSStyleSelector::applyProperty(int id, CSSValue *value)
     // check lookup table for implementations and use when available
     if (m_applyProperty.implements(property)) {
         if (isInherit)
-            m_applyProperty.inherit(property, this);
+            m_applyProperty.applyInheritValue(property, this);
         else if (isInitial)
-            m_applyProperty.initial(property, this);
+            m_applyProperty.applyInitialValue(property, this);
         else
-            m_applyProperty.value(property, this, value);
+            m_applyProperty.applyValue(property, this, value);
         return;
     }