Reviewed by Darin and then by Mitz.

        Patch for http://bugs.webkit.org/show_bug.cgi?id=12228
        Cleanup assorted CSS classes

        * css/CSSBorderImageValue.cpp:
        (WebCore::CSSBorderImageValue::CSSBorderImageValue):
        (WebCore::CSSBorderImageValue::cssText):
        * css/CSSBorderImageValue.h:
        * css/CSSComputedStyleDeclaration.cpp:
        (WebCore::valueForLength):
        (WebCore::valueForBorderStyle):
        (WebCore::valueForTextAlign):
        (WebCore::valueForShadow):
        (WebCore::getPositionOffsetValue):
        (WebCore::CSSComputedStyleDeclaration::cssText):
        (WebCore::primitiveValueFromLength):
        (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
        (WebCore::CSSComputedStyleDeclaration::getPropertyPriority):
        (WebCore::CSSComputedStyleDeclaration::removeProperty):
        (WebCore::CSSComputedStyleDeclaration::setProperty):
        (WebCore::CSSComputedStyleDeclaration::length):
        (WebCore::CSSComputedStyleDeclaration::item):
        * css/CSSComputedStyleDeclaration.h:
        * css/CSSCursorImageValue.cpp:
        (WebCore::CSSCursorImageValue::CSSCursorImageValue):
        * css/CSSCursorImageValue.h:
        * css/CSSGrammar.y:
        * css/CSSImageValue.cpp:
        * css/CSSImageValue.h:
        * css/CSSInheritedValue.cpp:
        * css/CSSInheritedValue.h:
        * css/CSSInitialValue.cpp:
        * css/CSSInitialValue.h:
        * css/CSSMutableStyleDeclaration.cpp:
        (WebCore::CSSMutableStyleDeclaration::CSSMutableStyleDeclaration):
        (WebCore::CSSMutableStyleDeclaration::operator=):
        (WebCore::CSSMutableStyleDeclaration::getPropertyValue):
        (WebCore::CSSMutableStyleDeclaration::get4Values):
        (WebCore::CSSMutableStyleDeclaration::getShorthandValue):
        (WebCore::CSSMutableStyleDeclaration::getPropertyCSSValue):
        (WebCore::CSSMutableStyleDeclaration::removeProperty):
        (WebCore::CSSMutableStyleDeclaration::setChanged):
        (WebCore::CSSMutableStyleDeclaration::getPropertyPriority):
        (WebCore::CSSMutableStyleDeclaration::getPropertyShorthand):
        (WebCore::CSSMutableStyleDeclaration::isPropertyImplicit):
        (WebCore::CSSMutableStyleDeclaration::setProperty):
        (WebCore::CSSMutableStyleDeclaration::setImageProperty):
        (WebCore::CSSMutableStyleDeclaration::parseDeclaration):
        (WebCore::CSSMutableStyleDeclaration::addParsedProperties):
        (WebCore::CSSMutableStyleDeclaration::setLengthProperty):
        (WebCore::CSSMutableStyleDeclaration::cssText):
        (WebCore::CSSMutableStyleDeclaration::merge):
        (WebCore::CSSMutableStyleDeclaration::removePropertiesInSet):
        * css/CSSMutableStyleDeclaration.h:
        (WebCore::CSSMutableStyleDeclaration::setProperty):
        (WebCore::CSSMutableStyleDeclaration::removeProperty):
        * css/CSSNamespace.h:
        (WebCore::CSSNamespace::CSSNamespace):
        * css/CSSPrimitiveValue.cpp:
        (WebCore::isCSSTokenizerIdentifier):
        (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
        (WebCore::CSSPrimitiveValue::cleanup):
        (WebCore::CSSPrimitiveValue::computeLengthInt):
        (WebCore::CSSPrimitiveValue::computeLengthIntForLength):
        (WebCore::CSSPrimitiveValue::computeLengthShort):
        (WebCore::CSSPrimitiveValue::computeLengthFloat):
        (WebCore::CSSPrimitiveValue::setFloatValue):
        (WebCore::scaleFactorForConversion):
        (WebCore::CSSPrimitiveValue::getFloatValue):
        (WebCore::CSSPrimitiveValue::setStringValue):
        (WebCore::CSSPrimitiveValue::getStringValue):
        (WebCore::CSSPrimitiveValue::parseString):
        (WebCore::CSSPrimitiveValue::getIdent):
        (WebCore::CSSPrimitiveValue::cssText):
        * css/CSSPrimitiveValue.h:
        (WebCore::CSSPrimitiveValue::getCounterValue):
        (WebCore::CSSPrimitiveValue::getRectValue):
        (WebCore::CSSPrimitiveValue::getRGBColorValue):
        (WebCore::CSSPrimitiveValue::getPairValue):
        (WebCore::CSSPrimitiveValue::getDashboardRegionValue):
        * css/CSSProperty.cpp:
        (WebCore::operator==):
        * css/CSSProperty.h:
        (WebCore::CSSProperty::operator=):
        * css/CSSQuirkPrimitiveValue.h:
        (WebCore::CSSQuirkPrimitiveValue::CSSQuirkPrimitiveValue):
        * css/CSSRuleList.cpp:
        (WebCore::CSSRuleList::CSSRuleList):
        (WebCore::CSSRuleList::deleteRule):
        (WebCore::CSSRuleList::insertRule):
        * css/CSSRuleList.h:
        (WebCore::CSSRuleList::item):
        * css/CSSSelector.cpp:
        (WebCore::CSSSelector::print):
        (WebCore::CSSSelector::specificity):
        (WebCore::CSSSelector::extractPseudoType):
        (WebCore::CSSSelector::operator==):
        (WebCore::CSSSelector::selectorText):
        * css/CSSSelector.h:
        (WebCore::CSSSelector::CSSSelector):
        (WebCore::CSSSelector::~CSSSelector):
        (WebCore::CSSSelector::append):
        (WebCore::CSSSelector::next):
        (WebCore::CSSSelector::pseudoType):
        (WebCore::CSSSelector::hasTag):
        (WebCore::CSSSelector::hasAttribute):
        * css/CSSStyleDeclaration.cpp:
        (WebCore::propertyID):
        (WebCore::CSSStyleDeclaration::getPropertyPriority):
        (WebCore::CSSStyleDeclaration::getPropertyShorthand):
        (WebCore::CSSStyleDeclaration::isPropertyImplicit):
        (WebCore::CSSStyleDeclaration::setProperty):
        (WebCore::CSSStyleDeclaration::isPropertyName):
        (WebCore::CSSStyleDeclaration::parentRule):
        (WebCore::CSSStyleDeclaration::diff):
        (WebCore::CSSStyleDeclaration::copyPropertiesInSet):
        * css/CSSStyleDeclaration.h:
        * css/Counter.h:
        (WebCore::Counter::Counter):
        (WebCore::Counter::listStyleNumber):
        * css/csshelper.cpp:
        (WebCore::parseURL):
        * css/csshelper.h:
        * css/cssparser.h:
        (WebCore::domString):
        (WebCore::atomicString):
        (WebCore::CSSParser::operator|):
        * css/cssstyleselector.cpp:
        (WebCore::CSSStyleSelector::matchRulesForList):
        (WebCore::CSSStyleSelector::checkSelector):
        (WebCore::CSSStyleSelector::checkOneSelector):
        (WebCore::CSSRuleSet::addRule):



git-svn-id: svn://svn.chromium.org/blink/trunk@18850 bbb929c8-8fbe-4397-9dbb-9b2b20218538

third_party/WebKit/WebCore/ChangeLog

+2007-01-14  Sam Weinig  <sam@webkit.org>
+
+        Reviewed by Darin and then by Mitz.
+
+        Patch for http://bugs.webkit.org/show_bug.cgi?id=12228
+        Cleanup assorted CSS classes
+
+        * css/CSSBorderImageValue.cpp:
+        (WebCore::CSSBorderImageValue::CSSBorderImageValue):
+        (WebCore::CSSBorderImageValue::cssText):
+        * css/CSSBorderImageValue.h:
+        * css/CSSComputedStyleDeclaration.cpp:
+        (WebCore::valueForLength):
+        (WebCore::valueForBorderStyle):
+        (WebCore::valueForTextAlign):
+        (WebCore::valueForShadow):
+        (WebCore::getPositionOffsetValue):
+        (WebCore::CSSComputedStyleDeclaration::cssText):
+        (WebCore::primitiveValueFromLength):
+        (WebCore::CSSComputedStyleDeclaration::getPropertyCSSValue):
+        (WebCore::CSSComputedStyleDeclaration::getPropertyPriority):
+        (WebCore::CSSComputedStyleDeclaration::removeProperty):
+        (WebCore::CSSComputedStyleDeclaration::setProperty):
+        (WebCore::CSSComputedStyleDeclaration::length):
+        (WebCore::CSSComputedStyleDeclaration::item):
+        * css/CSSComputedStyleDeclaration.h:
+        * css/CSSCursorImageValue.cpp:
+        (WebCore::CSSCursorImageValue::CSSCursorImageValue):
+        * css/CSSCursorImageValue.h:
+        * css/CSSGrammar.y:
+        * css/CSSImageValue.cpp:
+        * css/CSSImageValue.h:
+        * css/CSSInheritedValue.cpp:
+        * css/CSSInheritedValue.h:
+        * css/CSSInitialValue.cpp:
+        * css/CSSInitialValue.h:
+        * css/CSSMutableStyleDeclaration.cpp:
+        (WebCore::CSSMutableStyleDeclaration::CSSMutableStyleDeclaration):
+        (WebCore::CSSMutableStyleDeclaration::operator=):
+        (WebCore::CSSMutableStyleDeclaration::getPropertyValue):
+        (WebCore::CSSMutableStyleDeclaration::get4Values):
+        (WebCore::CSSMutableStyleDeclaration::getShorthandValue):
+        (WebCore::CSSMutableStyleDeclaration::getPropertyCSSValue):
+        (WebCore::CSSMutableStyleDeclaration::removeProperty):
+        (WebCore::CSSMutableStyleDeclaration::setChanged):
+        (WebCore::CSSMutableStyleDeclaration::getPropertyPriority):
+        (WebCore::CSSMutableStyleDeclaration::getPropertyShorthand):
+        (WebCore::CSSMutableStyleDeclaration::isPropertyImplicit):
+        (WebCore::CSSMutableStyleDeclaration::setProperty):
+        (WebCore::CSSMutableStyleDeclaration::setImageProperty):
+        (WebCore::CSSMutableStyleDeclaration::parseDeclaration):
+        (WebCore::CSSMutableStyleDeclaration::addParsedProperties):
+        (WebCore::CSSMutableStyleDeclaration::setLengthProperty):
+        (WebCore::CSSMutableStyleDeclaration::cssText):
+        (WebCore::CSSMutableStyleDeclaration::merge):
+        (WebCore::CSSMutableStyleDeclaration::removePropertiesInSet):
+        * css/CSSMutableStyleDeclaration.h:
+        (WebCore::CSSMutableStyleDeclaration::setProperty):
+        (WebCore::CSSMutableStyleDeclaration::removeProperty):
+        * css/CSSNamespace.h:
+        (WebCore::CSSNamespace::CSSNamespace):
+        * css/CSSPrimitiveValue.cpp:
+        (WebCore::isCSSTokenizerIdentifier):
+        (WebCore::CSSPrimitiveValue::CSSPrimitiveValue):
+        (WebCore::CSSPrimitiveValue::cleanup):
+        (WebCore::CSSPrimitiveValue::computeLengthInt):
+        (WebCore::CSSPrimitiveValue::computeLengthIntForLength):
+        (WebCore::CSSPrimitiveValue::computeLengthShort):
+        (WebCore::CSSPrimitiveValue::computeLengthFloat):
+        (WebCore::CSSPrimitiveValue::setFloatValue):
+        (WebCore::scaleFactorForConversion):
+        (WebCore::CSSPrimitiveValue::getFloatValue):
+        (WebCore::CSSPrimitiveValue::setStringValue):
+        (WebCore::CSSPrimitiveValue::getStringValue):
+        (WebCore::CSSPrimitiveValue::parseString):
+        (WebCore::CSSPrimitiveValue::getIdent):
+        (WebCore::CSSPrimitiveValue::cssText):
+        * css/CSSPrimitiveValue.h:
+        (WebCore::CSSPrimitiveValue::getCounterValue):
+        (WebCore::CSSPrimitiveValue::getRectValue):
+        (WebCore::CSSPrimitiveValue::getRGBColorValue):
+        (WebCore::CSSPrimitiveValue::getPairValue):
+        (WebCore::CSSPrimitiveValue::getDashboardRegionValue):
+        * css/CSSProperty.cpp:
+        (WebCore::operator==):
+        * css/CSSProperty.h:
+        (WebCore::CSSProperty::operator=):
+        * css/CSSQuirkPrimitiveValue.h:
+        (WebCore::CSSQuirkPrimitiveValue::CSSQuirkPrimitiveValue):
+        * css/CSSRuleList.cpp:
+        (WebCore::CSSRuleList::CSSRuleList):
+        (WebCore::CSSRuleList::deleteRule):
+        (WebCore::CSSRuleList::insertRule):
+        * css/CSSRuleList.h:
+        (WebCore::CSSRuleList::item):
+        * css/CSSSelector.cpp:
+        (WebCore::CSSSelector::print):
+        (WebCore::CSSSelector::specificity):
+        (WebCore::CSSSelector::extractPseudoType):
+        (WebCore::CSSSelector::operator==):
+        (WebCore::CSSSelector::selectorText):
+        * css/CSSSelector.h:
+        (WebCore::CSSSelector::CSSSelector):
+        (WebCore::CSSSelector::~CSSSelector):
+        (WebCore::CSSSelector::append):
+        (WebCore::CSSSelector::next):
+        (WebCore::CSSSelector::pseudoType):
+        (WebCore::CSSSelector::hasTag):
+        (WebCore::CSSSelector::hasAttribute):
+        * css/CSSStyleDeclaration.cpp:
+        (WebCore::propertyID):
+        (WebCore::CSSStyleDeclaration::getPropertyPriority):
+        (WebCore::CSSStyleDeclaration::getPropertyShorthand):
+        (WebCore::CSSStyleDeclaration::isPropertyImplicit):
+        (WebCore::CSSStyleDeclaration::setProperty):
+        (WebCore::CSSStyleDeclaration::isPropertyName):
+        (WebCore::CSSStyleDeclaration::parentRule):
+        (WebCore::CSSStyleDeclaration::diff):
+        (WebCore::CSSStyleDeclaration::copyPropertiesInSet):
+        * css/CSSStyleDeclaration.h:
+        * css/Counter.h:
+        (WebCore::Counter::Counter):
+        (WebCore::Counter::listStyleNumber):
+        * css/csshelper.cpp:
+        (WebCore::parseURL):
+        * css/csshelper.h:
+        * css/cssparser.h:
+        (WebCore::domString):
+        (WebCore::atomicString):
+        (WebCore::CSSParser::operator|):
+        * css/cssstyleselector.cpp:
+        (WebCore::CSSStyleSelector::matchRulesForList):
+        (WebCore::CSSStyleSelector::checkSelector):
+        (WebCore::CSSStyleSelector::checkOneSelector):
+        (WebCore::CSSRuleSet::addRule):
+
 2007-01-14  Mitz Pettel  <mitz@webkit.org>
 
         Reviewed by Adam.

third_party/WebKit/WebCore/css/CSSBorderImageValue.cpp

@@ -19,6 +19,7 @@
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
+
 #include "config.h"
 #include "CSSBorderImageValue.h"
 
@@ -28,8 +29,7 @@
 
 namespace WebCore {
 
-CSSBorderImageValue::CSSBorderImageValue(PassRefPtr<CSSImageValue> image,
-    PassRefPtr<RectImpl> imageRect, int horizontalRule, int verticalRule)
+CSSBorderImageValue::CSSBorderImageValue(PassRefPtr<CSSImageValue> image, PassRefPtr<RectImpl> imageRect, int horizontalRule, int verticalRule)
     : m_image(image)
     , m_imageSliceRect(imageRect)
     , m_horizontalSizeRule(horizontalRule)
@@ -42,7 +42,7 @@ String CSSBorderImageValue::cssText() const
     // Image first.
     String text(m_image->cssText());
     text += " ";
-    
+
     // Now the rect, but it isn't really a rect, so we dump manually
     text += m_imageSliceRect->top()->cssText();
     text += " ";
@@ -51,7 +51,7 @@ String CSSBorderImageValue::cssText() const
     text += m_imageSliceRect->bottom()->cssText();
     text += " ";
     text += m_imageSliceRect->left()->cssText();
-    
+
     // Now the keywords.
     text += " ";
     text += CSSPrimitiveValue(m_horizontalSizeRule).cssText();
@@ -61,4 +61,4 @@ String CSSBorderImageValue::cssText() const
     return text;
 }
 
-}
+} // namespace WebCore

