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

Unified Diff: Source/core/html/RadioNodeList.h

Issue 119063002: Have HTMLFormElement's named getter return a RadioNodeList. (Closed) Base URL: https://chromium.googlesource.com/chromium/blink.git@master
Patch Set: Test style adjustments + const'ify new RadioNodeList field. 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
« no previous file with comments | « Source/core/html/HTMLFormElement.idl ('k') | Source/core/html/RadioNodeList.cpp » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/html/RadioNodeList.h
diff --git a/Source/core/html/RadioNodeList.h b/Source/core/html/RadioNodeList.h
index b735b0c337725375cb1987026805aa19baa527a4..8a034a0df3ca3214c6ff791ce70d0b7580b2754b 100644
--- a/Source/core/html/RadioNodeList.h
+++ b/Source/core/html/RadioNodeList.h
@@ -36,8 +36,8 @@ class RadioNodeList FINAL : public LiveNodeList {
public:
static PassRefPtr<RadioNodeList> create(Node* ownerNode, CollectionType type, const AtomicString& name)
{
- ASSERT_UNUSED(type, type == RadioNodeListType);
- return adoptRef(new RadioNodeList(ownerNode, name));
+ ASSERT_UNUSED(type, type == RadioNodeListType || type == RadioImgNodeListType);
+ return adoptRef(new RadioNodeList(ownerNode, name, type));
}
virtual ~RadioNodeList();
@@ -46,15 +46,17 @@ public:
void setValue(const String&);
private:
- RadioNodeList(Node*, const AtomicString& name);
+ RadioNodeList(Node*, const AtomicString& name, CollectionType);
+
bool checkElementMatchesRadioNodeListFilter(Element*) const;
virtual bool nodeMatches(Element*) const OVERRIDE;
AtomicString m_name;
+ const bool m_onlyMatchImgElements;
};
-} // namepsace
+} // namespace
#endif
« no previous file with comments | « Source/core/html/HTMLFormElement.idl ('k') | Source/core/html/RadioNodeList.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698