Renamed CharacterData::str to m_str to match coding style
authorddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 8 Jan 2008 13:02:46 +0000 (13:02 +0000)
committerddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 8 Jan 2008 13:02:46 +0000 (13:02 +0000)
        Rubber-stamped by Adam.

        No test cases added since there is no change in behavior.

        * dom/CDATASection.cpp:
        * dom/CharacterData.cpp:
        (WebCore::CharacterData::CharacterData):
        (WebCore::CharacterData::setData):
        (WebCore::CharacterData::substringData):
        (WebCore::CharacterData::appendData):
        (WebCore::CharacterData::insertData):
        (WebCore::CharacterData::deleteData):
        (WebCore::CharacterData::replaceData):
        (WebCore::CharacterData::nodeValue):
        (WebCore::CharacterData::containsOnlyWhitespace):
        (WebCore::CharacterData::dispatchModifiedEvent):
        (WebCore::CharacterData::checkCharDataOperation):
        (WebCore::CharacterData::dump):
        * dom/CharacterData.h:
        * dom/Comment.cpp:
        * dom/Text.cpp:
        (WebCore::Text::splitText):
        (WebCore::Text::createRenderer):
        (WebCore::Text::recalcStyle):

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

WebCore/ChangeLog
WebCore/dom/CDATASection.cpp
WebCore/dom/CharacterData.cpp
WebCore/dom/CharacterData.h
WebCore/dom/Comment.cpp
WebCore/dom/Text.cpp

index e992c2e60e53e1e629b63846c7ba108aa5e66c20..a5d24fa10dfb8927b15a5380e93adebad48e2012 100644 (file)
@@ -1,3 +1,32 @@
+2008-01-08  David D. Kilzer  <ddkilzer@webkit.org>
+
+        Renamed CharacterData::str to m_str to match coding style
+
+        Rubber-stamped by Adam.
+
+        No test cases added since there is no change in behavior.
+
+        * dom/CDATASection.cpp:
+        * dom/CharacterData.cpp:
+        (WebCore::CharacterData::CharacterData):
+        (WebCore::CharacterData::setData):
+        (WebCore::CharacterData::substringData):
+        (WebCore::CharacterData::appendData):
+        (WebCore::CharacterData::insertData):
+        (WebCore::CharacterData::deleteData):
+        (WebCore::CharacterData::replaceData):
+        (WebCore::CharacterData::nodeValue):
+        (WebCore::CharacterData::containsOnlyWhitespace):
+        (WebCore::CharacterData::dispatchModifiedEvent):
+        (WebCore::CharacterData::checkCharDataOperation):
+        (WebCore::CharacterData::dump):
+        * dom/CharacterData.h:
+        * dom/Comment.cpp:
+        * dom/Text.cpp:
+        (WebCore::Text::splitText):
+        (WebCore::Text::createRenderer):
+        (WebCore::Text::recalcStyle):
+
 2008-01-08  Oliver Hunt  <oliver@apple.com>
 
         Reviewed by NOBODY (Build fix).
index 829e911a3e9d0e021b8865822fc6d2b28862df53..9103d02aedaeb56147a934cc4c384fd505a56b98 100644 (file)
@@ -47,7 +47,7 @@ Node::NodeType CDATASection::nodeType() const
 
 PassRefPtr<Node> CDATASection::cloneNode(bool /*deep*/)
 {
-    return new CDATASection(document(), str);
+    return new CDATASection(document(), m_str);
 }
 
 // DOM Section 1.1.1
index 24e5c3c05ce361c1893223aa95fa4dea24c9c32d..84de10c58264dcccc5a4f47a2534259429014693 100644 (file)
@@ -42,7 +42,7 @@ CharacterData::CharacterData(Document *doc)
 CharacterData::CharacterData(Document *doc, const String &_text)
     : EventTargetNode(doc)
 {
-    str = _text.impl() ? _text.impl() : StringImpl::empty();
+    m_str = _text.impl() ? _text.impl() : StringImpl::empty();
 }
 
 CharacterData::~CharacterData()
@@ -57,17 +57,17 @@ void CharacterData::setData(const String& data, ExceptionCode& ec)
         return;
     }
 
-    if (equal(str.get(), data.impl()))
+    if (equal(m_str.get(), data.impl()))
         return;
 
-    RefPtr<StringImpl> oldStr = str;
-    str = data.impl();
+    RefPtr<StringImpl> oldStr = m_str;
+    m_str = data.impl();
     
     if ((!renderer() || !rendererIsNeeded(renderer()->style())) && attached()) {
         detach();
         attach();
     } else if (renderer())
-        static_cast<RenderText*>(renderer())->setText(str);
+        static_cast<RenderText*>(renderer())->setText(m_str);
     
     dispatchModifiedEvent(oldStr.get());
     
@@ -81,7 +81,7 @@ String CharacterData::substringData( const unsigned offset, const unsigned count
     if (ec)
         return String();
 
-    return str->substring(offset, count);
+    return m_str->substring(offset, count);
 }
 
 void CharacterData::appendData( const String &arg, ExceptionCode& ec)