third_party/WebKit/WebCore/css/CSSBorderImageValue.h

@@ -20,8 +20,8 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSBorderImageValue_H
-#define CSSBorderImageValue_H
+#ifndef CSSBorderImageValue_h
+#define CSSBorderImageValue_h
 
 #include "CSSValue.h"
 #include <wtf/PassRefPtr.h>
@@ -32,8 +32,7 @@ namespace WebCore {
 class CSSImageValue;
 class RectImpl;
 
-class CSSBorderImageValue : public CSSValue
-{
+class CSSBorderImageValue : public CSSValue {
 public:
     CSSBorderImageValue();
     CSSBorderImageValue(PassRefPtr<CSSImageValue>, PassRefPtr<RectImpl>, int horizontalRule, int verticalRule);
@@ -47,12 +46,12 @@ public:
     // These four values are used to make "cuts" in the image.  They can be numbers
     // or percentages.
     RefPtr<RectImpl> m_imageSliceRect;
-    
+
     // Values for how to handle the scaling/stretching/tiling of the image slices.
     int m_horizontalSizeRule; // Rule for how to adjust the widths of the top/middle/bottom
     int m_verticalSizeRule; // Rule for how to adjust the heights of the left/middle/right
 };
 
-} // namespace
+} // namespace WebCore
 
