JavaScriptCore:
authormjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Jan 2006 00:56:32 +0000 (00:56 +0000)
committermjs <mjs@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 24 Jan 2006 00:56:32 +0000 (00:56 +0000)
        Rubber stamped by Tim Hatcher.

        - renamed inert() operation on HashSet, HashCountedSet and HashTable to add()
        for consistency with HashMap

        * kjs/array_object.cpp:
        (ArrayProtoFunc::callAsFunction):
        * kjs/collector.cpp:
        (KJS::Collector::protect):
        * kjs/identifier.cpp:
        (KJS::Identifier::add):
        * kxmlcore/HashCountedSet.h:
        (KXMLCore::::add):
        * kxmlcore/HashMap.h:
        (KXMLCore::::inlineAdd):
        * kxmlcore/HashSet.h:
        (KXMLCore::::add):
        * kxmlcore/HashTable.h:
        (KXMLCore::HashTable::add):
        (KXMLCore::::add):
        (KXMLCore::::HashTable):

WebCore:

        Rubber stamped by Tim Hatcher.

        - renamed inert() operation on HashSet, HashCountedSet and HashTable to add()
        for consistency with HashMap

        * bridge/mac/MacFrame.mm:
        (MacFrame::didTellBridgeAboutLoad):
        * khtml/ecma/kjs_dom.cpp:
        (KJS::DOMNode::mark):
        * khtml/html/HTMLElementImpl.cpp:
        (WebCore::HTMLElementImpl::isRecognizedTagName):
        (WebCore::inlineTagList):
        (WebCore::blockTagList):
        * khtml/html/HTMLFormCollectionImpl.cpp:
        (WebCore::HTMLFormCollectionImpl::updateNameCache):
        * khtml/html/htmlparser.cpp:
        (HTMLParser::isHeaderTag):
        (HTMLParser::isResidualStyleTag):
        (HTMLParser::isAffectedByResidualStyle):
        * khtml/xml/DocumentImpl.cpp:
        (WebCore::DocumentImpl::addElementById):
        (WebCore::DocumentImpl::registerDisconnectedNodeWithEventListeners):
        * khtml/xml/NodeImpl.cpp:
        (WebCore::NodeImpl::registerNodeList):
        * khtml/xml/dom_atomicstring.cpp:
        (DOM::AtomicString::add):
        * khtml/xml/dom_qname.cpp:
        (DOM::QualifiedName::QualifiedName):
        * loader/CachedObject.cpp:
        (WebCore::CachedObject::ref):
        * page/Frame.cpp:
        (Frame::keepAlive):
        * rendering/render_canvas.cpp:
        (RenderCanvas::addWidget):
        * rendering/render_frames.cpp:
        (WebCore::RenderPartObject::updateWidget):
        * rendering/render_line.cpp:
        (WebCore::InlineFlowBox::paint):
        * xml/xmlhttprequest.cpp:
        (WebCore::XMLHttpRequest::addToRequestsByDocument):

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

24 files changed:
JavaScriptCore/ChangeLog
JavaScriptCore/kjs/array_object.cpp
JavaScriptCore/kjs/collector.cpp
JavaScriptCore/kjs/identifier.cpp
JavaScriptCore/kxmlcore/HashCountedSet.h
JavaScriptCore/kxmlcore/HashMap.h
JavaScriptCore/kxmlcore/HashSet.h
JavaScriptCore/kxmlcore/HashTable.h
WebCore/ChangeLog
WebCore/bridge/mac/MacFrame.mm
WebCore/khtml/ecma/kjs_dom.cpp
WebCore/khtml/html/HTMLElementImpl.cpp
WebCore/khtml/html/HTMLFormCollectionImpl.cpp
WebCore/khtml/html/htmlparser.cpp
WebCore/khtml/xml/DocumentImpl.cpp
WebCore/khtml/xml/NodeImpl.cpp
WebCore/khtml/xml/dom_atomicstring.cpp
WebCore/khtml/xml/dom_qname.cpp
WebCore/loader/CachedObject.cpp
WebCore/page/Frame.cpp
WebCore/rendering/render_canvas.cpp
WebCore/rendering/render_frames.cpp
WebCore/rendering/render_line.cpp
WebCore/xml/xmlhttprequest.cpp

index fd8922fc95ef0dc6f54548195a75248e79a8015c..2f3b0c740ec52a31d53fa45f28e28a4b0c05dafe 100644 (file)
@@ -1,3 +1,27 @@
+2006-01-23  Maciej Stachowiak  <mjs@apple.com>
+
+        Rubber stamped by Tim Hatcher.
+        
+        - renamed inert() operation on HashSet, HashCountedSet and HashTable to add()
+        for consistency with HashMap
+
+        * kjs/array_object.cpp:
+        (ArrayProtoFunc::callAsFunction):
+        * kjs/collector.cpp:
+        (KJS::Collector::protect):
+        * kjs/identifier.cpp:
+        (KJS::Identifier::add):
+        * kxmlcore/HashCountedSet.h:
+        (KXMLCore::::add):
+        * kxmlcore/HashMap.h:
+        (KXMLCore::::inlineAdd):
+        * kxmlcore/HashSet.h:
+        (KXMLCore::::add):
+        * kxmlcore/HashTable.h:
+        (KXMLCore::HashTable::add):
+        (KXMLCore::::add):
+        (KXMLCore::::HashTable):
+
 2006-01-23  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by thatcher