@@ -94,17 +94,17 @@ void CharacterData::appendData( const String &arg, ExceptionCode& ec)
         return;
     }
 
-    String newStr = str;
+    String newStr = m_str;
     newStr.append(arg);
 
-    RefPtr<StringImpl> oldStr = str;
-    str = newStr.impl();
+    RefPtr<StringImpl> oldStr = m_str;
+    m_str = newStr.impl();
 
     if ((!renderer() || !rendererIsNeeded(renderer()->style())) && attached()) {
         detach();
         attach();
     } else if (renderer())
-        static_cast<RenderText*>(renderer())->setTextWithOffset(str, oldStr->length(), 0);
+        static_cast<RenderText*>(renderer())->setTextWithOffset(m_str, oldStr->length(), 0);
     
     dispatchModifiedEvent(oldStr.get());
 }
@@ -116,17 +116,17 @@ void CharacterData::insertData( const unsigned offset, const String &arg, Except
     if (ec)
         return;
 
-    String newStr = str;
+    String newStr = m_str;
     newStr.insert(arg, offset);
 
-    RefPtr<StringImpl> oldStr = str;
-    str = newStr.impl();
+    RefPtr<StringImpl> oldStr = m_str;
+    m_str = newStr.impl();
 
     if ((!renderer() || !rendererIsNeeded(renderer()->style())) && attached()) {
         detach();
         attach();
     } else if (renderer())
-        static_cast<RenderText*>(renderer())->setTextWithOffset(str, offset, 0);
+        static_cast<RenderText*>(renderer())->setTextWithOffset(m_str, offset, 0);
     
     dispatchModifiedEvent(oldStr.get());
     
@@ -142,17 +142,17 @@ void CharacterData::deleteData( const unsigned offset, const unsigned count, Exc
     if (ec)
         return;
 
-    String newStr = str;
+    String newStr = m_str;
     newStr.remove(offset, count);
 
-    RefPtr<StringImpl> oldStr = str;
-    str = newStr.impl();
+    RefPtr<StringImpl> oldStr = m_str;
+    m_str = newStr.impl();
     
     if ((!renderer() || !rendererIsNeeded(renderer()->style())) && attached()) {
         detach();
         attach();
     } else if (renderer())
-        static_cast<RenderText*>(renderer())->setTextWithOffset(str, offset, count);
+        static_cast<RenderText*>(renderer())->setTextWithOffset(m_str, offset, count);
 
     dispatchModifiedEvent(oldStr.get());
 
@@ -169,23 +169,23 @@ void CharacterData::replaceData( const unsigned offset, const unsigned count, co
         return;
 
     unsigned realCount;
-    if (offset + count > str->length())
-        realCount = str->length()-offset;
+    if (offset + count > m_str->length())
+        realCount = m_str->length()-offset;
     else
         realCount = count;
 
-    String newStr = str;
+    String newStr = m_str;
     newStr.remove(offset, realCount);
     newStr.insert(arg, offset);
 
-    RefPtr<StringImpl> oldStr = str;
-    str = newStr.impl();
+    RefPtr<StringImpl> oldStr = m_str;
+    m_str = newStr.impl();
 
     if ((!renderer() || !rendererIsNeeded(renderer()->style())) && attached()) {
         detach();
         attach();
     } else if (renderer())
-        static_cast<RenderText*>(renderer())->setTextWithOffset(str, offset, count);
+        static_cast<RenderText*>(renderer())->setTextWithOffset(m_str, offset, count);
     
     dispatchModifiedEvent(oldStr.get());
     
@@ -197,13 +197,13 @@ void CharacterData::replaceData( const unsigned offset, const unsigned count, co
 
 String CharacterData::nodeValue() const
 {
-    return str;
+    return m_str;
 }
 
 bool CharacterData::containsOnlyWhitespace() const
 {
-    if (str)
-        return str->containsOnlyWhitespace();
+    if (m_str)
+        return m_str->containsOnlyWhitespace();
     return true;
 }
 
@@ -219,7 +219,7 @@ void CharacterData::dispatchModifiedEvent(StringImpl *prevValue)
         parentNode()->childrenChanged();
     if (document()->hasListenerType(Document::DOMCHARACTERDATAMODIFIED_LISTENER)) {
         ExceptionCode ec;
-        dispatchEvent(new MutationEvent(DOMCharacterDataModifiedEvent, true, false, 0, prevValue, str, String(), 0), ec);
+        dispatchEvent(new MutationEvent(DOMCharacterDataModifiedEvent, true, false, 0, prevValue, m_str, String(), 0), ec);
     }
     dispatchSubtreeModifiedEvent();
 }
@@ -230,7 +230,7 @@ void CharacterData::checkCharDataOperation( const unsigned offset, ExceptionCode
 
     // INDEX_SIZE_ERR: Raised if the specified offset is negative or greater than the number of 16-bit
     // units in data.
-    if (offset > str->length()) {
+    if (offset > m_str->length()) {
         ec = INDEX_SIZE_ERR;
         return;
     }
@@ -249,7 +249,7 @@ int CharacterData::maxCharacterOffset() const
 
 bool CharacterData::rendererIsNeeded(RenderStyle *style)
 {
-    if (!str || str->length() == 0)
+    if (!m_str || m_str->length() == 0)
         return false;
     return EventTargetNode::rendererIsNeeded(style);
 }
@@ -262,9 +262,9 @@ bool CharacterData::offsetInCharacters() const
 #ifndef NDEBUG
 void CharacterData::dump(TextStream *stream, DeprecatedString ind) const
 {
-    *stream << " str=\"" << String(str).utf8().data() << "\"";
+    *stream << " m_str=\"" << String(m_str).utf8().data() << "\"";
 
-    EventTargetNode::dump(stream,ind);
+    EventTargetNode::dump(stream, ind);
 }
 #endif
 
index bf65ff1517a68711c8805544ad6010cb2bd29afc..8952784bccc84a0efa9ad7d715272b241560df44 100644 (file)
@@ -35,9 +35,9 @@ public:
 
     // DOM methods & attributes for CharacterData
 
-    String data() const { return str; }
+    String data() const { return m_str; }
     void setData(const String&, ExceptionCode&);
-    unsigned length() const { return str->length(); }
+    unsigned length() const { return m_str->length(); }
     String substringData(unsigned offset, unsigned count, ExceptionCode&);
     void appendData(const String&, ExceptionCode&);
     void insertData(unsigned offset, const String&, ExceptionCode&);
@@ -55,7 +55,7 @@ public:
 
     virtual bool isCharacterDataNode() const { return true; }
     virtual int maxCharacterOffset() const;
-    StringImpl* string() { return str.get(); }
+    StringImpl* string() { return m_str.get(); }
     virtual void checkCharDataOperation(unsigned offset, ExceptionCode&);
 
     virtual bool offsetInCharacters() const;
@@ -66,7 +66,7 @@ public:
 #endif
 
 protected:
-    RefPtr<StringImpl> str;
+    RefPtr<StringImpl> m_str;
 
     void dispatchModifiedEvent(StringImpl* oldValue);
 };
index d4a02a2de1e93d1c32366e91258c0809a1086315..9c89d0bc9cc5d48c8cc37126c5dc35da0388eab9 100644 (file)
@@ -59,7 +59,7 @@ Node::NodeType Comment::nodeType() const
 
 PassRefPtr<Node> Comment::cloneNode(bool /*deep*/)
 {
-    return document()->createComment(str);
+    return document()->createComment(m_str);
 }
 
 // DOM Section 1.1.1
index fe128795ca5cd5115241589d788bc6d149bf4ce0..b42b6bc260e8257979d2ae9ae00981cb9c6c1fb0 100644 (file)
@@ -59,7 +59,7 @@ PassRefPtr<Text> Text::splitText(unsigned offset, ExceptionCode& ec)
     
     // INDEX_SIZE_ERR: Raised if the specified offset is negative or greater than
     // the number of 16-bit units in data.
-    if (offset > str->length()) {
+    if (offset > m_str->length()) {
         ec = INDEX_SIZE_ERR;
         return 0;
     }
@@ -70,9 +70,9 @@ PassRefPtr<Text> Text::splitText(unsigned offset, ExceptionCode& ec)
         return 0;
     }
 
-    RefPtr<StringImpl> oldStr = str;
+    RefPtr<StringImpl> oldStr = m_str;
     PassRefPtr<Text> newText = createNew(oldStr->substring(offset));
-    str = oldStr->substring(0, offset);
+    m_str = oldStr->substring(0, offset);
 
     dispatchModifiedEvent(oldStr.get());
 
@@ -82,7 +82,7 @@ PassRefPtr<Text> Text::splitText(unsigned offset, ExceptionCode& ec)
         return 0;
 
     if (renderer())
-        static_cast<RenderText*>(renderer())->setText(str);
+        static_cast<RenderText*>(renderer())->setText(m_str);
 
     return newText;
 }
@@ -104,7 +104,7 @@ Node::NodeType Text::nodeType() const
 
 PassRefPtr<Node> Text::cloneNode(bool /*deep*/)
 {
-    return document()->createTextNode(str);
+    return document()->createTextNode(m_str);
 }
 
 bool Text::rendererIsNeeded(RenderStyle *style)
@@ -153,10 +153,10 @@ RenderObject *Text::createRenderer(RenderArena *arena, RenderStyle *style)
 {
 #if ENABLE(SVG)
     if (parentNode()->isSVGElement())
-        return new (arena) RenderSVGInlineText(this, str);
+        return new (arena) RenderSVGInlineText(this, m_str);
 #endif // ENABLE(SVG)
     
-    return new (arena) RenderText(this, str);
+    return new (arena) RenderText(this, m_str);
 }
 
 void Text::attach()
@@ -171,7 +171,7 @@ void Text::recalcStyle( StyleChange change )
         if (renderer())
             renderer()->setStyle(parentNode()->renderer()->style());
     if (changed() && renderer() && renderer()->isText())
-        static_cast<RenderText*>(renderer())->setText(str);
+        static_cast<RenderText*>(renderer())->setText(m_str);
     setChanged(NoStyleChange);
 }