-#endif
+#endif // CSSBorderImageValue_h

third_party/WebKit/WebCore/css/CSSComputedStyleDeclaration.h

 /**
- * CSSComputedStyleDeclaration.h
  *
  * Copyright (C)  2004  Zack Rusin <zack@kde.org>
  * Copyright (C) 2004, 2005, 2006 Apple Computer, Inc.
@@ -20,8 +19,8 @@
  * 02111-1307  USA
  */
 
-#ifndef CSS_COMPUTEDSTYLE_H
-#define CSS_COMPUTEDSTYLE_H
+#ifndef CSSComputedStyleDeclaration_h
+#define CSSComputedStyleDeclaration_h
 
 #include "CSSStyleDeclaration.h"
 #include "Node.h"
@@ -62,12 +61,13 @@ public:
 
 private:
     virtual void setCssText(const String&, ExceptionCode&);
+
     virtual String removeProperty(int propertyID, ExceptionCode&);
     virtual void setProperty(int propertyId, const String& value, bool important, ExceptionCode&);
 
     RefPtr<Node> m_node;
 };
 
-}
+} // namespace WebCore
 
-#endif
+#endif // CSSComputedStyleDeclaration_h

third_party/WebKit/WebCore/css/CSSCursorImageValue.cpp

@@ -18,18 +18,20 @@
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
+
 #include "config.h"
 #include "CSSCursorImageValue.h"
 
 namespace WebCore {
 
-CSSCursorImageValue::CSSCursorImageValue(const String& url, const IntPoint& p, StyleBase* style)
+CSSCursorImageValue::CSSCursorImageValue(const String& url, const IntPoint& hotspot, StyleBase* style)
     : CSSImageValue(url, style)
-    , m_hotspot(p)
+    , m_hotspot(hotspot)
 {
 }
 
 CSSCursorImageValue::~CSSCursorImageValue()
 {
 }
-}
+
+} // namespace WebCore

