Chromium Code Reviews| Index: Source/core/html/ValidityState.h |
| diff --git a/Source/core/html/ValidityState.h b/Source/core/html/ValidityState.h |
| index 44d05b8cc8c2d5444259b7c11e4ee801fb82a168..829cc4e97a4f11009d5216889c7cfea89eddc8a1 100644 |
| --- a/Source/core/html/ValidityState.h |
| +++ b/Source/core/html/ValidityState.h |
| @@ -30,16 +30,20 @@ |
| namespace WebCore { |
| -class ValidityState : public ScriptWrappable { |
| - WTF_MAKE_NONCOPYABLE(ValidityState); WTF_MAKE_FAST_ALLOCATED; |
| +class ValidityState : public NoBaseWillBeGarbageCollectedFinalized<ValidityState>, public ScriptWrappable { |
|
zerny-chromium
2014/05/08 07:48:24
This can just be GarbageCollected<>
haraken
2014/05/08 08:00:33
Unfortunately no, due to ScriptWrappable :)
|
| + WTF_MAKE_NONCOPYABLE(ValidityState); |
| + WTF_MAKE_FAST_ALLOCATED_WILL_BE_REMOVED; |
| public: |
| - static PassOwnPtr<ValidityState> create(FormAssociatedElement* control) |
| + static PassOwnPtrWillBeRawPtr<ValidityState> create(FormAssociatedElement* control) |
| { |
| - return adoptPtr(new ValidityState(control)); |
| + return adoptPtrWillBeNoop(new ValidityState(control)); |
| } |
| + void trace(Visitor* visitor) { visitor->trace(m_control); } |
| +#if !ENABLE(OILPAN) |
| void ref() { m_control->ref(); } |
| void deref() { m_control->deref(); } |
| +#endif |
| String validationMessage() const; |
| @@ -57,12 +61,12 @@ public: |
| bool valid() const; |
| private: |
| - ValidityState(FormAssociatedElement* control) : m_control(control) |
| + explicit ValidityState(FormAssociatedElement* control) : m_control(control) |
| { |
| ScriptWrappable::init(this); |
| } |
| - FormAssociatedElement* m_control; |
| + RawPtrWillBeMember<FormAssociatedElement> m_control; |
| }; |
| } // namespace WebCore |