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

Unified Diff: Source/core/dom/CheckedRadioButtons.cpp

Issue 19804005: Remove AtomicStringImpl. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 7 years, 5 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « Source/core/dom/CheckedRadioButtons.h ('k') | Source/core/dom/DocumentOrderedMap.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Source/core/dom/CheckedRadioButtons.cpp
diff --git a/Source/core/dom/CheckedRadioButtons.cpp b/Source/core/dom/CheckedRadioButtons.cpp
index a628b439cfd2b6e88bb3c0062a6339583c40e243..d8d7a46d11324b619780933d27a877b4e6c5770a 100644
--- a/Source/core/dom/CheckedRadioButtons.cpp
+++ b/Source/core/dom/CheckedRadioButtons.cpp
@@ -260,7 +260,7 @@ void CheckedRadioButtons::removeButton(HTMLInputElement* element)
if (it->value->isEmpty()) {
// FIXME: We may skip deallocating the empty RadioButtonGroup for
// performance improvement. If we do so, we need to change the key type
- // of m_nameToGroupMap from AtomicStringImpl* to RefPtr<AtomicStringImpl>.
+ // of m_nameToGroupMap from StringImpl* to AtomicString.
m_nameToGroupMap->remove(it);
if (m_nameToGroupMap->isEmpty())
m_nameToGroupMap.clear();
« no previous file with comments | « Source/core/dom/CheckedRadioButtons.h ('k') | Source/core/dom/DocumentOrderedMap.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698