Have DocumentOrderedMap API deal with AtomicString type instead of StringImpl*

Source/core/dom/DocumentOrderedMap.h

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010 Apple Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are
  * met:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 16 matching lines @@
  * (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
  * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 
 #ifndef DocumentOrderedMap_h
 #define DocumentOrderedMap_h
 
 #include "platform/heap/Handle.h"
 #include "wtf/Forward.h"
 #include "wtf/HashMap.h"
+#include "wtf/text/AtomicString.h"
 #include "wtf/text/StringImpl.h"
 
 namespace blink {
 
 class Element;
 class TreeScope;
 
 class DocumentOrderedMap : public NoBaseWillBeGarbageCollected<DocumentOrderedMap> {
 public:
     static PassOwnPtrWillBeRawPtr<DocumentOrderedMap> create();
-    void add(StringImpl*, Element*);
-    void remove(StringImpl*, Element*);
+    void add(const AtomicString&, Element*);
+    void remove(const AtomicString&, Element*);
 
-    bool contains(StringImpl*) const;
-    bool containsMultiple(StringImpl*) const;
+    bool contains(const AtomicString&) const;
+    bool containsMultiple(const AtomicString&) const;
     // concrete instantiations of the get<>() method template
-    Element* getElementById(StringImpl*, const TreeScope*) const;
-    const WillBeHeapVector<RawPtrWillBeMember<Element> >& getAllElementsById(StringImpl*, const TreeScope*) const;
-    Element* getElementByMapName(StringImpl*, const TreeScope*) const;
-    Element* getElementByLowercasedMapName(StringImpl*, const TreeScope*) const;
-    Element* getElementByLabelForAttribute(StringImpl*, const TreeScope*) const;
+    Element* getElementById(const AtomicString&, const TreeScope*) const;
+    const WillBeHeapVector<RawPtrWillBeMember<Element> >& getAllElementsById(const AtomicString&, const TreeScope*) const;
+    Element* getElementByMapName(const AtomicString&, const TreeScope*) const;
+    Element* getElementByLowercasedMapName(const AtomicString&, const TreeScope*) const;
+    Element* getElementByLabelForAttribute(const AtomicString&, const TreeScope*) const;
 
     void trace(Visitor*);
 
 private:
-    template<bool keyMatches(StringImpl*, Element&)> Element* get(StringImpl*, const TreeScope*) const;
+    template<bool keyMatches(const AtomicString&, const Element&)>
+    Element* get(const AtomicString&, const TreeScope*) const;
 
     class MapEntry : public NoBaseWillBeGarbageCollected<MapEntry> {
     public:
         explicit MapEntry(Element* firstElement)
             : element(firstElement)
             , count(1)
         {
         }
 
         void trace(Visitor*);
 
         RawPtrWillBeMember<Element> element;
         unsigned count;
         WillBeHeapVector<RawPtrWillBeMember<Element> > orderedList;
     };
 
     typedef WillBeHeapHashMap<StringImpl*, OwnPtrWillBeMember<MapEntry> > Map;
 
     mutable Map m_map;
 };
 
-inline bool DocumentOrderedMap::contains(StringImpl* id) const
+inline bool DocumentOrderedMap::contains(const AtomicString& id) const
 {
-    return m_map.contains(id);
+    return m_map.contains(id.impl());
 }
 
-inline bool DocumentOrderedMap::containsMultiple(StringImpl* id) const
+inline bool DocumentOrderedMap::containsMultiple(const AtomicString& id) const
 {
-    Map::const_iterator it = m_map.find(id);
+    Map::const_iterator it = m_map.find(id.impl());
     return it != m_map.end() && it->value->count > 1;
 }
 
 } // namespace blink
 
 #endif // DocumentOrderedMap_h