Reviewed by Beth.
authordarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 25 Oct 2005 00:07:48 +0000 (00:07 +0000)
committerdarin <darin@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 25 Oct 2005 00:07:48 +0000 (00:07 +0000)
        - got ready to make m_parent private by removing references to it
          also removed tabs (and used spaces instead) to files I touched
        - fixed code that initialized m_parent twice

        * khtml/css/css_base.cpp:
        (StyleBaseImpl::checkLoaded): Use parent().
        (StyleBaseImpl::stylesheet): Ditto.
        * khtml/css/css_base.h:
        (DOM::StyleBaseImpl::StyleBaseImpl): Initialize base TreeShared with the appropriate
        parent instead of first setting m_parent to 0 and then to the correct value. Also
        removed setParent, since the base class setParent works fine.
        * khtml/css/css_ruleimpl.cpp:
        (DOM::CSSRuleImpl::parentStyleSheet): Use parent().
        (DOM::CSSRuleImpl::parentRule): Ditto.
        * khtml/css/css_stylesheetimpl.cpp:
        (StyleSheetImpl::parentStyleSheet): Ditto.
        (MediaListImpl::parentStyleSheet): Ditto.
        (MediaListImpl::parentRule): Ditto.
        * khtml/css/css_valueimpl.cpp:
        (DOM::CSSStyleDeclarationImpl::parentRule): Ditto.
        * khtml/rendering/render_replaced.cpp: Reformatting only, and removed some
        APPLE_CHANGES-only instead.
        * khtml/xml/dom_nodeimpl.cpp:
        (DOM::NodeImpl::isContentEditable): Use parent().
        * khtml/xml/dom_nodeimpl.h:
        (DOM::NodeImpl::parentNode): Ditto.
        * khtml/xsl/xsl_stylesheetimpl.cpp:
        (DOM::XSLImportRuleImpl::parentStyleSheet): Ditto.

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

WebCore/ChangeLog-2005-12-19
WebCore/khtml/css/css_base.cpp
WebCore/khtml/css/css_base.h
WebCore/khtml/css/css_ruleimpl.cpp
WebCore/khtml/css/css_stylesheetimpl.cpp
WebCore/khtml/css/css_valueimpl.cpp
WebCore/khtml/rendering/render_replaced.cpp
WebCore/khtml/xml/dom_nodeimpl.cpp
WebCore/khtml/xml/dom_nodeimpl.h
WebCore/khtml/xsl/xsl_stylesheetimpl.cpp

index e34ab71..1b8e796 100644 (file)
@@ -1,3 +1,36 @@
+2005-10-24  Darin Adler  <darin@apple.com>
+
+        Reviewed by Beth.
+
+        - got ready to make m_parent private by removing references to it
+          also removed tabs (and used spaces instead) to files I touched
+        - fixed code that initialized m_parent twice
+
+        * khtml/css/css_base.cpp:
+        (StyleBaseImpl::checkLoaded): Use parent().
+        (StyleBaseImpl::stylesheet): Ditto.
+        * khtml/css/css_base.h:
+        (DOM::StyleBaseImpl::StyleBaseImpl): Initialize base TreeShared with the appropriate
+        parent instead of first setting m_parent to 0 and then to the correct value. Also
+        removed setParent, since the base class setParent works fine.
+        * khtml/css/css_ruleimpl.cpp:
+        (DOM::CSSRuleImpl::parentStyleSheet): Use parent().
+        (DOM::CSSRuleImpl::parentRule): Ditto.
+        * khtml/css/css_stylesheetimpl.cpp:
+        (StyleSheetImpl::parentStyleSheet): Ditto.
+        (MediaListImpl::parentStyleSheet): Ditto.
+        (MediaListImpl::parentRule): Ditto.
+        * khtml/css/css_valueimpl.cpp:
+        (DOM::CSSStyleDeclarationImpl::parentRule): Ditto.
+        * khtml/rendering/render_replaced.cpp: Reformatting only, and removed some
+        APPLE_CHANGES-only instead.
+        * khtml/xml/dom_nodeimpl.cpp:
+        (DOM::NodeImpl::isContentEditable): Use parent().
+        * khtml/xml/dom_nodeimpl.h:
+        (DOM::NodeImpl::parentNode): Ditto.
+        * khtml/xsl/xsl_stylesheetimpl.cpp:
+        (DOM::XSLImportRuleImpl::parentStyleSheet): Ditto.
+
 2005-10-24  David Hyatt  <hyatt@apple.com>
 
        Fix for 5485, make sure generated content's "rightmost selector"
index ab5d72f..3bc8d96 100644 (file)
@@ -41,14 +41,14 @@ using namespace DOM;
 
 void StyleBaseImpl::checkLoaded()
 {
-    if(m_parent) m_parent->checkLoaded();
+    if (parent()) parent()->checkLoaded();
 }
 
 StyleSheetImpl* StyleBaseImpl::stylesheet()
 {
     StyleBaseImpl *b = this;
-    while(b && !b->isStyleSheet())
-        b = b->m_parent;
+    while (b && !b->isStyleSheet())
+        b = b->parent();
     return static_cast<StyleSheetImpl *>(b);
 }
 