third_party/WebKit/WebCore/css/CSSCursorImageValue.h

@@ -19,18 +19,17 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSCursorImageValue_H
-#define CSSCursorImageValue_H
+#ifndef CSSCursorImageValue_h
+#define CSSCursorImageValue_h
 
-#include "IntPoint.h"
 #include "CSSImageValue.h"
+#include "IntPoint.h"
 
 namespace WebCore {
 
-class CSSCursorImageValue : public CSSImageValue
-{
+class CSSCursorImageValue : public CSSImageValue {
 public:
-    CSSCursorImageValue(const String& url, const IntPoint&, StyleBase*);
+    CSSCursorImageValue(const String& url, const IntPoint& hotspot, StyleBase*);
     virtual ~CSSCursorImageValue();
 
     IntPoint hotspot() const { return m_hotspot; }
@@ -38,6 +37,7 @@ public:
 protected:
     IntPoint m_hotspot;
 };
-} // namespace
 
-#endif
+} // namespace WebCore
+
+#endif // CSSCursorImageValue_h

third_party/WebKit/WebCore/css/CSSImageValue.cpp

@@ -19,12 +19,13 @@
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
+
 #include "config.h"
 #include "CSSImageValue.h"
 
+#include "CSSValueKeywords.h"
 #include "Cache.h"
 #include "CachedImage.h"
-#include "CSSValueKeywords.h"
 #include "DocLoader.h"
 
 namespace WebCore {
@@ -67,4 +68,4 @@ CachedImage* CSSImageValue::image(DocLoader* loader)
     return m_image;
 }
 
-}
+} // namespace WebCore

third_party/WebKit/WebCore/css/CSSImageValue.h

@@ -20,8 +20,8 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSImageValue_H
-#define CSSImageValue_H
+#ifndef CSSImageValue_h
+#define CSSImageValue_h
 
 #include "CSSPrimitiveValue.h"
 #include "CachedResourceClient.h"
@@ -30,8 +30,7 @@ namespace WebCore {
 
 class DocLoader;
 
-class CSSImageValue : public CSSPrimitiveValue, public CachedResourceClient
-{
+class CSSImageValue : public CSSPrimitiveValue, public CachedResourceClient {
 public:
     CSSImageValue();
     CSSImageValue(const String& url, StyleBase*);
@@ -44,6 +43,6 @@ protected:
     bool m_accessedImage;
 };
 
-} // namespace
+} // namespace WebCore
 
-#endif
+#endif // CSSImageValue_h

third_party/WebKit/WebCore/css/CSSInheritedValue.cpp

@@ -19,6 +19,7 @@
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
+
 #include "config.h"
 #include "CSSInheritedValue.h"
 
@@ -36,4 +37,4 @@ String CSSInheritedValue::cssText() const
     return "inherit";
 }
 
-}
+} // namespace WebCore

third_party/WebKit/WebCore/css/CSSInheritedValue.h

@@ -20,20 +20,19 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSInheritedValue_H
-#define CSSInheritedValue_H
+#ifndef CSSInheritedValue_h
+#define CSSInheritedValue_h
 
 #include "CSSValue.h"
 
 namespace WebCore {
 
-class CSSInheritedValue : public CSSValue
-{
+class CSSInheritedValue : public CSSValue {
 public:
     virtual unsigned short cssValueType() const;
     virtual String cssText() const;
 };
 
-} // namespace
+} // namespace WebCore
 
