Renamed CharacterData::m_str to m_data
authorddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 8 Jan 2008 21:27:13 +0000 (21:27 +0000)
committerddkilzer@apple.com <ddkilzer@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 8 Jan 2008 21:27:13 +0000 (21:27 +0000)
        Rubber-stamped by Adam again.

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