Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(712)

Unified Diff: Source/core/dom/ClassNodeList.h

Issue 112843002: Make calls to AtomicString(const String&) explicit in dom/ (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Added comments Created 7 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: Source/core/dom/ClassNodeList.h
diff --git a/Source/core/dom/ClassNodeList.h b/Source/core/dom/ClassNodeList.h
index 2e08ea819288c606479a0d1664b800acdefecfd3..a1dcbf4571846cbd4414f487557a0c0ab1e6c7e1 100644
--- a/Source/core/dom/ClassNodeList.h
+++ b/Source/core/dom/ClassNodeList.h
@@ -39,8 +39,11 @@ namespace WebCore {
class ClassNodeList FINAL : public LiveNodeList {
public:
- static PassRefPtr<ClassNodeList> create(PassRefPtr<Node> rootNode, const String& classNames)
+ // classNames argument is an AtomicString because it is common for Elements to share the same class names.
+ // It is also used to construct a SpaceSplitString (m_classNames) and its constructor requires an AtomicString.
+ static PassRefPtr<ClassNodeList> create(PassRefPtr<Node> rootNode, CollectionType type, const AtomicString& classNames)
{
+ ASSERT_UNUSED(type, type == ClassNodeListType);
return adoptRef(new ClassNodeList(rootNode, classNames));
}
@@ -49,12 +52,12 @@ public:
bool nodeMatchesInlined(Element*) const;
private:
- ClassNodeList(PassRefPtr<Node> rootNode, const String& classNames);
+ ClassNodeList(PassRefPtr<Node> rootNode, const AtomicString& classNames);
virtual bool nodeMatches(Element*) const OVERRIDE;
SpaceSplitString m_classNames;
- String m_originalClassNames;
+ AtomicString m_originalClassNames;
};
inline bool ClassNodeList::nodeMatchesInlined(Element* testNode) const

Powered by Google App Engine
This is Rietveld 408576698