Chromium Code Reviews| Index: Source/core/dom/DocumentInit.h |
| diff --git a/Source/core/dom/DocumentInit.h b/Source/core/dom/DocumentInit.h |
| index ffe4e9293cacc1d16874a681048dbddc7b441bdf..3458f02678ee6ba92ed2c26341e383c5e82261ae 100644 |
| --- a/Source/core/dom/DocumentInit.h |
| +++ b/Source/core/dom/DocumentInit.h |
| @@ -47,7 +47,7 @@ class Settings; |
| class DocumentInit FINAL { |
| STACK_ALLOCATED(); |
| public: |
| - explicit DocumentInit(const KURL& = KURL(), LocalFrame* = 0, WeakPtr<Document> = WeakPtr<Document>(), HTMLImportsController* = 0); |
| + explicit DocumentInit(const KURL& = KURL(), LocalFrame* = 0, WeakPtrWillBeRawPtr<Document> = nullptr, HTMLImportsController* = 0); |
|
tkent
2014/06/02 07:56:07
WeakPtr(std::nullptr_t) is necessary because RawPt
sof
2014/06/02 08:01:34
Yes, exactly so. As RawPtr() (for the Oilpan case)
|
| DocumentInit(const DocumentInit&); |
| ~DocumentInit(); |
| @@ -70,18 +70,18 @@ public: |
| DocumentInit& withRegistrationContext(CustomElementRegistrationContext*); |
| DocumentInit& withNewRegistrationContext(); |
| PassRefPtrWillBeRawPtr<CustomElementRegistrationContext> registrationContext(Document*) const; |
| - WeakPtr<Document> contextDocument() const; |
| + WeakPtrWillBeRawPtr<Document> contextDocument() const; |
| - static DocumentInit fromContext(WeakPtr<Document> contextDocument, const KURL& = KURL()); |
| + static DocumentInit fromContext(WeakPtrWillBeRawPtr<Document> contextDocument, const KURL& = KURL()); |
| private: |
| LocalFrame* frameForSecurityContext() const; |
| KURL m_url; |
| LocalFrame* m_frame; |
| - RefPtr<Document> m_parent; |
| - RefPtr<Document> m_owner; |
| - WeakPtr<Document> m_contextDocument; |
| + RefPtrWillBeMember<Document> m_parent; |
| + RefPtrWillBeMember<Document> m_owner; |
| + WeakPtrWillBeMember<Document> m_contextDocument; |
| RawPtrWillBeMember<HTMLImportsController> m_importsController; |
| RefPtrWillBeMember<CustomElementRegistrationContext> m_registrationContext; |
| bool m_createNewRegistrationContext; |