index 20d0654..13cd2a4 100644 (file)
@@ -21,8 +21,8 @@
  * Boston, MA 02111-1307, USA.
  */
 
-#ifndef _CSS_BASE_H
-#define _CSS_BASE_H
+#ifndef CSS_BASE_H
+#define CSS_BASE_H
 
 #include "dom/dom_string.h"
 #include "dom/dom_misc.h"
@@ -116,7 +116,7 @@ namespace DOM {
         {
             None = 0,
             Id,
-        Class,
+            Class,
             Exact,
             Set,
             List,
@@ -200,10 +200,10 @@ namespace DOM {
     class StyleBaseImpl : public khtml::TreeShared<StyleBaseImpl>
     {
     public:
-        StyleBaseImpl()  { m_parent = 0; strictParsing = true; multiLength = false; }
-        StyleBaseImpl(StyleBaseImpl *p) {
-            m_parent = p;
-            strictParsing = (m_parent ? m_parent->useStrictParsing() : true);
+        typedef khtml::TreeShared<StyleBaseImpl> TreeShared;
+        StyleBaseImpl()  { strictParsing = true; multiLength = false; }
+        StyleBaseImpl(StyleBaseImpl *p) : TreeShared(p) {
+            strictParsing = (p ? p->useStrictParsing() : true);
             multiLength = false;
         }
 
@@ -232,8 +232,6 @@ namespace DOM {
         virtual bool isValueList() { return false; }
         virtual bool isValueCustom() { return false; }
 
-        void setParent(StyleBaseImpl *parent) { m_parent = parent; }
-
         virtual bool parseString(const DOMString &/*cssString*/, bool = false) { return false; }
 
         virtual void checkLoaded();
index 4d3efc9..94a163f 100644 (file)
 #include "misc/loader.h"
 #include "xml/dom_docimpl.h"
 
-using namespace DOM;
-
 #include <kdebug.h>
 
+namespace DOM {
+
 CSSStyleSheetImpl *CSSRuleImpl::parentStyleSheet() const
 {
-    return ( m_parent && m_parent->isCSSStyleSheet() )  ?
-       static_cast<CSSStyleSheetImpl *>(m_parent) : 0;
+    return (parent() && parent()->isCSSStyleSheet()) ? static_cast<CSSStyleSheetImpl *>(parent()) : 0;
 }
 
 CSSRuleImpl *CSSRuleImpl::parentRule() const
 {
-    return ( m_parent && m_parent->isRule() )  ?
-       static_cast<CSSRuleImpl *>(m_parent) : 0;
+    return (parent() && parent()->isRule()) ? static_cast<CSSRuleImpl *>(parent()) : 0;
 }
 
 DOM::DOMString CSSRuleImpl::cssText() const
@@ -86,7 +84,7 @@ CSSImportRuleImpl::CSSImportRuleImpl( StyleBaseImpl *parent,
 
     m_lstMedia = media;
     if ( !m_lstMedia )
-       m_lstMedia = new MediaListImpl( this, DOMString() );
+        m_lstMedia = new MediaListImpl( this, DOMString() );
     m_lstMedia->setParent( this );
     m_lstMedia->ref();
 
@@ -118,8 +116,8 @@ CSSImportRuleImpl::CSSImportRuleImpl( StyleBaseImpl *parent,
 CSSImportRuleImpl::~CSSImportRuleImpl()
 {
     if( m_lstMedia ) {
-       m_lstMedia->setParent( 0 );
-       m_lstMedia->deref();
+        m_lstMedia->setParent( 0 );
+        m_lstMedia->deref();
     }
     if(m_styleSheet) {
         m_styleSheet->setParent(0);
@@ -156,9 +154,9 @@ void CSSImportRuleImpl::init()
     StyleBaseImpl *root = this;
     StyleBaseImpl *parent;
     while ( ( parent = root->parent()) )
-       root = parent;
+        root = parent;
     if (root->isCSSStyleSheet())
-       docLoader = static_cast<CSSStyleSheetImpl*>(root)->docLoader();
+        docLoader = static_cast<CSSStyleSheetImpl*>(root)->docLoader();
 
     DOMString absHref = m_strHref;
     CSSStyleSheetImpl *parentSheet = parentStyleSheet();
@@ -246,13 +244,13 @@ CSSMediaRuleImpl::CSSMediaRuleImpl( StyleBaseImpl *parent, const DOM::DOMString
 CSSMediaRuleImpl::~CSSMediaRuleImpl()
 {
     if( m_lstMedia ) {
-       m_lstMedia->setParent( 0 );
+        m_lstMedia->setParent( 0 );
         m_lstMedia->deref();
     }
 
     int length = m_lstCSSRules->length();
     for (int i = 0; i < length; i++) {
-       m_lstCSSRules->item( i )->setParent( 0 );
+        m_lstCSSRules->item( i )->setParent( 0 );
     }
     m_lstCSSRules->deref();
 }
@@ -260,7 +258,7 @@ CSSMediaRuleImpl::~CSSMediaRuleImpl()
 unsigned CSSMediaRuleImpl::append( CSSRuleImpl *rule )
 {
     if (!rule) {
-       return 0;
+        return 0;
     }
 
     rule->setParent(this);
@@ -274,7 +272,7 @@ unsigned CSSMediaRuleImpl::insertRule( const DOMString &rule,
     CSSRuleImpl *newRule = p.parseRule( parentStyleSheet(), rule );
 
     if (!newRule) {
-       return 0;
+        return 0;
     }
 
     newRule->setParent(this);
@@ -362,8 +360,8 @@ CSSStyleRuleImpl::CSSStyleRuleImpl(StyleBaseImpl *parent)
 CSSStyleRuleImpl::~CSSStyleRuleImpl()
 {
     if(m_style) {
-       m_style->setParent( 0 );
-       m_style->deref();
+        m_style->setParent( 0 );
+        m_style->deref();
     }
     delete m_selector;
 }
@@ -435,3 +433,5 @@ unsigned CSSRuleListImpl::insertRule( CSSRuleImpl *rule,
     // ### Should throw INDEX_SIZE_ERR exception instead! (TODO)
     return 0;
 }
+
+}
index 89bc63f..ab27f12 100644 (file)
@@ -75,25 +75,23 @@ StyleSheetImpl::StyleSheetImpl(StyleBaseImpl *owner, DOMString href)
 
 StyleSheetImpl::~StyleSheetImpl()
 {
-    if(m_media) {
-       m_media->setParent( 0 );
-       m_media->deref();
+    if (m_media) {
+        m_media->setParent( 0 );
+        m_media->deref();
     }
 }
 
 StyleSheetImpl *StyleSheetImpl::parentStyleSheet() const
 {
-    if( !m_parent ) return 0;
-    if( m_parent->isStyleSheet() ) return static_cast<StyleSheetImpl *>(m_parent);
-    return 0;
+    return (parent() && parent()->isStyleSheet()) ? static_cast<StyleSheetImpl *>(parent()) : 0;
 }
 
 void StyleSheetImpl::setMedia( MediaListImpl *media )
 {
-    if( media )
-       media->ref();
-    if( m_media )
-       m_media->deref();
+    if (media)
+        media->ref();
+    if (m_media)
+        m_media->deref();
     m_media = media;
 }
 
@@ -160,22 +158,20 @@ CSSStyleSheetImpl::CSSStyleSheetImpl(CSSRuleImpl *ownerRule, CSSStyleSheetImpl *
 
 CSSRuleImpl *CSSStyleSheetImpl::ownerRule() const
 {
-    if( !m_parent ) return 0;
-    if( m_parent->isRule() ) return static_cast<CSSRuleImpl *>(m_parent);
-    return 0;
+    return (parent() && parent()->isRule()) ? static_cast<CSSRuleImpl *>(parent()) : 0;
 }
 
 unsigned CSSStyleSheetImpl::insertRule( const DOMString &rule, unsigned index, int &exceptioncode )
 {
     exceptioncode = 0;
-    if(index > m_lstChildren->count()) {
+    if (index > m_lstChildren->count()) {
         exceptioncode = DOMException::INDEX_SIZE_ERR;
         return 0;
     }
     CSSParser p( strictParsing );
     CSSRuleImpl *r = p.parseRule( this, rule );
 
-    if(!r) {
+    if (!r) {
         exceptioncode = CSSException::SYNTAX_ERR + CSSException::_EXCEPTION_OFFSET;
         return 0;
     }
@@ -202,7 +198,7 @@ void CSSStyleSheetImpl::deleteRule( unsigned index, int &exceptioncode )
 {
     exceptioncode = 0;
     StyleBaseImpl *b = m_lstChildren->take(index);
-    if(!b) {
+    if (!b) {
         exceptioncode = DOMException::INDEX_SIZE_ERR;
         return;
     }
@@ -253,13 +249,13 @@ bool CSSStyleSheetImpl::isLoading()
     StyleBaseImpl *rule;
     for ( rule = m_lstChildren->first(); rule != 0; rule = m_lstChildren->next() )
     {
-        if(rule->isImportRule())
+        if (rule->isImportRule())
         {
             CSSImportRuleImpl *import = static_cast<CSSImportRuleImpl *>(rule);
 #ifdef CSS_STYLESHEET_DEBUG
             kdDebug( 6080 ) << "found import" << endl;
 #endif
-            if(import->isLoading())
+            if (import->isLoading())
             {
 #ifdef CSS_STYLESHEET_DEBUG
                 kdDebug( 6080 ) << "--> not loaded" << endl;
@@ -273,9 +269,9 @@ bool CSSStyleSheetImpl::isLoading()
 
 void CSSStyleSheetImpl::checkLoaded()
 {
-    if(isLoading()) return;
-    if(m_parent) m_parent->checkLoaded();
-    if(m_parentNode) m_parentNode->sheetLoaded();
+    if (isLoading()) return;
+    if (parent()) parent()->checkLoaded();
+    if (m_parentNode) m_parentNode->sheetLoaded();
 }
 
 khtml::DocLoader *CSSStyleSheetImpl::docLoader()
@@ -359,20 +355,18 @@ bool MediaListImpl::contains( const DOMString &medium ) const
 
 CSSStyleSheetImpl *MediaListImpl::parentStyleSheet() const
 {
-    if( m_parent->isCSSStyleSheet() ) return static_cast<CSSStyleSheetImpl *>(m_parent);
-    return 0;
+    return parent()->isCSSStyleSheet() ? static_cast<CSSStyleSheetImpl *>(parent()) : 0;
 }
 
 CSSRuleImpl *MediaListImpl::parentRule() const
 {
-    if( m_parent->isRule() ) return static_cast<CSSRuleImpl *>(m_parent);
-    return 0;
+    return parent()->isRule() ? static_cast<CSSRuleImpl *>(parent()) : 0;
 }
 
 void MediaListImpl::deleteMedium( const DOMString &oldMedium )
 {
     for ( QValueList<DOMString>::Iterator it = m_lstMedia.begin(); it != m_lstMedia.end(); ++it ) {
-        if( (*it) == oldMedium ) {
+        if ((*it) == oldMedium) {
             m_lstMedia.remove( it );
             return;
         }
@@ -398,7 +392,7 @@ void MediaListImpl::setMediaText(const DOM::DOMString &value)
     for ( QStringList::Iterator it = list.begin(); it != list.end(); ++it )
     {
         DOMString medium = (*it).stripWhiteSpace();
-        if( medium != "" )
+        if (!medium.isEmpty())
             m_lstMedia.append( medium );
     }
 }
index cbd9b5f..56ed886 100644 (file)
@@ -449,8 +449,8 @@ bool CSSMutableStyleDeclarationImpl::setProperty(int propertyID, const DOMString
     bool success = parser.parseValue(this, propertyID, value, important);
     if (!success) {
 #if !APPLE_CHANGES
-       kdDebug( 6080 ) << "CSSMutableStyleDeclarationImpl::setProperty invalid property: [" << getPropertyName(id).qstring()
-                       << "] value: [" << value.qstring() << "]"<< endl;
+        kdDebug( 6080 ) << "CSSMutableStyleDeclarationImpl::setProperty invalid property: [" << getPropertyName(id).qstring()
+                        << "] value: [" << value.qstring() << "]"<< endl;
 #endif
         exceptionCode = CSSException::SYNTAX_ERR + CSSException::_EXCEPTION_OFFSET;
     } else if (notifyChanged)
@@ -523,8 +523,7 @@ DOMString CSSMutableStyleDeclarationImpl::item(unsigned i) const
 
 CSSRuleImpl *CSSStyleDeclarationImpl::parentRule() const
 {
-    return (m_parent && m_parent->isRule() ) ?
-       static_cast<CSSRuleImpl *>(m_parent) : 0;
+    return (parent() && parent()->isRule()) ? static_cast<CSSRuleImpl *>(parent()) : 0;
 }
 
 DOM::DOMString CSSMutableStyleDeclarationImpl::cssText() const
@@ -712,11 +711,8 @@ CSSValueListImpl::CSSValueListImpl()
 
 CSSValueListImpl::~CSSValueListImpl()
 {
-    CSSValueImpl *val = m_values.first();
-    while( val ) {
-       val->deref();
-       val = m_values.next();
-    }
+    for (CSSValueImpl *val = m_values.first(); val; val = m_values.next())
+        val->deref();
 }
 
 unsigned short CSSValueListImpl::cssValueType() const
@@ -735,10 +731,9 @@ DOM::DOMString CSSValueListImpl::cssText() const
     DOMString result = "";
 
     for (QPtrListIterator<CSSValueImpl> iterator(m_values); iterator.current(); ++iterator) {
-       if (result.length() != 0) {
-           result += ", ";
-       }
-       result += iterator.current()->cssText();
+        if (!result.isEmpty())
+            result += ", ";
+        result += iterator.current()->cssText();
     }
     
     return result;
@@ -776,7 +771,7 @@ CSSPrimitiveValueImpl::CSSPrimitiveValueImpl(CounterImpl *c)
 {
     m_value.counter = c;
     if (m_value.counter)
-       m_value.counter->ref();
+        m_value.counter->ref();
     m_type = CSSPrimitiveValue::CSS_COUNTER;
 }
 
@@ -784,7 +779,7 @@ CSSPrimitiveValueImpl::CSSPrimitiveValueImpl( RectImpl *r)
 {
     m_value.rect = r;
     if (m_value.rect)
-       m_value.rect->ref();
+        m_value.rect->ref();
     m_type = CSSPrimitiveValue::CSS_RECT;
 }
 
@@ -792,7 +787,7 @@ CSSPrimitiveValueImpl::CSSPrimitiveValueImpl( DashboardRegionImpl *r)
 {
     m_value.region = r;
     if (m_value.region)
-       m_value.region->ref();
+        m_value.region->ref();
     m_type = CSSPrimitiveValue::CSS_DASHBOARD_REGION;
 }
 
@@ -806,7 +801,7 @@ CSSPrimitiveValueImpl::CSSPrimitiveValueImpl(PairImpl* p)
 {
     m_value.pair = p;
     if (p)
-       p->ref();
+        p->ref();
     m_type = CSSPrimitiveValue::CSS_PAIR;
 }
 
@@ -890,7 +885,7 @@ double CSSPrimitiveValueImpl::computeLengthFloat( khtml::RenderStyle *style, QPa
         // FIXME: We have a bug right now where the zoom will be applied multiple times to EX units.
         // We really need to compute EX using fontMetrics for the original specifiedSize and not use
         // our actual constructed rendering font.
-       {
+        {
         QFontMetrics fm = style->fontMetrics();
 #if APPLE_CHANGES
         factor = fm.xHeight();
@@ -899,14 +894,14 @@ double CSSPrimitiveValueImpl::computeLengthFloat( khtml::RenderStyle *style, QPa
         factor = b.height();
 #endif
         break;
-       }
+        }
     case CSSPrimitiveValue::CSS_PX:
         break;
     case CSSPrimitiveValue::CSS_CM:
-       factor = dpiY/2.54; //72dpi/(2.54 cm/in)
+        factor = dpiY/2.54; //72dpi/(2.54 cm/in)
         break;
     case CSSPrimitiveValue::CSS_MM:
-       factor = dpiY/25.4;
+        factor = dpiY/25.4;
         break;
     case CSSPrimitiveValue::CSS_IN:
             factor = dpiY;
@@ -930,9 +925,9 @@ void CSSPrimitiveValueImpl::setFloatValue( unsigned short unitType, double float
     exceptioncode = 0;
     cleanup();
     // ### check if property supports this type
-    if(m_type > CSSPrimitiveValue::CSS_DIMENSION) {
-       exceptioncode = CSSException::SYNTAX_ERR + CSSException::_EXCEPTION_OFFSET;
-       return;
+    if (m_type > CSSPrimitiveValue::CSS_DIMENSION) {
+        exceptioncode = CSSException::SYNTAX_ERR + CSSException::_EXCEPTION_OFFSET;
+        return;
     }
     //if(m_type > CSSPrimitiveValue::CSS_DIMENSION) throw DOMException(DOMException::INVALID_ACCESS_ERR);
     m_value.num = floatValue;
@@ -945,15 +940,15 @@ void CSSPrimitiveValueImpl::setStringValue( unsigned short stringType, const DOM
     cleanup();
     //if(m_type < CSSPrimitiveValue::CSS_STRING) throw DOMException(DOMException::INVALID_ACCESS_ERR);
     //if(m_type > CSSPrimitiveValue::CSS_ATTR) throw DOMException(DOMException::INVALID_ACCESS_ERR);
-    if(m_type < CSSPrimitiveValue::CSS_STRING || m_type >> CSSPrimitiveValue::CSS_ATTR) {
-       exceptioncode = CSSException::SYNTAX_ERR + CSSException::_EXCEPTION_OFFSET;
-       return;
+    if (m_type < CSSPrimitiveValue::CSS_STRING || m_type >> CSSPrimitiveValue::CSS_ATTR) {
+        exceptioncode = CSSException::SYNTAX_ERR + CSSException::_EXCEPTION_OFFSET;
+        return;
     }
     if(stringType != CSSPrimitiveValue::CSS_IDENT)
     {
-       m_value.string = stringValue.impl();
-       m_value.string->ref();
-       m_type = stringType;
+        m_value.string = stringValue.impl();
+        m_value.string->ref();
+        m_type = stringType;
     }
     // ### parse ident
 }
@@ -998,90 +993,90 @@ DOM::DOMString CSSPrimitiveValueImpl::cssText() const
     // name if it was specified) - check what spec says about this
     DOMString text;
     switch ( m_type ) {
-       case CSSPrimitiveValue::CSS_UNKNOWN:
-           // ###
-           break;
-       case CSSPrimitiveValue::CSS_NUMBER:
-           text = DOMString(QString::number( m_value.num ));
-           break;
-       case CSSPrimitiveValue::CSS_PERCENTAGE:
-           text = DOMString(QString::number( m_value.num ) + "%");
-           break;
-       case CSSPrimitiveValue::CSS_EMS:
-           text = DOMString(QString::number( m_value.num ) + "em");
-           break;
-       case CSSPrimitiveValue::CSS_EXS:
-           text = DOMString(QString::number( m_value.num ) + "ex");
-           break;
-       case CSSPrimitiveValue::CSS_PX:
-           text = DOMString(QString::number( m_value.num ) + "px");
-           break;
-       case CSSPrimitiveValue::CSS_CM:
-           text = DOMString(QString::number( m_value.num ) + "cm");
-           break;
-       case CSSPrimitiveValue::CSS_MM:
-           text = DOMString(QString::number( m_value.num ) + "mm");
-           break;
-       case CSSPrimitiveValue::CSS_IN:
-           text = DOMString(QString::number( m_value.num ) + "in");
-           break;
-       case CSSPrimitiveValue::CSS_PT:
-           text = DOMString(QString::number( m_value.num ) + "pt");
-           break;
-       case CSSPrimitiveValue::CSS_PC:
-           text = DOMString(QString::number( m_value.num ) + "pc");
-           break;
-       case CSSPrimitiveValue::CSS_DEG:
-           text = DOMString(QString::number( m_value.num ) + "deg");
-           break;
-       case CSSPrimitiveValue::CSS_RAD:
-           text = DOMString(QString::number( m_value.num ) + "rad");
-           break;
-       case CSSPrimitiveValue::CSS_GRAD:
-           text = DOMString(QString::number( m_value.num ) + "grad");
-           break;
-       case CSSPrimitiveValue::CSS_MS:
-           text = DOMString(QString::number( m_value.num ) + "ms");
-           break;
-       case CSSPrimitiveValue::CSS_S:
-           text = DOMString(QString::number( m_value.num ) + "s");
-           break;
-       case CSSPrimitiveValue::CSS_HZ:
-           text = DOMString(QString::number( m_value.num ) + "hz");
-           break;
-       case CSSPrimitiveValue::CSS_KHZ:
-           text = DOMString(QString::number( m_value.num ) + "khz");
-           break;
-       case CSSPrimitiveValue::CSS_DIMENSION:
-           // ###
-           break;
-       case CSSPrimitiveValue::CSS_STRING:
-           text = quoteStringIfNeeded(m_value.string);
-           break;
-       case CSSPrimitiveValue::CSS_URI:
+        case CSSPrimitiveValue::CSS_UNKNOWN:
+            // ###
+            break;
+        case CSSPrimitiveValue::CSS_NUMBER:
+            text = DOMString(QString::number( m_value.num ));
+            break;
+        case CSSPrimitiveValue::CSS_PERCENTAGE:
+            text = DOMString(QString::number( m_value.num ) + "%");
+            break;
+        case CSSPrimitiveValue::CSS_EMS:
+            text = DOMString(QString::number( m_value.num ) + "em");
+            break;
+        case CSSPrimitiveValue::CSS_EXS:
+            text = DOMString(QString::number( m_value.num ) + "ex");
+            break;
+        case CSSPrimitiveValue::CSS_PX:
+            text = DOMString(QString::number( m_value.num ) + "px");
+            break;
+        case CSSPrimitiveValue::CSS_CM:
+            text = DOMString(QString::number( m_value.num ) + "cm");
+            break;
+        case CSSPrimitiveValue::CSS_MM:
+            text = DOMString(QString::number( m_value.num ) + "mm");
+            break;
+        case CSSPrimitiveValue::CSS_IN:
+            text = DOMString(QString::number( m_value.num ) + "in");
+            break;
+        case CSSPrimitiveValue::CSS_PT:
+            text = DOMString(QString::number( m_value.num ) + "pt");
+            break;
+        case CSSPrimitiveValue::CSS_PC:
+            text = DOMString(QString::number( m_value.num ) + "pc");
+            break;
+        case CSSPrimitiveValue::CSS_DEG:
+            text = DOMString(QString::number( m_value.num ) + "deg");
+            break;
+        case CSSPrimitiveValue::CSS_RAD:
+            text = DOMString(QString::number( m_value.num ) + "rad");
+            break;
+        case CSSPrimitiveValue::CSS_GRAD:
+            text = DOMString(QString::number( m_value.num ) + "grad");
+            break;
+        case CSSPrimitiveValue::CSS_MS:
+            text = DOMString(QString::number( m_value.num ) + "ms");
+            break;
+        case CSSPrimitiveValue::CSS_S:
+            text = DOMString(QString::number( m_value.num ) + "s");
+            break;
+        case CSSPrimitiveValue::CSS_HZ:
+            text = DOMString(QString::number( m_value.num ) + "hz");
+            break;
+        case CSSPrimitiveValue::CSS_KHZ:
+            text = DOMString(QString::number( m_value.num ) + "khz");
+            break;
+        case CSSPrimitiveValue::CSS_DIMENSION:
+            // ###
+            break;
+        case CSSPrimitiveValue::CSS_STRING:
+            text = quoteStringIfNeeded(m_value.string);
+            break;
+        case CSSPrimitiveValue::CSS_URI:
             text  = "url(";
-           text += DOMString( m_value.string );
+            text += DOMString( m_value.string );
             text += ")";
-           break;
-       case CSSPrimitiveValue::CSS_IDENT:
-           text = getValueName(m_value.ident);
-           break;
-       case CSSPrimitiveValue::CSS_ATTR:
-           // ###
-           break;
-       case CSSPrimitiveValue::CSS_COUNTER:
-           // ###
-           break;
+            break;
+        case CSSPrimitiveValue::CSS_IDENT:
+            text = getValueName(m_value.ident);
+            break;
+        case CSSPrimitiveValue::CSS_ATTR:
+            // ###
+            break;
+        case CSSPrimitiveValue::CSS_COUNTER:
+            // ###
+            break;
         case CSSPrimitiveValue::CSS_RECT: {
-           RectImpl* rectVal = getRectValue();
+            RectImpl* rectVal = getRectValue();
             text = "rect(";
             text += rectVal->top()->cssText() + " ";
             text += rectVal->right()->cssText() + " ";
             text += rectVal->bottom()->cssText() + " ";
             text += rectVal->left()->cssText() + ")";
-           break;
+            break;
         }
-       case CSSPrimitiveValue::CSS_RGBCOLOR: {
+        case CSSPrimitiveValue::CSS_RGBCOLOR: {
             QColor color(m_value.rgbcolor);
             if (qAlpha(m_value.rgbcolor) < 0xFF)
                 text = "rgba(";
@@ -1358,38 +1353,33 @@ DOMString FontValueImpl::cssText() const
     DOMString result("");
 
     if (style) {
-       result += style->cssText();
+        result += style->cssText();
     }
     if (variant) {
-       if (result.length() > 0) {
-           result += " ";
-       }
-       result += variant->cssText();
+        if (!result.isEmpty())
+            result += " ";
+        result += variant->cssText();
     }
     if (weight) {
-       if (result.length() > 0) {
-           result += " ";
-       }
-       result += weight->cssText();
+        if (!result.isEmpty())
+            result += " ";
+        result += weight->cssText();
     }
     if (size) {
-       if (result.length() > 0) {
-           result += " ";
-       }
-       result += size->cssText();
+        if (!result.isEmpty())
+            result += " ";
+        result += size->cssText();
     }
     if (lineHeight) {
-       if (!size) {
-           result += " ";
-       }
-       result += "/";
-       result += lineHeight->cssText();
+        if (!size)
+            result += " ";
+        result += "/";
+        result += lineHeight->cssText();
     }
     if (family) {
-       if (result.length() > 0) {
-           result += " ";
-       }
-       result += family->cssText();
+        if (!result.isEmpty())
+            result += " ";
+        result += family->cssText();
     }
 
     return result;
@@ -1399,7 +1389,7 @@ DOMString FontValueImpl::cssText() const
 // Used for text-shadow and box-shadow
 ShadowValueImpl::ShadowValueImpl(CSSPrimitiveValueImpl* _x, CSSPrimitiveValueImpl* _y,
                                  CSSPrimitiveValueImpl* _blur, CSSPrimitiveValueImpl* _color)
-:x(_x), y(_y), blur(_blur), color(_color)      
+:x(_x), y(_y), blur(_blur), color(_color)
 {}
 
 ShadowValueImpl::~ShadowValueImpl()
@@ -1413,26 +1403,22 @@ ShadowValueImpl::~ShadowValueImpl()
 DOMString ShadowValueImpl::cssText() const
 {
     DOMString text("");
-    if (color) {
-       text += color->cssText();
-    }
+    if (color)
+        text += color->cssText();
     if (x) {
-       if (text.length() > 0) {
-           text += " ";
-       }
-       text += x->cssText();
+        if (!text.isEmpty())
+            text += " ";
+        text += x->cssText();
     }
     if (y) {
-       if (text.length() > 0) {
-           text += " ";
-       }
-       text += y->cssText();
+        if (!text.isEmpty())
+            text += " ";
+        text += y->cssText();
     }
     if (blur) {
-       if (text.length() > 0) {
-           text += " ";
-       }
-       text += blur->cssText();
+        if (!text.isEmpty())
+            text += " ";
+        text += blur->cssText();
     }
 
     return text;
index 08f7bfc..7eb6578 100644 (file)
@@ -281,9 +281,8 @@ RenderWidget::~RenderWidget()
 {
     KHTMLAssert( refCount() <= 0 );
 
-    if (m_deleteWidget) {
-       delete m_widget;
-    }
+    if (m_deleteWidget)
+        delete m_widget;
 }
 
 void  RenderWidget::resizeWidget( QWidget *widget, int w, int h )
@@ -310,9 +309,8 @@ void RenderWidget::setQWidget(QWidget *widget, bool deleteWidget)
         if (m_widget) {
             m_widget->removeEventFilter(this);
             disconnect( m_widget, SIGNAL( destroyed()), this, SLOT( slotWidgetDestructed()));
-           if (m_deleteWidget) {
-               delete m_widget;
-           }
+            if (m_deleteWidget)
+                delete m_widget;
             m_widget = 0;
         }
         m_widget = widget;
@@ -323,23 +321,21 @@ void RenderWidget::setQWidget(QWidget *widget, bool deleteWidget)
             // widget immediately, but we have to have really been full constructed (with a non-null
             // style pointer).
             if (!needsLayout() && style()) {
-               resizeWidget( m_widget,
-                             m_width-borderLeft()-borderRight()-paddingLeft()-paddingRight(),
-                             m_height-borderLeft()-borderRight()-paddingLeft()-paddingRight() );
+                resizeWidget( m_widget,
+                              m_width-borderLeft()-borderRight()-paddingLeft()-paddingRight(),
+                              m_height-borderLeft()-borderRight()-paddingLeft()-paddingRight() );
             }
             else
                 setPos(xPos(), -500000);
 
-#if APPLE_CHANGES
-           if (style()) {
-               if (style()->visibility() != VISIBLE)
+            if (style()) {
+                if (style()->visibility() != VISIBLE)
                     m_widget->hide();
-               else
-                   m_widget->show();
-           }
-#endif
+                else
+                    m_widget->show();
+            }
         }
-       m_view->addChild( m_widget, -500000, 0 );
+        m_view->addChild( m_widget, -500000, 0 );
     }
     m_deleteWidget = deleteWidget;
 }
@@ -391,7 +387,6 @@ void RenderWidget::paint(PaintInfo& i, int _tx, int _ty)
     if (shouldPaintBackgroundOrBorder() && i.phase != PaintActionOutline) 
         paintBoxDecorations(i, _tx, _ty);
 
-#if APPLE_CHANGES
     if (!m_widget || !m_view || i.phase != PaintActionForeground ||
         style()->visibility() != VISIBLE)
         return;
@@ -415,56 +410,6 @@ void RenderWidget::paint(PaintInfo& i, int _tx, int _ty)
         QRect selRect(selectionRect());
         i.p->fillRect(selRect.x(), selRect.y(), selRect.width(), selRect.height(), brush);
     }
-    
-#else
-    if (!m_widget || !m_view || i.phase != PaintActionForeground)
-        return;
-    
-    if (style()->visibility() != VISIBLE) {
-        m_widget->hide();
-        return;
-    }
-
-    int xPos = _tx+borderLeft()+paddingLeft();
-    int yPos = _ty+borderTop()+paddingTop();
-
-    int childw = m_widget->width();
-    int childh = m_widget->height();
-    if ( (childw == 2000 || childh == 3072) && m_widget->inherits( "KHTMLView" ) ) {
-       KHTMLView *vw = static_cast<KHTMLView *>(m_widget);
-       int cy = m_view->contentsY();
-       int ch = m_view->visibleHeight();
-
-
-       int childx = m_view->childX( m_widget );
-       int childy = m_view->childY( m_widget );
-
-       int xNew = xPos;
-       int yNew = childy;
-
-       //      qDebug("cy=%d, ch=%d, childy=%d, childh=%d", cy, ch, childy, childh );
-       if ( childh == 3072 ) {
-           if ( cy + ch > childy + childh ) {
-               yNew = cy + ( ch - childh )/2;
-           } else if ( cy < childy ) {
-               yNew = cy + ( ch - childh )/2;
-           }
-//         qDebug("calculated yNew=%d", yNew);
-       }
-       yNew = kMin( yNew, yPos + m_height - childh );
-       yNew = kMax( yNew, yPos );
-       if ( yNew != childy || xNew != childx ) {
-           if ( vw->contentsHeight() < yNew - yPos + childh )
-               vw->resizeContents( vw->contentsWidth(), yNew - yPos + childh );
-           vw->setContentsPos( xNew - xPos, yNew - yPos );
-       }
-       xPos = xNew;
-       yPos = yNew;
-    }
-
-    m_view->addChild(m_widget, xPos, yPos );
-    m_widget->show();
-#endif
 }
 
 void RenderWidget::handleFocusOut()
@@ -497,7 +442,7 @@ bool RenderWidget::eventFilter(QObject* /*o*/, QEvent* e)
 //                 KHTMLPartBrowserExtension *ext = static_cast<KHTMLPartBrowserExtension *>( elem->view->part()->browserExtension() );
 //                 if ( ext )  ext->editableWidgetBlurred( m_widget );
 //             }
-           handleFocusOut();
+            handleFocusOut();
         }
         break;
     case QEvent::FocusIn:
index f91e07c..f50ab16 100644 (file)
@@ -345,7 +345,7 @@ NodeImpl *NodeImpl::addChild(NodeImpl *)
 
 bool NodeImpl::isContentEditable() const
 {
-    return m_parent ? m_parent->isContentEditable() : false;
+    return parent() && parent()->isContentEditable();
 }
 
 QRect NodeImpl::getRect() const
index d6c48a5..a44ef73 100644 (file)
@@ -88,7 +88,7 @@ public:
     virtual DOMString nodeValue() const;
     virtual void setNodeValue( const DOMString &_nodeValue, int &exceptioncode );
     virtual unsigned short nodeType() const = 0;
-    NodeImpl *parentNode() const { return m_parent; }
+    NodeImpl *parentNode() const { return parent(); }
     NodeImpl *previousSibling() const { return m_previous; }
     NodeImpl *nextSibling() const { return m_next; }
     virtual SharedPtr<NodeListImpl> childNodes();
index 6b05991..27dd0df 100644 (file)
@@ -82,8 +82,8 @@ void XSLStyleSheetImpl::checkLoaded()
 {
     if (isLoading()) 
         return;
-    if (m_parent)
-        m_parent->checkLoaded();
+    if (parent())
+        parent()->checkLoaded();
     if (m_parentNode)
         m_parentNode->sheetLoaded();
 }
@@ -178,7 +178,7 @@ void XSLStyleSheetImpl::loadChildSheet(const QString& href)
 
 XSLStyleSheetImpl* XSLImportRuleImpl::parentStyleSheet() const
 {
-    return (m_parent && m_parent->isXSLStyleSheet()) ? static_cast<XSLStyleSheetImpl*>(m_parent) : 0;
+    return (parent() && parent()->isXSLStyleSheet()) ? static_cast<XSLStyleSheetImpl*>(parent()) : 0;
 }
 
 xsltStylesheetPtr XSLStyleSheetImpl::compileStyleSheet()
@@ -276,9 +276,9 @@ void XSLImportRuleImpl::loadSheet()
     StyleBaseImpl* root = this;
     StyleBaseImpl* parent;
     while ((parent = root->parent()))
-       root = parent;
+        root = parent;
     if (root->isXSLStyleSheet())
-       docLoader = static_cast<XSLStyleSheetImpl*>(root)->docLoader();
+        docLoader = static_cast<XSLStyleSheetImpl*>(root)->docLoader();
     
     DOMString absHref = m_strHref;
     XSLStyleSheetImpl* parentSheet = parentStyleSheet();