-#endif
+#endif // CSSInheritedValue_h

third_party/WebKit/WebCore/css/CSSInitialValue.cpp

@@ -19,6 +19,7 @@
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
+
 #include "config.h"
 #include "CSSInitialValue.h"
 
@@ -36,4 +37,4 @@ String CSSInitialValue::cssText() const
     return "initial";
 }
 
-}
+} // namespace WebCore

third_party/WebKit/WebCore/css/CSSInitialValue.h

@@ -20,20 +20,19 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSInitialValue_H
-#define CSSInitialValue_H
+#ifndef CSSInitialValue_h
+#define CSSInitialValue_h
 
 #include "CSSValue.h"
 
 namespace WebCore {
 
-class CSSInitialValue : public CSSValue
-{
+class CSSInitialValue : public CSSValue {
 public:
     virtual unsigned short cssValueType() const;
     virtual String cssText() const;
 };
 
-} // namespace
+} // namespace WebCore
 
-#endif
+#endif // CSSInitialValue_h

third_party/WebKit/WebCore/css/CSSMutableStyleDeclaration.h

@@ -20,11 +20,11 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSMutableStyleDeclaration_H
-#define CSSMutableStyleDeclaration_H
+#ifndef CSSMutableStyleDeclaration_h
+#define CSSMutableStyleDeclaration_h
 
 #include "CSSStyleDeclaration.h"
-#include "CSSPrimitiveValue.h" // for CSSPrimitiveValue::UnitTypes
+#include "CSSPrimitiveValue.h"
 #include "DeprecatedValueList.h"
 #include "PlatformString.h"
 
@@ -33,15 +33,14 @@ namespace WebCore {
 class CSSProperty;
 class Node;
 
-class CSSMutableStyleDeclaration : public CSSStyleDeclaration
-{
+class CSSMutableStyleDeclaration : public CSSStyleDeclaration {
 public:
     CSSMutableStyleDeclaration();
     CSSMutableStyleDeclaration(CSSRule* parentRule);
     CSSMutableStyleDeclaration(CSSRule* parentRule, const DeprecatedValueList<CSSProperty>&);
-    CSSMutableStyleDeclaration(CSSRule* parentRule, const CSSProperty * const *, int numProperties);
+    CSSMutableStyleDeclaration(CSSRule* parentRule, const CSSProperty* const *, int numProperties);
 
-    CSSMutableStyleDeclaration &operator=(const CSSMutableStyleDeclaration &);
+    CSSMutableStyleDeclaration& operator=(const CSSMutableStyleDeclaration&);
 
     void setNode(Node* node) { m_node = node; }
 
@@ -68,11 +67,17 @@ public:
     bool setProperty(int propertyID, int value, bool important = false, bool notifyChanged = true);
     bool setProperty(int propertyID, const String& value, bool important, bool notifyChanged, ExceptionCode&);
     bool setProperty(int propertyId, const String& value, bool important = false, bool notifyChanged = true)
-        { ExceptionCode ec; return setProperty(propertyId, value, important, notifyChanged, ec); }
+    { 
+        ExceptionCode ec;
+        return setProperty(propertyId, value, important, notifyChanged, ec);
+    }
 
     String removeProperty(int propertyID, bool notifyChanged, ExceptionCode&);
     String removeProperty(int propertyID, bool notifyChanged = true)
-        { ExceptionCode ec; return removeProperty(propertyID, notifyChanged, ec); }
+    {
+        ExceptionCode ec;
+        return removeProperty(propertyID, notifyChanged, ec);
+    }
 
     void clear();
 
@@ -81,29 +86,29 @@ public:
     // setLengthProperty treats integers as pixels! (Needed for conversion of HTML attributes.)
     void setLengthProperty(int propertyId, const String& value, bool important, bool multiLength = false);
     void setStringProperty(int propertyId, const String& value, CSSPrimitiveValue::UnitTypes, bool important = false); // parsed string value
-    void setImageProperty(int propertyId, const String& URL, bool important = false);
+    void setImageProperty(int propertyId, const String& url, bool important = false);
  
     // The following parses an entire new style declaration.
     void parseDeclaration(const String& styleDeclaration);
 
     // Besides adding the properties, this also removes any existing properties with these IDs.
     // It does no notification since it's called by the parser.
-    void addParsedProperties(const CSSProperty * const *, int numProperties);
+    void addParsedProperties(const CSSProperty* const *, int numProperties);
  
     PassRefPtr<CSSMutableStyleDeclaration> copyBlockProperties() const;
     void removeBlockProperties();
     void removePropertiesInSet(const int* set, unsigned length, bool notifyChanged = true);
 
     void merge(CSSMutableStyleDeclaration*, bool argOverridesOnConflict = true);
- 
+
 private:
-    String getShortHandValue(const int* properties, int number) const;
+    String getShorthandValue(const int* properties, int number) const;
     String get4Values(const int* properties) const;
  
     DeprecatedValueList<CSSProperty> m_values;
     Node* m_node;
 };
 
-} // namespace
+} // namespace WebCore
 
-#endif
+#endif // CSSMutableStyleDeclaration_h

third_party/WebKit/WebCore/css/CSSNamespace.h

@@ -21,8 +21,8 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSNamespace_H
-#define CSSNamespace_H
+#ifndef CSSNamespace_h
+#define CSSNamespace_h
 
 #include "AtomicString.h"
 
