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

Unified Diff: third_party/WebKit/Source/core/html/forms/RadioButtonGroupScope.cpp

Issue 1686483002: Oilpan: Remove most WillBe types from the code base (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 4 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 side-by-side diff with in-line comments
Download patch
Index: third_party/WebKit/Source/core/html/forms/RadioButtonGroupScope.cpp
diff --git a/third_party/WebKit/Source/core/html/forms/RadioButtonGroupScope.cpp b/third_party/WebKit/Source/core/html/forms/RadioButtonGroupScope.cpp
index c60402ae16ec7684cbee795cec2d332b544be582..66c7e175efbd600999bbc7a3c06185b7b685e875 100644
--- a/third_party/WebKit/Source/core/html/forms/RadioButtonGroupScope.cpp
+++ b/third_party/WebKit/Source/core/html/forms/RadioButtonGroupScope.cpp
@@ -26,10 +26,9 @@
namespace blink {
-class RadioButtonGroup : public NoBaseWillBeGarbageCollected<RadioButtonGroup> {
- USING_FAST_MALLOC_WILL_BE_REMOVED(RadioButtonGroup);
+class RadioButtonGroup : public GarbageCollected<RadioButtonGroup> {
public:
- static PassOwnPtrWillBeRawPtr<RadioButtonGroup> create();
+ static RawPtr<RadioButtonGroup> create();
bool isEmpty() const { return m_members.isEmpty(); }
bool isRequired() const { return m_requiredCount; }
HTMLInputElement* checkedButton() const { return m_checkedButton; }
@@ -49,7 +48,7 @@ private:
void setCheckedButton(HTMLInputElement*);
// The map records the 'required' state of each (button) element.
- using Members = WillBeHeapHashMap<RawPtrWillBeMember<HTMLInputElement>, bool>;
+ using Members = HeapHashMap<Member<HTMLInputElement>, bool>;
#if ENABLE(OILPAN)
using MemberKeyValue = WTF::KeyValuePair<Member<HTMLInputElement>, bool>;
@@ -60,7 +59,7 @@ private:
void updateRequiredButton(MemberKeyValue&, bool isRequired);
Members m_members;
- RawPtrWillBeMember<HTMLInputElement> m_checkedButton;
+ Member<HTMLInputElement> m_checkedButton;
size_t m_requiredCount;
};
@@ -70,7 +69,7 @@ RadioButtonGroup::RadioButtonGroup()
{
}
-PassOwnPtrWillBeRawPtr<RadioButtonGroup> RadioButtonGroup::create()
+RawPtr<RadioButtonGroup> RadioButtonGroup::create()
{
return adoptPtrWillBeNoop(new RadioButtonGroup);
}
@@ -239,7 +238,7 @@ void RadioButtonGroupScope::addButton(HTMLInputElement* element)
if (!m_nameToGroupMap)
m_nameToGroupMap = adoptPtrWillBeNoop(new NameToGroupMap);
- OwnPtrWillBeMember<RadioButtonGroup>& group = m_nameToGroupMap->add(element->name(), nullptr).storedValue->value;
+ Member<RadioButtonGroup>& group = m_nameToGroupMap->add(element->name(), nullptr).storedValue->value;
if (!group)
group = RadioButtonGroup::create();
group->add(element);

Powered by Google App Engine
This is Rietveld 408576698