Use more references instead of pointers in DocumentOrderedMap
Created attachment 213431 [details] Cleanup
Comment on attachment 213431 [details] Cleanup View in context: https://bugs.webkit.org/attachment.cgi?id=213431&action=review Super neato! r=me > Source/WebCore/bindings/js/JSHTMLDocumentCustom.cpp:76 > + Node* node = document->documentNamedItem(*atomicPropertyName); This should be an Element*. That'll remove a branch from hasTagName() below. > Source/WebCore/dom/DocumentOrderedMap.cpp:109 > + Map::iterator it = m_map.find(&key); I'd use auto for this. > Source/WebCore/dom/DocumentOrderedMap.cpp:130 > + Map::iterator it = m_map.find(&key); Ditto. > Source/WebCore/dom/DocumentOrderedMap.cpp:194 > + Map::iterator it = m_map.find(&key); Ditto. > Source/WebCore/dom/DocumentOrderedMap.h:96 > + Map::const_iterator it = m_map.find(&id); Ditto. > Source/WebCore/dom/DocumentOrderedMap.h:107 > + Map::const_iterator it = m_map.find(&id); Ditto.
Comment on attachment 213431 [details] Cleanup View in context: https://bugs.webkit.org/attachment.cgi?id=213431&action=review > Source/WebCore/dom/IdTargetObserverRegistry.h:67 > ASSERT(!m_notifyingObserversInSet); Oops, I need to assert that id is not empty here. > Source/WebCore/dom/TreeScope.cpp:244 > + return m_imageMapsByName->getElementByLowercasedMapName(*AtomicString(name.lower()).impl(), *this); > + return m_imageMapsByName->getElementByMapName(*AtomicString(name).impl(), *this); We should store AtomicString in a local variable here.
Committed r156940: <http://trac.webkit.org/changeset/156940>