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

Side by Side Diff: Source/WebCore/dom/CheckedRadioButtons.h

Issue 9805002: Revert 105710 - Introduce RadioButtonGroup class to keep track of the group members and required st… (Closed) Base URL: http://svn.webkit.org/repository/webkit/branches/chromium/1025/
Patch Set: Created 8 years, 9 months 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 unified diff | Download patch
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2007, 2008, 2009 Apple Inc. All rights reserved. 2 * Copyright (C) 2007, 2008, 2009 Apple Inc. All rights reserved.
3 * 3 *
4 * This library is free software; you can redistribute it and/or 4 * This library is free software; you can redistribute it and/or
5 * modify it under the terms of the GNU Library General Public 5 * modify it under the terms of the GNU Library General Public
6 * License as published by the Free Software Foundation; either 6 * License as published by the Free Software Foundation; either
7 * version 2 of the License, or (at your option) any later version. 7 * version 2 of the License, or (at your option) any later version.
8 * 8 *
9 * This library is distributed in the hope that it will be useful, 9 * This library is distributed in the hope that it will be useful,
10 * but WITHOUT ANY WARRANTY; without even the implied warranty of 10 * but WITHOUT ANY WARRANTY; without even the implied warranty of
(...skipping 10 matching lines...) Expand all
21 #ifndef CheckedRadioButtons_h 21 #ifndef CheckedRadioButtons_h
22 #define CheckedRadioButtons_h 22 #define CheckedRadioButtons_h
23 23
24 #include <wtf/Forward.h> 24 #include <wtf/Forward.h>
25 #include <wtf/HashMap.h> 25 #include <wtf/HashMap.h>
26 #include <wtf/OwnPtr.h> 26 #include <wtf/OwnPtr.h>
27 27
28 namespace WebCore { 28 namespace WebCore {
29 29
30 class HTMLInputElement; 30 class HTMLInputElement;
31 class RadioButtonGroup;
32 31
33 // FIXME: Rename the class. The class was a simple map from a name to a checked
34 // radio button. It manages RadioButtonGroup objects now.
35 class CheckedRadioButtons { 32 class CheckedRadioButtons {
36 public: 33 public:
37 CheckedRadioButtons();
38 ~CheckedRadioButtons();
39 void addButton(HTMLInputElement*); 34 void addButton(HTMLInputElement*);
40 void updateCheckedState(HTMLInputElement*);
41 void requiredAttributeChanged(HTMLInputElement*);
42 void removeButton(HTMLInputElement*); 35 void removeButton(HTMLInputElement*);
43 HTMLInputElement* checkedButtonForGroup(const AtomicString& groupName) const ; 36 HTMLInputElement* checkedButtonForGroup(const AtomicString& groupName) const ;
44 bool isInRequiredGroup(HTMLInputElement*) const;
45 37
46 private: 38 private:
47 typedef HashMap<AtomicStringImpl*, OwnPtr<RadioButtonGroup> > NameToGroupMap ; 39 typedef HashMap<AtomicStringImpl*, HTMLInputElement*> NameToInputMap;
48 OwnPtr<NameToGroupMap> m_nameToGroupMap; 40 OwnPtr<NameToInputMap> m_nameToCheckedRadioButtonMap;
49 }; 41 };
50 42
51 } // namespace WebCore 43 } // namespace WebCore
52 44
53 #endif // CheckedRadioButtons_h 45 #endif // CheckedRadioButtons_h
OLDNEW
« no previous file with comments | « LayoutTests/perf/adding-radio-buttons-expected.txt ('k') | Source/WebCore/dom/CheckedRadioButtons.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698