Oilpan: assert that new Persistents belong to Oilpan threads.

Oilpan: assert that new Persistents belong to Oilpan threads.

R=haraken
BUG=477577
Committed: https://src.chromium.org/viewvc/blink?view=rev&revision=193882

[haraken, 2015-04-16 07:57:02]

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

[haraken, 2015-04-16 07:57:03]

LGTM

[sof, 2015-04-16 11:08:46]

https://codereview.chromium.org/1089763002/diff/1/Source/platform/heap/Handle.h
File Source/platform/heap/Handle.h (right):

https://codereview.chromium.org/1089763002/diff/1/Source/platform/heap/Handle...
Source/platform/heap/Handle.h:166: ASSERT(ThreadState::current());
The assert in the dtor cannot be used, as the ThreadState may have been
detached&finalized by then.

The asserts during construction, along with the 'same roots' assert below
catches what we want to during destruction in any case.

[sof, 2015-04-16 14:35:20]

The CQ bit was checked by sigbjornf@opera.com

[sof, 2015-04-16 14:35:21]

The patchset sent to the CQ was uploaded after l-g-t-m from haraken@chromium.org
Link to the patchset: https://codereview.chromium.org/1089763002/#ps20001
(title: "remove assert in ~PersistentBase")

[commit-bot: I haz the power, 2015-04-16 14:35:46]

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

[commit-bot: I haz the power, 2015-04-16 16:02:05]

Committed patchset #2 (id:20001) as
https://src.chromium.org/viewvc/blink?view=rev&revision=193882