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

Issues Search
    My Issues | Starred     Open | Closed | All

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

Issue 18196004: Revert 153294 "Remove AtomicStringImpl." (Closed) Base URL: svn://svn.chromium.org/blink/
Patch Set: Created 7 years, 6 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 | « trunk/Source/core/dom/CheckedRadioButtons.h ('k') | trunk/Source/core/dom/DocumentOrderedMap.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: trunk/Source/core/dom/CheckedRadioButtons.cpp
===================================================================
--- trunk/Source/core/dom/CheckedRadioButtons.cpp (revision 153372)
+++ trunk/Source/core/dom/CheckedRadioButtons.cpp (working copy)
@@ -260,7 +260,7 @@
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 StringImpl* to RefPtr<StringImpl>.
+ // of m_nameToGroupMap from AtomicStringImpl* to RefPtr<AtomicStringImpl>.
m_nameToGroupMap->remove(it);
if (m_nameToGroupMap->isEmpty())
m_nameToGroupMap.clear();
Property changes on: trunk/Source/core/dom/CheckedRadioButtons.cpp
___________________________________________________________________
Modified: svn:eol-style
- LF
+ native
« no previous file with comments | « trunk/Source/core/dom/CheckedRadioButtons.h ('k') | trunk/Source/core/dom/DocumentOrderedMap.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698