@@ -33,15 +33,17 @@ namespace WebCore {
         AtomicString m_uri;
         CSSNamespace* m_parent;
 
-        CSSNamespace(const AtomicString& p, const AtomicString& u, CSSNamespace* parent) 
-            : m_prefix(p)
-            , m_uri(u)
-            , m_parent(parent) {}
+        CSSNamespace(const AtomicString& prefix, const AtomicString& uri, CSSNamespace* parent)
+            : m_prefix(prefix)
+            , m_uri(uri)
+            , m_parent(parent)
+        {
+        }
         ~CSSNamespace() { delete m_parent; }
-        
+
         const AtomicString& uri() { return m_uri; }
         const AtomicString& prefix() { return m_prefix; }
-        
+
         CSSNamespace* namespaceForPrefix(const AtomicString& prefix)
         {
             if (prefix == m_prefix)
@@ -51,6 +53,7 @@ namespace WebCore {
             return 0;
         }
     };
-}
 
-#endif
+} // namespace WebCore
+
+#endif // CSSNamespace_h

third_party/WebKit/WebCore/css/CSSPrimitiveValue.h

@@ -20,8 +20,8 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSPrimitiveValue_H
-#define CSSPrimitiveValue_H
+#ifndef CSSPrimitiveValue_h
+#define CSSPrimitiveValue_h
 
 #include "CSSValue.h"
 #include <wtf/PassRefPtr.h>
@@ -37,8 +37,7 @@ class StringImpl;
 
 typedef int ExceptionCode;
 
-class CSSPrimitiveValue : public CSSValue
-{
+class CSSPrimitiveValue : public CSSValue {
 public:
     enum UnitTypes {
         CSS_UNKNOWN = 0,
@@ -117,30 +116,18 @@ public:
 
     void setStringValue(unsigned short stringType, const String& stringValue, ExceptionCode&);
     String getStringValue() const;
-    
-    Counter* getCounterValue () const {
-        return m_type != CSS_COUNTER ? 0 : m_value.counter;
-    }
 
-    RectImpl* getRectValue () const {
-        return m_type != CSS_RECT ? 0 : m_value.rect;
-    }
-
-    unsigned getRGBColorValue() const {
-        return m_type != CSS_RGBCOLOR ? 0 : m_value.rgbcolor;
-    }
-
-    Pair* getPairValue() const {
-        return m_type != CSS_PAIR ? 0 : m_value.pair;
-    }
+    Counter* getCounterValue () const { return m_type != CSS_COUNTER ? 0 : m_value.counter; }
+    RectImpl* getRectValue () const { return m_type != CSS_RECT ? 0 : m_value.rect; }
+    unsigned getRGBColorValue() const { return m_type != CSS_RGBCOLOR ? 0 : m_value.rgbcolor; }
+    Pair* getPairValue() const { return m_type != CSS_PAIR ? 0 : m_value.pair; }
 
 #if PLATFORM(MAC)
-    DashboardRegion *getDashboardRegionValue () const {
-        return m_type != CSS_DASHBOARD_REGION ? 0 : m_value.region;
-    }
+    DashboardRegion* getDashboardRegionValue () const { return m_type != CSS_DASHBOARD_REGION ? 0 : m_value.region; }
 #endif
 
     virtual bool isPrimitiveValue() const { return true; }
+
     virtual unsigned short cssValueType() const;
 
     int getIdent();
@@ -166,6 +153,6 @@ protected:
     } m_value;
 };
 
-} // namespace
+} // namespace WebCore
 
-#endif
+#endif // CSSPrimitiveValue_h

third_party/WebKit/WebCore/css/CSSProperty.cpp

@@ -19,8 +19,10 @@
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
+
 #include "config.h"
 #include "CSSProperty.h"
+
 #include "PlatformString.h"
 
 // Not in any header, so just declare it here for now.
@@ -33,9 +35,9 @@ String CSSProperty::cssText() const
     return getPropertyName(id()) + ": " + m_value->cssText() + (isImportant() ? " !important" : "") + "; ";
 }
 
-bool operator==(const CSSProperty &a, const CSSProperty &b)
+bool operator==(const CSSProperty& a, const CSSProperty& b)
 {
     return a.m_id == b.m_id && a.m_important == b.m_important && a.m_value == b.m_value;
 }
 
-}
+} // namespace WebCore

third_party/WebKit/WebCore/css/CSSProperty.h

@@ -20,8 +20,8 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSProperty_H
-#define CSSProperty_H
+#ifndef CSSProperty_h
+#define CSSProperty_h
 
 #include "CSSValue.h"
 #include <wtf/PassRefPtr.h>
@@ -39,19 +39,19 @@ public:
         , m_value(value)
     {
     }
-    
-    CSSProperty &operator=(const CSSProperty& o)
+
+    CSSProperty& operator=(const CSSProperty& other)
     {
-        m_id = o.m_id;
-        m_shorthandID = o.m_shorthandID;
-        m_important = o.m_important;
-        m_value = o.m_value;
+        m_id = other.m_id;
+        m_shorthandID = other.m_shorthandID;
+        m_important = other.m_important;
+        m_value = other.m_value;
         return *this;
     }
-    
+
     int id() const { return m_id; }
     int shorthandID() const { return m_shorthandID; }
-    
+
     bool isImportant() const { return m_important; }
     bool isImplicit() const { return m_implicit; }
 
@@ -59,17 +59,17 @@ public:
     
     String cssText() const;
 
+    friend bool operator==(const CSSProperty&, const CSSProperty&);
+
     // make sure the following fits in 4 bytes.
     int m_id;
     int m_shorthandID;  // If this property was set as part of a shorthand, gives the shorthand.
     bool m_important : 1;
     bool m_implicit : 1; // Whether or not the property was set implicitly as the result of a shorthand.
 
-    friend bool operator==(const CSSProperty &, const CSSProperty &);
-
     RefPtr<CSSValue> m_value;
 };
 
-} // namespace
+} // namespace WebCore
 
-#endif
+#endif // CSSProperty_h

third_party/WebKit/WebCore/css/CSSQuirkPrimitiveValue.h

@@ -20,8 +20,8 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSQuirkPrimitiveValue_H
-#define CSSQuirkPrimitiveValue_H
+#ifndef CSSQuirkPrimitiveValue_h
+#define CSSQuirkPrimitiveValue_h
 
 #include "CSSPrimitiveValue.h"
 
