Index: third_party/WebKit/Source/platform/heap/Heap.h |
diff --git a/third_party/WebKit/Source/platform/heap/Heap.h b/third_party/WebKit/Source/platform/heap/Heap.h |
index 40482db58d209521ff8e441509afa93a356ac2cb..42256b33d4d725d82d0d933137d720549ec5588d 100644 |
--- a/third_party/WebKit/Source/platform/heap/Heap.h |
+++ b/third_party/WebKit/Source/platform/heap/Heap.h |
@@ -202,6 +202,11 @@ public: |
// always 'alive'. |
if (!object) |
return true; |
+ // TODO(keishi): some tests create CrossThreadPersistent on non attached threads. |
keishi
2016/05/26 04:33:02
The tests that require this are:
ExtensionApiTest.
haraken
2016/05/26 07:52:29
Conceptually it should be valid to support CrossTh
keishi
2016/05/26 11:52:54
OK. Leaving the TODO comment.
|
+ if (!ThreadState::current()) |
+ return true; |
+ if (&ThreadState::current()->heap() != &pageFromObject(object)->arena()->getThreadState()->heap()) |
+ return true; |
haraken
2016/05/25 15:41:22
Would you help me understand why we need to add th
keishi
2016/05/26 04:29:38
Added this while debugging but it looks like it wa
|
return ObjectAliveTrait<T>::isHeapObjectAlive(object); |
} |
template<typename T> |