Oilpan: account for DocumentThreadableLoader::m_document back ref.

Oilpan: account for DocumentThreadableLoader::m_document back ref.

R=haraken
BUG=553613
Committed: https://crrev.com/7dd9d71a302d3633d451c37efd3615075bc2b805
Cr-Commit-Position: refs/heads/master@{#360025}

[sof, 2015-11-12 10:53:38]

sigbjornf@opera.com changed reviewers:
+ oilpan-reviews@chromium.org

[haraken, 2015-11-12 14:23:54]

haraken@chromium.org changed reviewers:
+ haraken@chromium.org

[haraken, 2015-11-12 14:23:54]

https://codereview.chromium.org/1435263002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/loader/DocumentThreadableLoader.h (right):

https://codereview.chromium.org/1435263002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/DocumentThreadableLoader.h:153:
RawPtrWillBeUntracedMember<Document> m_document;

So this should be WeakPersistent if it's hard to move ThreadableLoader to the
heap, IMHO.

[sof, 2015-11-16 21:16:18]

https://codereview.chromium.org/1435263002/diff/1/third_party/WebKit/Source/c...
File third_party/WebKit/Source/core/loader/DocumentThreadableLoader.h (right):

https://codereview.chromium.org/1435263002/diff/1/third_party/WebKit/Source/c...
third_party/WebKit/Source/core/loader/DocumentThreadableLoader.h:153:
RawPtrWillBeUntracedMember<Document> m_document;
On 2015/11/12 14:23:54, haraken wrote:
> 
> So this should be WeakPersistent if it's hard to move ThreadableLoader to the
> heap, IMHO.

WeakPersistent<> it is (for now.)

[haraken, 2015-11-16 23:27:47]

On 2015/11/16 21:16:18, sof wrote:
>
https://codereview.chromium.org/1435263002/diff/1/third_party/WebKit/Source/c...
> File third_party/WebKit/Source/core/loader/DocumentThreadableLoader.h (right):
> 
>
https://codereview.chromium.org/1435263002/diff/1/third_party/WebKit/Source/c...
> third_party/WebKit/Source/core/loader/DocumentThreadableLoader.h:153:
> RawPtrWillBeUntracedMember<Document> m_document;
> On 2015/11/12 14:23:54, haraken wrote:
> > 
> > So this should be WeakPersistent if it's hard to move ThreadableLoader to
the
> > heap, IMHO.
> 
> WeakPersistent<> it is (for now.)

LGTM

[sof, 2015-11-17 06:18:28]

Description was changed from

==========
Oilpan: account for DocumentThreadableLoader::m_document back ref.

R=
BUG=553613
==========

to

==========
Oilpan: account for DocumentThreadableLoader::m_document back ref.

R=haraken
BUG=553613
==========

[sof, 2015-11-17 06:18:34]

The CQ bit was checked by sigbjornf@opera.com

[commit-bot: I haz the power, 2015-11-17 06:19:35]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1435263002/20001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1435263002/20001

[commit-bot: I haz the power, 2015-11-17 06:48:34]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-11-17 06:49:27]

Patchset 2 (id:??) landed as
https://crrev.com/7dd9d71a302d3633d451c37efd3615075bc2b805
Cr-Commit-Position: refs/heads/master@{#360025}