@@ -31,15 +31,16 @@ namespace WebCore {
 // The basic idea is that a stylesheet can use the value __qem (for quirky em) instead of em
 // in a stylesheet.  When the quirky value is used, if you're in quirks mode, the margin will
 // collapse away inside a table cell.
-class CSSQuirkPrimitiveValue : public CSSPrimitiveValue
-{
+class CSSQuirkPrimitiveValue : public CSSPrimitiveValue {
 public:
     CSSQuirkPrimitiveValue(double num, UnitTypes type)
-        : CSSPrimitiveValue(num, type) {}
+        : CSSPrimitiveValue(num, type)
+    {
+    }
 
     virtual bool isQuirkValue() { return true; }
 };
 
-} // namespace
+} // namespace WebCore
 
-#endif
+#endif // CSSQuirkPrimitiveValue_h

third_party/WebKit/WebCore/css/CSSRuleList.cpp

@@ -20,6 +20,7 @@
  * the Free Software Foundation, Inc., 59 Temple Place - Suite 330,
  * Boston, MA 02111-1307, USA.
  */
+
 #include "config.h"
 #include "CSSRuleList.h"
 
@@ -32,12 +33,12 @@ CSSRuleList::CSSRuleList()
 {
 }
 
-CSSRuleList::CSSRuleList(StyleList* lst, bool omitCharsetRules)
+CSSRuleList::CSSRuleList(StyleList* list, bool omitCharsetRules)
 {
-    if (lst) {
-        unsigned len = lst->length();
+    if (list) {
+        unsigned len = list->length();
         for (unsigned i = 0; i < len; ++i) {
-            StyleBase* style = lst->item(i);
+            StyleBase* style = list->item(i);
             if (style->isRule() && !(omitCharsetRules && style->isCharsetRule()))
                 append(static_cast<CSSRule*>(style));
         }
@@ -57,7 +58,7 @@ void CSSRuleList::deleteRule(unsigned index)
     if (rule)
         rule->deref();
     else
-        ; // ### Throw INDEX_SIZE_ERR exception here (TODO)
+        ; // FIXME: Throw INDEX_SIZE_ERR exception here
 }
 
 void CSSRuleList::append(CSSRule* rule)
@@ -72,8 +73,8 @@ unsigned CSSRuleList::insertRule(CSSRule* rule, unsigned index)
         return index;
     }
 
-    // ### Should throw INDEX_SIZE_ERR exception instead! (TODO)
+    // FIXME: Should throw INDEX_SIZE_ERR exception instead!
     return 0;
 }
 
-}
+} // namespace WebCore

third_party/WebKit/WebCore/css/CSSRuleList.h

@@ -21,36 +21,35 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSRuleList_H
-#define CSSRuleList_H
+#ifndef CSSRuleList_h
+#define CSSRuleList_h
 
-#include "Shared.h"
 #include "DeprecatedPtrList.h"
+#include "Shared.h"
 
 namespace WebCore {
 
 class CSSRule;
 class StyleList;
 
-class CSSRuleList : public Shared<CSSRuleList>
-{
+class CSSRuleList : public Shared<CSSRuleList> {
 public:
     CSSRuleList();
     CSSRuleList(StyleList*, bool omitCharsetRules = false);
     ~CSSRuleList();
 
     unsigned length() const { return m_lstCSSRules.count(); }
-    CSSRule* item (unsigned index) { return m_lstCSSRules.at(index); }
+    CSSRule* item(unsigned index) { return m_lstCSSRules.at(index); }
 
     /* not part of the DOM */
-    unsigned insertRule (CSSRule* rule, unsigned index);
-    void deleteRule (unsigned index);
-    void append(CSSRule* rule);
+    unsigned insertRule(CSSRule*, unsigned index);
+    void deleteRule(unsigned index);
+    void append(CSSRule*);
 
 protected:
     DeprecatedPtrList<CSSRule> m_lstCSSRules;
 };
 
-} // namespace
+} // namespace WebCore
 
-#endif
+#endif // CSSRuleList_h

third_party/WebKit/WebCore/css/CSSSelector.h