index 2d3bb6d2f598aeadfff9a6c3d87f9ccbc15035e0..9baa812ab2111dee0c832f4d5e5590aa8021524c 100644 (file)
@@ -480,7 +480,7 @@ JSValue *ArrayProtoFunc::callAsFunction(ExecState *exec, JSObject *thisObj, cons
     UString separator = ",";
     UString str = "";
 
-    visitedElems.insert(thisObj);
+    visitedElems.add(thisObj);
     if (id == Join && !args[0]->isUndefined())
       separator = args[0]->toString(exec);
     for (unsigned int k = 0; k < length; k++) {
index e887d877535059f07597942fbae69d592ce0534f..312fd3e390040ad0b8d98ab6dd632ca707d007db 100644 (file)
@@ -411,7 +411,7 @@ void Collector::protect(JSValue *k)
     if (SimpleNumber::is(k))
       return;
 
-    protectedValues().insert(k->downcast());
+    protectedValues().add(k->downcast());
 }
 
 void Collector::unprotect(JSValue *k)
index f78017a8336227901f5e33744b0067dcd3345838..b762feb5ec34e9deac69fac1561a72eb0a4a43ff 100644 (file)
@@ -138,7 +138,7 @@ PassRefPtr<UString::Rep> Identifier::add(const char *c)
     if (length == 0)
         return &UString::Rep::empty;
     
-    return *identifierTable().insert<const char *, CStringTranslator>(c).first;
+    return *identifierTable().add<const char *, CStringTranslator>(c).first;
 }
 
 struct UCharBuffer {
@@ -179,7 +179,7 @@ PassRefPtr<UString::Rep> Identifier::add(const UChar *s, int length)
         return &UString::Rep::empty;
     
     UCharBuffer buf = {s, length}; 
-    return *identifierTable().insert<UCharBuffer, UCharBufferTranslator>(buf).first;
+    return *identifierTable().add<UCharBuffer, UCharBufferTranslator>(buf).first;
 }
 
 PassRefPtr<UString::Rep> Identifier::add(UString::Rep *r)
@@ -190,7 +190,7 @@ PassRefPtr<UString::Rep> Identifier::add(UString::Rep *r)
     if (r->len == 0)
         return &UString::Rep::empty;
 
-    UString::Rep *result = *identifierTable().insert(r).first;
+    UString::Rep *result = *identifierTable().add(r).first;
     if (result == r)
         r->isIdentifier = true;
     return result;