@@ -21,56 +21,58 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSSelector_H
-#define CSSSelector_H
+#ifndef CSSSelector_h
+#define CSSSelector_h
 
 #include "QualifiedName.h"
 
 namespace WebCore {
 
-// this class represents a selector for a StyleRule
-    class CSSSelector
-    {
+    // this class represents a selector for a StyleRule
+    class CSSSelector {
     public:
         CSSSelector()
-            : tagHistory(0)
-            , simpleSelector(0)
-            , nextSelector(0)
-            , argument(nullAtom)
-            , attr(anyQName())
-            , tag(anyQName())
+            : m_tagHistory(0)
+            , m_simpleSelector(0)
+            , m_nextSelector(0)
+            , m_argument(nullAtom)
+            , m_attr(anyQName())
+            , m_tag(anyQName())
             , m_relation(Descendant)
-            , match(None)
-            , pseudoId(0)
-            , _pseudoType(PseudoNotParsed)
-        {}
-        
+            , m_match(None)
+            , m_pseudoType(PseudoNotParsed)
+        {
+        }
+
         CSSSelector(const QualifiedName& qName)
-            : tagHistory(0)
-            , simpleSelector(0)
-            , nextSelector(0)
-            , argument(nullAtom)
-            , attr(anyQName())
-            , tag(qName)
+            : m_tagHistory(0)
+            , m_simpleSelector(0)
+            , m_nextSelector(0)
+            , m_argument(nullAtom)
+            , m_attr(anyQName())
+            , m_tag(qName)
             , m_relation(Descendant)
-            , match(None)
-            , pseudoId(0)
-            , _pseudoType(PseudoNotParsed)
-        {}
-
-        ~CSSSelector() {
-            delete tagHistory;
-            delete simpleSelector;
-            delete nextSelector;
+            , m_match(None)
+            , m_pseudoType(PseudoNotParsed)
+        {
+        }
+
+        ~CSSSelector()
+        {
+            delete m_tagHistory;
+            delete m_simpleSelector;
+            delete m_nextSelector;
         }
 
-        void append(CSSSelector* n) {
+        void append(CSSSelector* n)
+        {
             CSSSelector* end = this;
-            while (end->nextSelector)
-                end = end->nextSelector;
-            end->nextSelector = n;
+            while (end->m_nextSelector)
+                end = end->m_nextSelector;
+            end->m_nextSelector = n;
         }
-        CSSSelector* next() { return nextSelector; }
+
+        CSSSelector* next() { return m_nextSelector; }
 
         /**
          * Print debug output for this selector
@@ -83,7 +85,7 @@ namespace WebCore {
         String selectorText() const;
 
         // checks if the 2 selectors (including sub selectors) agree.
-        bool operator ==(const CSSSelector&);
+        bool operator==(const CSSSelector&);
 
         // tag == -1 means apply to all elements (Selector = *)
 
@@ -154,33 +156,33 @@ namespace WebCore {
 
         PseudoType pseudoType() const
         {
-            if (_pseudoType == PseudoNotParsed)
+            if (m_pseudoType == PseudoNotParsed)
                 extractPseudoType();
-            return static_cast<PseudoType>(_pseudoType);
+            return static_cast<PseudoType>(m_pseudoType);
         }
 
-        bool hasTag() const { return tag != anyQName(); }
-        bool hasAttribute() const { return attr != anyQName(); }
-
-        mutable AtomicString value;
-        CSSSelector* tagHistory;
-        CSSSelector* simpleSelector; // Used for :not.
-        CSSSelector* nextSelector; // used for ,-chained selectors
-        AtomicString argument; // Used for :contains, :lang and :nth-*
-        
-        QualifiedName attr;
-        QualifiedName tag;
-        
+        bool hasTag() const { return m_tag != anyQName(); }
+        bool hasAttribute() const { return m_attr != anyQName(); }
+
         Relation relation() const { return static_cast<Relation>(m_relation); }
 
-        unsigned m_relation          : 3; // enum Relation
-        mutable unsigned match       : 4; // enum Match
-        unsigned pseudoId            : 4;
-        mutable unsigned _pseudoType : 6; // PseudoType
+        mutable AtomicString m_value;
+        CSSSelector* m_tagHistory;
+        CSSSelector* m_simpleSelector; // Used for :not.
+        CSSSelector* m_nextSelector; // used for ,-chained selectors
+        AtomicString m_argument; // Used for :contains, :lang and :nth-*
+
+        QualifiedName m_attr;
+        QualifiedName m_tag;
+
+        unsigned m_relation           : 3; // enum Relation
+        mutable unsigned m_match      : 4; // enum Match
+        mutable unsigned m_pseudoType : 6; // PseudoType
 
     private:
         void extractPseudoType() const;
     };
-}
 
-#endif
+} // namespace WebCore
+
+#endif // CSSSelector_h

third_party/WebKit/WebCore/css/CSSStyleDeclaration.h

@@ -20,8 +20,8 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef CSSStyleDeclaration_H
-#define CSSStyleDeclaration_H
+#ifndef CSSStyleDeclaration_h
+#define CSSStyleDeclaration_h
 
 #include "StyleBase.h"
 #include <wtf/PassRefPtr.h>
@@ -81,6 +81,6 @@ private:
     CSSStyleDeclaration& operator=(const CSSStyleDeclaration&);
 };
 
-} // namespace
+} // namespace WebCore
 
-#endif
+#endif // CSSStyleDeclaration_h

third_party/WebKit/WebCore/css/Counter.h

@@ -20,8 +20,8 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef Counter_H
-#define Counter_H
+#ifndef Counter_h
+#define Counter_h
 
 #include "CSSPrimitiveValue.h"
 #include "PlatformString.h"
@@ -32,26 +32,29 @@ namespace WebCore {
 
 class Counter : public Shared<Counter> {
 public:
-    Counter(PassRefPtr<CSSPrimitiveValue> identifier, PassRefPtr<CSSPrimitiveValue> listStyle,
-        PassRefPtr<CSSPrimitiveValue> separator)
-        : m_identifier(identifier), m_listStyle(listStyle), m_separator(separator) { }
+    Counter(PassRefPtr<CSSPrimitiveValue> identifier, PassRefPtr<CSSPrimitiveValue> listStyle, PassRefPtr<CSSPrimitiveValue> separator)
+        : m_identifier(identifier)
+        , m_listStyle(listStyle)
+        , m_separator(separator)
+    {
+    }
 
     String identifier() const { return m_identifier ? m_identifier->getStringValue() : String(); }
     String listStyle() const { return m_listStyle ? m_listStyle->getStringValue() : String(); }
     String separator() const { return m_separator ? m_separator->getStringValue() : String(); }
 
-    int listStyleNumber() const { return m_listStyle ? (int) m_listStyle->getFloatValue() : 0; }
-    
+    int listStyleNumber() const { return m_listStyle ? static_cast<int>(m_listStyle->getFloatValue()) : 0; }
+
     void setIdentifier(PassRefPtr<CSSPrimitiveValue> identifier) { m_identifier = identifier; }
     void setListStyle(PassRefPtr<CSSPrimitiveValue> listStyle) { m_listStyle = listStyle; }
     void setSeparator(PassRefPtr<CSSPrimitiveValue> separator) { m_separator = separator; }
- 
+
 protected:
     RefPtr<CSSPrimitiveValue> m_identifier; // String
     RefPtr<CSSPrimitiveValue> m_listStyle;  // int
     RefPtr<CSSPrimitiveValue> m_separator;  // String
- };
+};
 
-} //namespace
+} // namespace WebCore
 
-#endif
+#endif // Counter_h