index d35307cd60a4269f36e5530e25781351b1c466da..c4503117b37b39722ad8bbb3e9bab3e20098e06b 100644 (file)
@@ -55,9 +55,9 @@ namespace KXMLCore {
         unsigned count(const ValueType& value) const;
 
         // increases the count if an equal value is already present
-        // returns value is a pair of an interator to the new value's location, 
-        // and a bool that is true if an actual new insertion was done
-        std::pair<iterator, bool> insert(const ValueType &value);
+        // the return value is a pair of an interator to the new value's location, 
+        // and a bool that is true if an new entry was added
+        std::pair<iterator, bool> add(const ValueType &value);
         
         // reduces the count of the value, and removes it if count
         // goes down to zero
@@ -137,7 +137,7 @@ namespace KXMLCore {
     }
     
     template<typename Value, typename HashFunctions, typename Traits>
-    inline std::pair<typename HashCountedSet<Value, HashFunctions, Traits>::iterator, bool> HashCountedSet<Value, HashFunctions, Traits>::insert(const ValueType &value)
+    inline std::pair<typename HashCountedSet<Value, HashFunctions, Traits>::iterator, bool> HashCountedSet<Value, HashFunctions, Traits>::add(const ValueType &value)
     {
         pair<iterator, bool> result = m_impl.add(value, 0); 
         ++result.first->second;
index 330a90d9c58bc678094c1981beb0b0ee7244f058..bb89ff648ff84cd3fb215077a910ac6ef9de3052 100644 (file)
@@ -173,7 +173,7 @@ bool HashMap<Key, Mapped, HashFunctions, KeyTraits, MappedTraits>::contains(cons
 template<typename Key, typename Mapped, typename HashFunctions, typename KeyTraits, typename MappedTraits>
 pair<typename HashMap<Key, Mapped, HashFunctions, KeyTraits, MappedTraits>::iterator, bool> HashMap<Key, Mapped, HashFunctions, KeyTraits, MappedTraits>::inlineAdd(const KeyType &key, const MappedType &mapped) 
 {
-    return m_impl.template insert<KeyType, MappedType, TranslatorType>(key, mapped); 
+    return m_impl.template add<KeyType, MappedType, TranslatorType>(key, mapped); 
 }
 
 template<typename Key, typename Mapped, typename HashFunctions, typename KeyTraits, typename MappedTraits>
index 5728627a0243841341064c28078f595dee4704a9..7d513ed329579eda1cbd6423495c297da6f390d2 100644 (file)
@@ -79,16 +79,18 @@ namespace KXMLCore {
         const_iterator find(const ValueType& value) const;
         bool contains(const ValueType& value) const;
         
-        std::pair<iterator, bool> insert(const ValueType &value);
+        // the return value is a pair of an interator to the new value's location, 
+        // and a bool that is true if an new entry was added
+        std::pair<iterator, bool> add(const ValueType &value);
         
-        // a special version of insert() that finds the object by hashing and comparing
+        // a special version of add() that finds the object by hashing and comparing
         // with some other type, to avoid the cost of type conversion if the object is already
         // in the table. HashTranslator should have the following methods:
         //   static unsigned hash(const T&);
         //   static bool equal(const ValueType&, const T&);
         //   static translate(ValueType&, const T&, unsigned hashCode);
         template<typename T, typename HashTranslator> 
-        std::pair<iterator, bool> insert(const T& value);
+        std::pair<iterator, bool> add(const T& value);
         
         void remove(const ValueType& value);
         void remove(iterator it);
@@ -159,16 +161,16 @@ namespace KXMLCore {
     }
     
     template<typename Value, typename HashFunctions, typename Traits>
-    std::pair<typename HashSet<Value, HashFunctions, Traits>::iterator, bool> HashSet<Value, HashFunctions, Traits>::insert(const ValueType &value)
+    std::pair<typename HashSet<Value, HashFunctions, Traits>::iterator, bool> HashSet<Value, HashFunctions, Traits>::add(const ValueType &value)
     {
-        return m_impl.insert(value); 
+        return m_impl.add(value); 
     }
     
     template<typename Value, typename HashFunctions, typename Traits>
     template<typename T, typename HashSetTranslator> 
-    std::pair<typename HashSet<Value, HashFunctions, Traits>::iterator, bool> HashSet<Value, HashFunctions, Traits>::insert(const T& value)
+    std::pair<typename HashSet<Value, HashFunctions, Traits>::iterator, bool> HashSet<Value, HashFunctions, Traits>::add(const T& value)
     {
-        return m_impl.template insert<T, T, HashSetTranslatorAdapter<ValueType, T, HashSetTranslator> >(value, value); 
+        return m_impl.template add<T, T, HashSetTranslatorAdapter<ValueType, T, HashSetTranslator> >(value, value); 
     }
     
     template<typename Value, typename HashFunctions, typename Traits>
index e8e03964a41b869f415d785d4d3a211f5e20f9bf..9235cba3217af3d72e78cdd9719f26d090939363 100644 (file)
@@ -284,12 +284,12 @@ namespace KXMLCore {
         int size() const { return m_keyCount; }
         int capacity() const { return m_tableSize; }
 
-        pair<iterator, bool> insert(const ValueType& value) { return insert<KeyType, ValueType, IdentityTranslatorType>(ExtractKey(value), value); }
+        pair<iterator, bool> add(const ValueType& value) { return add<KeyType, ValueType, IdentityTranslatorType>(ExtractKey(value), value); }
 
-        // A special version of insert() that finds the object by hashing and comparing
+        // A special version of add() that finds the object by hashing and comparing
         // with some other type, to avoid the cost of type conversion if the object is already
         // in the table.
-        template<typename T, typename Extra, typename HashTranslator> pair<iterator, bool> insert(const T& key, const Extra&);
+        template<typename T, typename Extra, typename HashTranslator> pair<iterator, bool> add(const T& key, const Extra&);
 
         iterator find(const KeyType&);
         const_iterator find(const KeyType&) const;
@@ -415,7 +415,7 @@ namespace KXMLCore {
 
     template<typename Key, typename Value, const Key& ExtractKey(const Value&), typename HashFunctions, typename Traits, typename KeyTraits>
     template<typename T, typename Extra, typename HashTranslator>
-    inline pair<typename HashTable<Key, Value, ExtractKey, HashFunctions, Traits, KeyTraits>::iterator, bool> HashTable<Key, Value, ExtractKey, HashFunctions, Traits, KeyTraits>::insert(const T& key, const Extra &extra)
+    inline pair<typename HashTable<Key, Value, ExtractKey, HashFunctions, Traits, KeyTraits>::iterator, bool> HashTable<Key, Value, ExtractKey, HashFunctions, Traits, KeyTraits>::add(const T& key, const Extra &extra)
     {
         invalidateIterators();
 
@@ -631,11 +631,11 @@ namespace KXMLCore {
         , m_iterators(0)
 #endif
     {
-        // Copy the hash table the dumb way, by inserting each element into the new table.
+        // Copy the hash table the dumb way, by adding each element to the new table.
         // It might be more efficient to copy the table slots, but it's not clear that efficiency is needed.
         const_iterator end = other.end();
         for (const_iterator it = other.begin(); it != end; ++it)
-            insert(*it);
+            add(*it);
     }
 
     template<typename Key, typename Value, const Key& ExtractKey(const Value&), typename HashFunctions, typename Traits, typename KeyTraits>
index 513b70bae9259c648ff8bbc1a413bce7ebdbc979..5d679e498a6cb7960071b0743634fd718dad8e95 100644 (file)
@@ -1,3 +1,46 @@
+2006-01-23  Maciej Stachowiak  <mjs@apple.com>
+
+        Rubber stamped by Tim Hatcher.
+        
+        - renamed inert() operation on HashSet, HashCountedSet and HashTable to add()
+        for consistency with HashMap
+
+        * bridge/mac/MacFrame.mm:
+        (MacFrame::didTellBridgeAboutLoad):
+        * khtml/ecma/kjs_dom.cpp:
+        (KJS::DOMNode::mark):
+        * khtml/html/HTMLElementImpl.cpp:
+        (WebCore::HTMLElementImpl::isRecognizedTagName):
+        (WebCore::inlineTagList):
+        (WebCore::blockTagList):
+        * khtml/html/HTMLFormCollectionImpl.cpp:
+        (WebCore::HTMLFormCollectionImpl::updateNameCache):
+        * khtml/html/htmlparser.cpp:
+        (HTMLParser::isHeaderTag):
+        (HTMLParser::isResidualStyleTag):
+        (HTMLParser::isAffectedByResidualStyle):
+        * khtml/xml/DocumentImpl.cpp:
+        (WebCore::DocumentImpl::addElementById):
+        (WebCore::DocumentImpl::registerDisconnectedNodeWithEventListeners):
+        * khtml/xml/NodeImpl.cpp:
+        (WebCore::NodeImpl::registerNodeList):
+        * khtml/xml/dom_atomicstring.cpp:
+        (DOM::AtomicString::add):
+        * khtml/xml/dom_qname.cpp:
+        (DOM::QualifiedName::QualifiedName):
+        * loader/CachedObject.cpp:
+        (WebCore::CachedObject::ref):
+        * page/Frame.cpp:
+        (Frame::keepAlive):
+        * rendering/render_canvas.cpp:
+        (RenderCanvas::addWidget):
+        * rendering/render_frames.cpp:
+        (WebCore::RenderPartObject::updateWidget):
+        * rendering/render_line.cpp:
+        (WebCore::InlineFlowBox::paint):
+        * xml/xmlhttprequest.cpp:
+        (WebCore::XMLHttpRequest::addToRequestsByDocument):
+
 2006-01-23  Justin Garcia  <justin.garcia@apple.com>
 
         Reviewed by thatcher
index b950cf0e3f958a7c418ea2d27823b90a76298f6d..ec4bf56627485ff46e787e72a38b1609e57e28de 100644 (file)
@@ -3095,7 +3095,7 @@ WebCoreKeyboardUIMode MacFrame::keyboardUIMode() const
 
 void MacFrame::didTellBridgeAboutLoad(const DOM::DOMString& URL)
 {
-    urlsBridgeKnowsAbout.insert(URL.impl());
+    urlsBridgeKnowsAbout.add(URL.impl());
 }
 
 bool MacFrame::haveToldBridgeAboutLoad(const DOM::DOMString& URL)
index 3bcd5f08bfca16ed0f6744412594d441032d6e91..8087d563ca9f6185a5aa512f3640c129c26bff9f 100644 (file)
@@ -142,7 +142,7 @@ void DOMNode::mark()
   DocumentImpl *document = m_impl->getDocument();
 
   // Mark the whole tree; use the global set of roots to avoid reentering.
-  markingRoots.insert(root);
+  markingRoots.add(root);
   for (NodeImpl *nodeToMark = root; nodeToMark; nodeToMark = nodeToMark->traverseNextNode()) {
     DOMNode *wrapper = ScriptInterpreter::getDOMNodeForDocument(document, nodeToMark);
     if (wrapper) {
index d0e573348ecbf2b2cff942c92cb1b4d1e8ca38b5..64f54704d18a048686f276bff8d25abe09173f09 100644 (file)
@@ -651,8 +651,8 @@ bool HTMLElementImpl::isRecognizedTagName(const QualifiedName& tagName)
 {
     static HashSet<DOMStringImpl*, PointerHash<DOMStringImpl*> > tagList;
     if (tagList.isEmpty()) {
-        #define INSERT_TAG(name) tagList.insert(name##Tag.localName().impl());
-        DOM_HTMLNAMES_FOR_EACH_TAG(INSERT_TAG)
+        #define ADD_TAG(name) tagList.add(name##Tag.localName().impl());
+        DOM_HTMLNAMES_FOR_EACH_TAG(ADD_TAG)
     }
     return tagList.contains(tagName.localName().impl());
 }
@@ -663,51 +663,51 @@ HashSet<DOMStringImpl*, PointerHash<DOMStringImpl*> >* inlineTagList()
 {
     static HashSet<DOMStringImpl*, PointerHash<DOMStringImpl*> > tagList;
     if (tagList.isEmpty()) {
-        tagList.insert(ttTag.localName().impl());
-        tagList.insert(iTag.localName().impl());
-        tagList.insert(bTag.localName().impl());
-        tagList.insert(uTag.localName().impl());
-        tagList.insert(sTag.localName().impl());
-        tagList.insert(strikeTag.localName().impl());
-        tagList.insert(bigTag.localName().impl());
-        tagList.insert(smallTag.localName().impl());
-        tagList.insert(emTag.localName().impl());
-        tagList.insert(strongTag.localName().impl());
-        tagList.insert(dfnTag.localName().impl());
-        tagList.insert(codeTag.localName().impl());
-        tagList.insert(sampTag.localName().impl());
-        tagList.insert(kbdTag.localName().impl());
-        tagList.insert(varTag.localName().impl());
-        tagList.insert(citeTag.localName().impl());
-        tagList.insert(abbrTag.localName().impl());
-        tagList.insert(acronymTag.localName().impl());
-        tagList.insert(aTag.localName().impl());
-        tagList.insert(canvasTag.localName().impl());
-        tagList.insert(imgTag.localName().impl());
-        tagList.insert(appletTag.localName().impl());
-        tagList.insert(objectTag.localName().impl());
-        tagList.insert(embedTag.localName().impl());
-        tagList.insert(fontTag.localName().impl());
-        tagList.insert(basefontTag.localName().impl());
-        tagList.insert(brTag.localName().impl());
-        tagList.insert(scriptTag.localName().impl());
-        tagList.insert(mapTag.localName().impl());
-        tagList.insert(qTag.localName().impl());
-        tagList.insert(subTag.localName().impl());
-        tagList.insert(supTag.localName().impl());
-        tagList.insert(spanTag.localName().impl());
-        tagList.insert(bdoTag.localName().impl());
-        tagList.insert(iframeTag.localName().impl());
-        tagList.insert(inputTag.localName().impl());
-        tagList.insert(keygenTag.localName().impl());
-        tagList.insert(selectTag.localName().impl());
-        tagList.insert(textareaTag.localName().impl());
-        tagList.insert(labelTag.localName().impl());
-        tagList.insert(buttonTag.localName().impl());
-        tagList.insert(insTag.localName().impl());
-        tagList.insert(delTag.localName().impl());
-        tagList.insert(nobrTag.localName().impl());
-        tagList.insert(wbrTag.localName().impl());
+        tagList.add(ttTag.localName().impl());
+        tagList.add(iTag.localName().impl());
+        tagList.add(bTag.localName().impl());
+        tagList.add(uTag.localName().impl());
+        tagList.add(sTag.localName().impl());
+        tagList.add(strikeTag.localName().impl());
+        tagList.add(bigTag.localName().impl());
+        tagList.add(smallTag.localName().impl());
+        tagList.add(emTag.localName().impl());
+        tagList.add(strongTag.localName().impl());
+        tagList.add(dfnTag.localName().impl());
+        tagList.add(codeTag.localName().impl());
+        tagList.add(sampTag.localName().impl());
+        tagList.add(kbdTag.localName().impl());
+        tagList.add(varTag.localName().impl());
+        tagList.add(citeTag.localName().impl());
+        tagList.add(abbrTag.localName().impl());
+        tagList.add(acronymTag.localName().impl());
+        tagList.add(aTag.localName().impl());
+        tagList.add(canvasTag.localName().impl());
+        tagList.add(imgTag.localName().impl());
+        tagList.add(appletTag.localName().impl());
+        tagList.add(objectTag.localName().impl());
+        tagList.add(embedTag.localName().impl());
+        tagList.add(fontTag.localName().impl());
+        tagList.add(basefontTag.localName().impl());
+        tagList.add(brTag.localName().impl());
+        tagList.add(scriptTag.localName().impl());
+        tagList.add(mapTag.localName().impl());
+        tagList.add(qTag.localName().impl());
+        tagList.add(subTag.localName().impl());
+        tagList.add(supTag.localName().impl());
+        tagList.add(spanTag.localName().impl());
+        tagList.add(bdoTag.localName().impl());
+        tagList.add(iframeTag.localName().impl());
+        tagList.add(inputTag.localName().impl());
+        tagList.add(keygenTag.localName().impl());
+        tagList.add(selectTag.localName().impl());
+        tagList.add(textareaTag.localName().impl());
+        tagList.add(labelTag.localName().impl());
+        tagList.add(buttonTag.localName().impl());
+        tagList.add(insTag.localName().impl());
+        tagList.add(delTag.localName().impl());
+        tagList.add(nobrTag.localName().impl());
+        tagList.add(wbrTag.localName().impl());
     }
     return &tagList;
 }
@@ -716,39 +716,39 @@ HashSet<DOMStringImpl*, PointerHash<DOMStringImpl*> >* blockTagList()
 {
     static HashSet<DOMStringImpl*, PointerHash<DOMStringImpl*> > tagList;
     if (tagList.isEmpty()) {
-        tagList.insert(pTag.localName().impl());
-        tagList.insert(h1Tag.localName().impl());
-        tagList.insert(h2Tag.localName().impl());
-        tagList.insert(h3Tag.localName().impl());
-        tagList.insert(h4Tag.localName().impl());
-        tagList.insert(h5Tag.localName().impl());
-        tagList.insert(h6Tag.localName().impl());
-        tagList.insert(ulTag.localName().impl());
-        tagList.insert(olTag.localName().impl());
-        tagList.insert(dirTag.localName().impl());
-        tagList.insert(menuTag.localName().impl());
-        tagList.insert(preTag.localName().impl());
-        tagList.insert(plaintextTag.localName().impl());
-        tagList.insert(xmpTag.localName().impl());
-        tagList.insert(dlTag.localName().impl());
-        tagList.insert(divTag.localName().impl());
-        tagList.insert(layerTag.localName().impl());
-        tagList.insert(centerTag.localName().impl());
-        tagList.insert(noscriptTag.localName().impl());
-        tagList.insert(noframesTag.localName().impl());
-        tagList.insert(noembedTag.localName().impl());
-        tagList.insert(nolayerTag.localName().impl());
-        tagList.insert(blockquoteTag.localName().impl());
-        tagList.insert(formTag.localName().impl());
-        tagList.insert(isindexTag.localName().impl());
-        tagList.insert(hrTag.localName().impl());
-        tagList.insert(tableTag.localName().impl());
-        tagList.insert(fieldsetTag.localName().impl());
-        tagList.insert(addressTag.localName().impl());
-        tagList.insert(liTag.localName().impl());
-        tagList.insert(ddTag.localName().impl());
-        tagList.insert(dtTag.localName().impl());
-        tagList.insert(marqueeTag.localName().impl());
+        tagList.add(pTag.localName().impl());
+        tagList.add(h1Tag.localName().impl());
+        tagList.add(h2Tag.localName().impl());
+        tagList.add(h3Tag.localName().impl());
+        tagList.add(h4Tag.localName().impl());
+        tagList.add(h5Tag.localName().impl());
+        tagList.add(h6Tag.localName().impl());
+        tagList.add(ulTag.localName().impl());
+        tagList.add(olTag.localName().impl());
+        tagList.add(dirTag.localName().impl());
+        tagList.add(menuTag.localName().impl());
+        tagList.add(preTag.localName().impl());
+        tagList.add(plaintextTag.localName().impl());
+        tagList.add(xmpTag.localName().impl());
+        tagList.add(dlTag.localName().impl());
+        tagList.add(divTag.localName().impl());
+        tagList.add(layerTag.localName().impl());
+        tagList.add(centerTag.localName().impl());
+        tagList.add(noscriptTag.localName().impl());
+        tagList.add(noframesTag.localName().impl());
+        tagList.add(noembedTag.localName().impl());
+        tagList.add(nolayerTag.localName().impl());
+        tagList.add(blockquoteTag.localName().impl());
+        tagList.add(formTag.localName().impl());
+        tagList.add(isindexTag.localName().impl());
+        tagList.add(hrTag.localName().impl());
+        tagList.add(tableTag.localName().impl());
+        tagList.add(fieldsetTag.localName().impl());
+        tagList.add(addressTag.localName().impl());
+        tagList.add(liTag.localName().impl());
+        tagList.add(ddTag.localName().impl());
+        tagList.add(dtTag.localName().impl());
+        tagList.add(marqueeTag.localName().impl());
     }
     return &tagList;
 }
index d6a08b185b6568f36dc47f3c04857ef5cabb7055..9ee6745042f226e46b4a88860059556181ca0c21 100644 (file)
@@ -249,7 +249,7 @@ void HTMLFormCollectionImpl::updateNameCache() const
                     info->idCache.add(idAttrVal.impl(), idVector);
                 }
                 appendToVector(idVector, static_cast<NodeImpl *>(e));
-                foundInputElements.insert(idAttrVal.impl());
+                foundInputElements.add(idAttrVal.impl());
             }
             if (!nameAttrVal.isEmpty() && idAttrVal != nameAttrVal) {
                 // add to name cache
@@ -259,7 +259,7 @@ void HTMLFormCollectionImpl::updateNameCache() const
                     info->nameCache.add(nameAttrVal.impl(), nameVector);
                 }
                 appendToVector(nameVector, static_cast<NodeImpl *>(e));
-                foundInputElements.insert(nameAttrVal.impl());
+                foundInputElements.add(nameAttrVal.impl());
             }
         }
     }
index 5ad5c08f1dc058f73ef9563898dc097895e19ab8..bdf68a7f4b4d9b2e5a11e2a25d49d25c06fa08c7 100644 (file)
@@ -837,12 +837,12 @@ bool HTMLParser::isHeaderTag(const AtomicString& tagName)
 {
     static HashSet<DOMStringImpl*, PointerHash<DOMStringImpl*> > headerTags;
     if (headerTags.isEmpty()) {
-        headerTags.insert(h1Tag.localName().impl());
-        headerTags.insert(h2Tag.localName().impl());
-        headerTags.insert(h3Tag.localName().impl());
-        headerTags.insert(h4Tag.localName().impl());
-        headerTags.insert(h5Tag.localName().impl());
-        headerTags.insert(h6Tag.localName().impl());
+        headerTags.add(h1Tag.localName().impl());
+        headerTags.add(h2Tag.localName().impl());
+        headerTags.add(h3Tag.localName().impl());
+        headerTags.add(h4Tag.localName().impl());
+        headerTags.add(h5Tag.localName().impl());
+        headerTags.add(h6Tag.localName().impl());
     }
     
     return headerTags.contains(tagName.impl());
@@ -889,25 +889,25 @@ bool HTMLParser::isResidualStyleTag(const AtomicString& tagName)
 {
     static HashSet<DOMStringImpl*, PointerHash<DOMStringImpl*> > residualStyleTags;
     if (residualStyleTags.isEmpty()) {
-        residualStyleTags.insert(aTag.localName().impl());
-        residualStyleTags.insert(fontTag.localName().impl());
-        residualStyleTags.insert(ttTag.localName().impl());
-        residualStyleTags.insert(uTag.localName().impl());
-        residualStyleTags.insert(bTag.localName().impl());
-        residualStyleTags.insert(iTag.localName().impl());
-        residualStyleTags.insert(sTag.localName().impl());
-        residualStyleTags.insert(strikeTag.localName().impl());
-        residualStyleTags.insert(bigTag.localName().impl());
-        residualStyleTags.insert(smallTag.localName().impl());
-        residualStyleTags.insert(emTag.localName().impl());
-        residualStyleTags.insert(strongTag.localName().impl());
-        residualStyleTags.insert(dfnTag.localName().impl());
-        residualStyleTags.insert(codeTag.localName().impl());
-        residualStyleTags.insert(sampTag.localName().impl());
-        residualStyleTags.insert(kbdTag.localName().impl());
-        residualStyleTags.insert(varTag.localName().impl());
-        residualStyleTags.insert(nobrTag.localName().impl());
-        residualStyleTags.insert(wbrTag.localName().impl());
+        residualStyleTags.add(aTag.localName().impl());
+        residualStyleTags.add(fontTag.localName().impl());
+        residualStyleTags.add(ttTag.localName().impl());
+        residualStyleTags.add(uTag.localName().impl());
+        residualStyleTags.add(bTag.localName().impl());
+        residualStyleTags.add(iTag.localName().impl());
+        residualStyleTags.add(sTag.localName().impl());
+        residualStyleTags.add(strikeTag.localName().impl());
+        residualStyleTags.add(bigTag.localName().impl());
+        residualStyleTags.add(smallTag.localName().impl());
+        residualStyleTags.add(emTag.localName().impl());
+        residualStyleTags.add(strongTag.localName().impl());
+        residualStyleTags.add(dfnTag.localName().impl());
+        residualStyleTags.add(codeTag.localName().impl());
+        residualStyleTags.add(sampTag.localName().impl());
+        residualStyleTags.add(kbdTag.localName().impl());
+        residualStyleTags.add(varTag.localName().impl());
+        residualStyleTags.add(nobrTag.localName().impl());
+        residualStyleTags.add(wbrTag.localName().impl());
     }
     
     return residualStyleTags.contains(tagName.impl());
@@ -920,24 +920,24 @@ bool HTMLParser::isAffectedByResidualStyle(const AtomicString& tagName)
 
     static HashSet<DOMStringImpl*, PointerHash<DOMStringImpl*> > affectedBlockTags;
     if (affectedBlockTags.isEmpty()) {
-        affectedBlockTags.insert(h1Tag.localName().impl());
-        affectedBlockTags.insert(h2Tag.localName().impl());
-        affectedBlockTags.insert(h3Tag.localName().impl());
-        affectedBlockTags.insert(h4Tag.localName().impl());
-        affectedBlockTags.insert(h5Tag.localName().impl());
-        affectedBlockTags.insert(h6Tag.localName().impl());
-        affectedBlockTags.insert(pTag.localName().impl());
-        affectedBlockTags.insert(divTag.localName().impl());
-        affectedBlockTags.insert(blockquoteTag.localName().impl());
-        affectedBlockTags.insert(addressTag.localName().impl());
-        affectedBlockTags.insert(centerTag.localName().impl());
-        affectedBlockTags.insert(ulTag.localName().impl());
-        affectedBlockTags.insert(olTag.localName().impl());
-        affectedBlockTags.insert(liTag.localName().impl());
-        affectedBlockTags.insert(dlTag.localName().impl());
-        affectedBlockTags.insert(dtTag.localName().impl());
-        affectedBlockTags.insert(ddTag.localName().impl());
-        affectedBlockTags.insert(preTag.localName().impl());
+        affectedBlockTags.add(h1Tag.localName().impl());
+        affectedBlockTags.add(h2Tag.localName().impl());
+        affectedBlockTags.add(h3Tag.localName().impl());
+        affectedBlockTags.add(h4Tag.localName().impl());
+        affectedBlockTags.add(h5Tag.localName().impl());
+        affectedBlockTags.add(h6Tag.localName().impl());
+        affectedBlockTags.add(pTag.localName().impl());
+        affectedBlockTags.add(divTag.localName().impl());
+        affectedBlockTags.add(blockquoteTag.localName().impl());
+        affectedBlockTags.add(addressTag.localName().impl());
+        affectedBlockTags.add(centerTag.localName().impl());
+        affectedBlockTags.add(ulTag.localName().impl());
+        affectedBlockTags.add(olTag.localName().impl());
+        affectedBlockTags.add(liTag.localName().impl());
+        affectedBlockTags.add(dlTag.localName().impl());
+        affectedBlockTags.add(dtTag.localName().impl());
+        affectedBlockTags.add(ddTag.localName().impl());
+        affectedBlockTags.add(preTag.localName().impl());
     }
     
     return affectedBlockTags.contains(tagName.impl());
index ad85a473cc74caa11c9b785b518e0c88fd993657..41ecd4fc5c37cec7b4bc415d90f43e3e0125addf 100644 (file)
@@ -649,7 +649,7 @@ void DocumentImpl::addElementById(const AtomicString& elementId, ElementImpl* el
     if (!m_elementsById.contains(elementId.impl()))
         m_elementsById.set(elementId.impl(), element);
     else
-        m_duplicateIds.insert(elementId.impl());
+        m_duplicateIds.add(elementId.impl());
 }
 
 void DocumentImpl::removeElementById(const AtomicString& elementId, ElementImpl* element)
@@ -999,7 +999,7 @@ void DocumentImpl::removeAllEventListenersFromAllNodes()
 
 void DocumentImpl::registerDisconnectedNodeWithEventListeners(NodeImpl* node)
 {
-    m_disconnectedNodesWithEventListeners.insert(node);
+    m_disconnectedNodesWithEventListeners.add(node);
 }
 
 void DocumentImpl::unregisterDisconnectedNodeWithEventListeners(NodeImpl* node)
index edebfe8c27a500c6a1edc5eb73b2f4075e5dfeed..d02b5c0870750ccc07af7e9221ec82469a7d6762 100644 (file)
@@ -807,7 +807,7 @@ void NodeImpl::registerNodeList(NodeListImpl* list)
 {
     if (!m_nodeLists)
         m_nodeLists = new NodeListSet;
-    m_nodeLists->insert(list);
+    m_nodeLists->add(list);
 }
 
 void NodeImpl::unregisterNodeList(NodeListImpl* list)
index ef4667f8ed793a8855ffabf0b4624c2cce052a67..ed3162af74fd18696fda26e560046453bab9dd9f 100644 (file)
@@ -82,7 +82,7 @@ DOMStringImpl *AtomicString::add(const char *c)
     if (length == 0)
         return DOMStringImpl::empty();
     
-    return *stringTable.insert<const char *, CStringTranslator>(c).first;
+    return *stringTable.add<const char *, CStringTranslator>(c).first;
 }
 
 struct QCharBuffer {
@@ -139,7 +139,7 @@ DOMStringImpl *AtomicString::add(const QChar *s, int length)
         return DOMStringImpl::empty();
     
     QCharBuffer buf = {s, length}; 
-    return *stringTable.insert<QCharBuffer, QCharBufferTranslator>(buf).first;
+    return *stringTable.add<QCharBuffer, QCharBufferTranslator>(buf).first;
 }
 
 DOMStringImpl *AtomicString::add(DOMStringImpl *r)
@@ -150,7 +150,7 @@ DOMStringImpl *AtomicString::add(DOMStringImpl *r)
     if (r->l == 0)
         return DOMStringImpl::empty();
     
-    DOMStringImpl *result = *stringTable.insert(r).first;
+    DOMStringImpl *result = *stringTable.add(r).first;
     if (result == r)
         r->_inTable = true;
     return result;
index 1b07fa9821a340937b72d7e797cde984093cf611..07ca4fd4abca3612b414dbc3c5f39eee7ba88588 100644 (file)
@@ -117,7 +117,7 @@ QualifiedName::QualifiedName(const AtomicString& p, const AtomicString& l, const
     if (!gNameCache)
         gNameCache = new QNameSet;
     QualifiedNameComponents components = { p.impl(), l.impl(), n.impl() };
-    m_impl = *gNameCache->insert<QualifiedNameComponents, QNameComponentsTranslator>(components).first;    
+    m_impl = *gNameCache->add<QualifiedNameComponents, QNameComponentsTranslator>(components).first;    
     ref();
 }
 
index 426aa53c348afce72b4ab899af27b3c97b6e96a3..7b4d276b84e5581f321974da05e99593f080a677 100644 (file)
@@ -87,7 +87,7 @@ void CachedObject::setRequest(Request *_request)
 
 void CachedObject::ref(CachedObjectClient *c)
 {
-    m_clients.insert(c);
+    m_clients.add(c);
     Cache::removeFromLRUList(this);
     increaseAccessCount();
 }
index c14f3b6e84850e8c0543e55df24f3cefe5860ed8..9e6b5f70de19a6e0e08686220cfed9452f078c82 100644 (file)
@@ -3016,7 +3016,7 @@ void Frame::keepAlive()
     ref();
     d->m_lifeSupportTimer.start(0, true);
 #ifndef NDEBUG
-    lifeSupportSet.insert(this);
+    lifeSupportSet.add(this);
 #endif
 }
 
index 25789162aefe75f6759f154f801691a1f883584b..3494124d0532a9a97c018bdc6eb2ef61bdd57de3 100644 (file)
@@ -564,7 +564,7 @@ void RenderCanvas::updateWidgetPositions()
 
 void RenderCanvas::addWidget(RenderObject *o)
 {
-    m_widgets.insert(o);
+    m_widgets.add(o);
 }
 
 void RenderCanvas::removeWidget(RenderObject *o)
index aea087d5d2f4357d05581845032bd154b81df0c8..13841bdf7b0e0781d376241d2bf6494a8fd4ee74 100644 (file)
@@ -851,7 +851,7 @@ void RenderPartObject::updateWidget()
                   }
               }
               if (!embed) {
-                  uniqueParamNames.insert(p->name().impl());
+                  uniqueParamNames.add(p->name().impl());
                   paramNames.append(p->name().qstring());
                   paramValues.append(p->value().qstring());
               }
@@ -867,7 +867,7 @@ void RenderPartObject::updateWidget()
       DOMString codebase;
       if (!embed && serviceType.lower() == "application/x-java-applet") {
           codebase = "codebase";
-          uniqueParamNames.insert(codebase.impl()); // pretend we found it in a PARAM already
+          uniqueParamNames.add(codebase.impl()); // pretend we found it in a PARAM already
       }
       
       // Turn the attributes of either the EMBED tag or OBJECT tag into arrays, but don't override PARAM values.
index 8099907a66ea8d42e34b6ccf1a504dd3a6e81c48..80cea08565b8c7983a17585db0cc811a1954a2cc 100644 (file)
@@ -760,7 +760,7 @@ void InlineFlowBox::paint(RenderObject::PaintInfo& i, int tx, int ty)
             // outlines.
             if (object()->style()->visibility() == VISIBLE && object()->style()->outlineWidth() > 0 &&
                 !object()->isInlineContinuation() && !isRootInlineBox()) {
-                i.outlineObjects.insert(flowObject());
+                i.outlineObjects.add(flowObject());
             }
         }
         else {
index c8ddb8d4a49ee96da1e957fa79d061dd1ffde2e8..d1c28bcfb5e1d460856067ce72a84350ccdffc30 100644 (file)
@@ -567,7 +567,7 @@ void XMLHttpRequest::addToRequestsByDocument()
   }
 
   assert(!requests->contains(this));
-  requests->insert(this);
+  requests->add(this);
 }
 
 void XMLHttpRequest::removeFromRequestsByDocument()