PR-URL: https://github.com/nodejs/node/pull/58070 Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com> Reviewed-By: Rafael Gonzaga <rafael.nunu@hotmail.com>
1090 lines
41 KiB
C++
1090 lines
41 KiB
C++
// Copyright 2018 the V8 project authors. All rights reserved.
|
|
// Use of this source code is governed by a BSD-style license that can be
|
|
// found in the LICENSE file.
|
|
|
|
#include "src/execution/isolate.h"
|
|
#include "src/handles/handles-inl.h"
|
|
#include "src/heap/factory-inl.h"
|
|
#include "src/heap/heap-inl.h"
|
|
#include "src/objects/js-objects.h"
|
|
#include "src/objects/js-weak-refs-inl.h"
|
|
#include "test/cctest/cctest.h"
|
|
#include "test/cctest/heap/heap-utils.h"
|
|
|
|
namespace v8 {
|
|
namespace internal {
|
|
|
|
namespace {
|
|
|
|
DirectHandle<JSFinalizationRegistry> ConstructJSFinalizationRegistry(
|
|
Isolate* isolate) {
|
|
Factory* factory = isolate->factory();
|
|
DirectHandle<String> finalization_registry_name =
|
|
factory->NewStringFromStaticChars("FinalizationRegistry");
|
|
DirectHandle<JSGlobalObject> global =
|
|
direct_handle(isolate->native_context()->global_object(), isolate);
|
|
DirectHandle<JSFunction> finalization_registry_fun = Cast<JSFunction>(
|
|
Object::GetProperty(isolate, global, finalization_registry_name)
|
|
.ToHandleChecked());
|
|
auto finalization_registry = Cast<JSFinalizationRegistry>(
|
|
JSObject::New(finalization_registry_fun, finalization_registry_fun,
|
|
Handle<AllocationSite>::null())
|
|
.ToHandleChecked());
|
|
|
|
// JSObject::New filled all of the internal fields with undefined. Some of
|
|
// them have more restrictive types, so set those now.
|
|
finalization_registry->set_native_context(*isolate->native_context());
|
|
finalization_registry->set_cleanup(
|
|
isolate->native_context()->empty_function());
|
|
finalization_registry->set_flags(0);
|
|
|
|
#ifdef VERIFY_HEAP
|
|
finalization_registry->JSFinalizationRegistryVerify(isolate);
|
|
#endif // VERIFY_HEAP
|
|
return finalization_registry;
|
|
}
|
|
|
|
Handle<JSWeakRef> ConstructJSWeakRef(DirectHandle<JSReceiver> target,
|
|
Isolate* isolate) {
|
|
Factory* factory = isolate->factory();
|
|
DirectHandle<String> weak_ref_name = factory->WeakRef_string();
|
|
DirectHandle<JSGlobalObject> global =
|
|
direct_handle(isolate->native_context()->global_object(), isolate);
|
|
DirectHandle<JSFunction> weak_ref_fun = Cast<JSFunction>(
|
|
Object::GetProperty(isolate, global, weak_ref_name).ToHandleChecked());
|
|
auto weak_ref = Cast<JSWeakRef>(
|
|
JSObject::New(weak_ref_fun, weak_ref_fun, Handle<AllocationSite>::null())
|
|
.ToHandleChecked());
|
|
weak_ref->set_target(*target);
|
|
#ifdef VERIFY_HEAP
|
|
weak_ref->JSWeakRefVerify(isolate);
|
|
#endif // VERIFY_HEAP
|
|
return weak_ref;
|
|
}
|
|
|
|
DirectHandle<JSObject> CreateKey(const char* key_prop_value, Isolate* isolate) {
|
|
Factory* factory = isolate->factory();
|
|
DirectHandle<String> key_string =
|
|
factory->NewStringFromStaticChars("key_string");
|
|
DirectHandle<JSObject> key =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
JSObject::AddProperty(isolate, key, key_string,
|
|
factory->NewStringFromAsciiChecked(key_prop_value),
|
|
NONE);
|
|
return key;
|
|
}
|
|
|
|
DirectHandle<WeakCell> FinalizationRegistryRegister(
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry,
|
|
DirectHandle<JSObject> target, DirectHandle<Object> held_value,
|
|
DirectHandle<Object> unregister_token, Isolate* isolate) {
|
|
Factory* factory = isolate->factory();
|
|
DirectHandle<JSFunction> regfunc = Cast<JSFunction>(
|
|
Object::GetProperty(isolate, finalization_registry,
|
|
factory->NewStringFromStaticChars("register"))
|
|
.ToHandleChecked());
|
|
DirectHandle<Object> args[] = {target, held_value, unregister_token};
|
|
Execution::Call(isolate, regfunc, finalization_registry, base::VectorOf(args))
|
|
.ToHandleChecked();
|
|
CHECK(IsWeakCell(finalization_registry->active_cells()));
|
|
DirectHandle<WeakCell> weak_cell = direct_handle(
|
|
Cast<WeakCell>(finalization_registry->active_cells()), isolate);
|
|
#ifdef VERIFY_HEAP
|
|
weak_cell->WeakCellVerify(isolate);
|
|
#endif // VERIFY_HEAP
|
|
return weak_cell;
|
|
}
|
|
|
|
DirectHandle<WeakCell> FinalizationRegistryRegister(
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry,
|
|
DirectHandle<JSObject> target, Isolate* isolate) {
|
|
DirectHandle<Object> undefined =
|
|
direct_handle(ReadOnlyRoots(isolate).undefined_value(), isolate);
|
|
return FinalizationRegistryRegister(finalization_registry, target, undefined,
|
|
undefined, isolate);
|
|
}
|
|
|
|
void NullifyWeakCell(DirectHandle<WeakCell> weak_cell, Isolate* isolate) {
|
|
auto empty_func = [](Tagged<HeapObject> object, ObjectSlot slot,
|
|
Tagged<Object> target) {};
|
|
weak_cell->Nullify(isolate, empty_func);
|
|
#ifdef VERIFY_HEAP
|
|
weak_cell->WeakCellVerify(isolate);
|
|
#endif // VERIFY_HEAP
|
|
}
|
|
|
|
Tagged<Object> PopClearedCellHoldings(
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry,
|
|
Isolate* isolate) {
|
|
bool may_need_key_map_shrink = false;
|
|
Tagged<WeakCell> weak_cell =
|
|
finalization_registry->PopClearedCell(isolate, &may_need_key_map_shrink);
|
|
return weak_cell->holdings();
|
|
}
|
|
|
|
// Usage: VerifyWeakCellChain(isolate, list_head, n, cell1, cell2, ..., celln);
|
|
// verifies that list_head == cell1 and cell1, cell2, ..., celln. form a list.
|
|
void VerifyWeakCellChain(Isolate* isolate, Tagged<Object> list_head, int n_args,
|
|
...) {
|
|
CHECK_GE(n_args, 0);
|
|
|
|
va_list args;
|
|
va_start(args, n_args);
|
|
|
|
if (n_args == 0) {
|
|
// Verify empty list
|
|
CHECK(IsUndefined(list_head, isolate));
|
|
} else {
|
|
Tagged<WeakCell> current =
|
|
Cast<WeakCell>(Tagged<Object>(va_arg(args, Address)));
|
|
CHECK_EQ(current, list_head);
|
|
CHECK(IsUndefined(current->prev(), isolate));
|
|
|
|
for (int i = 1; i < n_args; i++) {
|
|
Tagged<WeakCell> next =
|
|
Cast<WeakCell>(Tagged<Object>(va_arg(args, Address)));
|
|
CHECK_EQ(current->next(), next);
|
|
CHECK_EQ(next->prev(), current);
|
|
current = next;
|
|
}
|
|
CHECK(IsUndefined(current->next(), isolate));
|
|
}
|
|
va_end(args);
|
|
}
|
|
|
|
// Like VerifyWeakCellChain but verifies the chain created with key_list_prev
|
|
// and key_list_next instead of prev and next.
|
|
void VerifyWeakCellKeyChain(Isolate* isolate,
|
|
Tagged<SimpleNumberDictionary> key_map,
|
|
Tagged<Object> unregister_token, int n_args, ...) {
|
|
CHECK_GE(n_args, 0);
|
|
|
|
va_list args;
|
|
va_start(args, n_args);
|
|
|
|
Tagged<Object> hash = Object::GetHash(unregister_token);
|
|
InternalIndex entry = InternalIndex::NotFound();
|
|
if (!IsUndefined(hash, isolate)) {
|
|
uint32_t key = Smi::ToInt(hash);
|
|
entry = key_map->FindEntry(isolate, key);
|
|
}
|
|
if (n_args == 0) {
|
|
// Verify empty list
|
|
CHECK(entry.is_not_found());
|
|
} else {
|
|
CHECK(entry.is_found());
|
|
Tagged<WeakCell> current =
|
|
Cast<WeakCell>(Tagged<Object>(va_arg(args, Address)));
|
|
Tagged<Object> list_head = key_map->ValueAt(entry);
|
|
CHECK_EQ(current, list_head);
|
|
CHECK(IsUndefined(current->key_list_prev(), isolate));
|
|
|
|
for (int i = 1; i < n_args; i++) {
|
|
Tagged<WeakCell> next =
|
|
Cast<WeakCell>(Tagged<Object>(va_arg(args, Address)));
|
|
CHECK_EQ(current->key_list_next(), next);
|
|
CHECK_EQ(next->key_list_prev(), current);
|
|
current = next;
|
|
}
|
|
CHECK(IsUndefined(current->key_list_next(), isolate));
|
|
}
|
|
va_end(args);
|
|
}
|
|
|
|
Handle<JSWeakRef> MakeWeakRefAndKeepDuringJob(Isolate* isolate) {
|
|
HandleScope inner_scope(isolate);
|
|
|
|
DirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
Handle<JSWeakRef> inner_weak_ref = ConstructJSWeakRef(js_object, isolate);
|
|
isolate->heap()->KeepDuringJob(js_object);
|
|
|
|
return inner_scope.CloseAndEscape(inner_weak_ref);
|
|
}
|
|
|
|
} // namespace
|
|
|
|
TEST(TestRegister) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
HandleScope outer_scope(isolate);
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry =
|
|
ConstructJSFinalizationRegistry(isolate);
|
|
DirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
|
|
// Register a weak reference and verify internal data structures.
|
|
DirectHandle<WeakCell> weak_cell1 =
|
|
FinalizationRegistryRegister(finalization_registry, js_object, isolate);
|
|
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 1,
|
|
*weak_cell1);
|
|
CHECK(IsUndefined(weak_cell1->key_list_prev(), isolate));
|
|
CHECK(IsUndefined(weak_cell1->key_list_next(), isolate));
|
|
|
|
CHECK(IsUndefined(finalization_registry->cleared_cells(), isolate));
|
|
|
|
// No key was used during registration, key-based map stays uninitialized.
|
|
CHECK(IsUndefined(finalization_registry->key_map(), isolate));
|
|
|
|
// Register another weak reference and verify internal data structures.
|
|
DirectHandle<WeakCell> weak_cell2 =
|
|
FinalizationRegistryRegister(finalization_registry, js_object, isolate);
|
|
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 2,
|
|
*weak_cell2, *weak_cell1);
|
|
CHECK(IsUndefined(weak_cell2->key_list_prev(), isolate));
|
|
CHECK(IsUndefined(weak_cell2->key_list_next(), isolate));
|
|
|
|
CHECK(IsUndefined(finalization_registry->cleared_cells(), isolate));
|
|
CHECK(IsUndefined(finalization_registry->key_map(), isolate));
|
|
}
|
|
|
|
TEST(TestRegisterWithKey) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
HandleScope outer_scope(isolate);
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry =
|
|
ConstructJSFinalizationRegistry(isolate);
|
|
DirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
|
|
DirectHandle<JSObject> token1 = CreateKey("token1", isolate);
|
|
DirectHandle<JSObject> token2 = CreateKey("token2", isolate);
|
|
DirectHandle<Object> undefined =
|
|
direct_handle(ReadOnlyRoots(isolate).undefined_value(), isolate);
|
|
|
|
// Register a weak reference with a key and verify internal data structures.
|
|
DirectHandle<WeakCell> weak_cell1 = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token1, isolate);
|
|
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 1, *weak_cell1);
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token2, 0);
|
|
}
|
|
|
|
// Register another weak reference with a different key and verify internal
|
|
// data structures.
|
|
DirectHandle<WeakCell> weak_cell2 = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token2, isolate);
|
|
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 1, *weak_cell1);
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token2, 1, *weak_cell2);
|
|
}
|
|
|
|
// Register another weak reference with token1 and verify internal data
|
|
// structures.
|
|
DirectHandle<WeakCell> weak_cell3 = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token1, isolate);
|
|
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 2, *weak_cell3,
|
|
*weak_cell1);
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token2, 1, *weak_cell2);
|
|
}
|
|
}
|
|
|
|
TEST(TestWeakCellNullify1) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
HandleScope outer_scope(isolate);
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry =
|
|
ConstructJSFinalizationRegistry(isolate);
|
|
DirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
|
|
DirectHandle<WeakCell> weak_cell1 =
|
|
FinalizationRegistryRegister(finalization_registry, js_object, isolate);
|
|
DirectHandle<WeakCell> weak_cell2 =
|
|
FinalizationRegistryRegister(finalization_registry, js_object, isolate);
|
|
|
|
// Nullify the first WeakCell and verify internal data structures.
|
|
NullifyWeakCell(weak_cell1, isolate);
|
|
CHECK_EQ(finalization_registry->active_cells(), *weak_cell2);
|
|
CHECK(IsUndefined(weak_cell2->prev(), isolate));
|
|
CHECK(IsUndefined(weak_cell2->next(), isolate));
|
|
CHECK_EQ(finalization_registry->cleared_cells(), *weak_cell1);
|
|
CHECK(IsUndefined(weak_cell1->prev(), isolate));
|
|
CHECK(IsUndefined(weak_cell1->next(), isolate));
|
|
|
|
// Nullify the second WeakCell and verify internal data structures.
|
|
NullifyWeakCell(weak_cell2, isolate);
|
|
CHECK(IsUndefined(finalization_registry->active_cells(), isolate));
|
|
CHECK_EQ(finalization_registry->cleared_cells(), *weak_cell2);
|
|
CHECK_EQ(weak_cell2->next(), *weak_cell1);
|
|
CHECK(IsUndefined(weak_cell2->prev(), isolate));
|
|
CHECK_EQ(weak_cell1->prev(), *weak_cell2);
|
|
CHECK(IsUndefined(weak_cell1->next(), isolate));
|
|
}
|
|
|
|
TEST(TestWeakCellNullify2) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
HandleScope outer_scope(isolate);
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry =
|
|
ConstructJSFinalizationRegistry(isolate);
|
|
DirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
|
|
DirectHandle<WeakCell> weak_cell1 =
|
|
FinalizationRegistryRegister(finalization_registry, js_object, isolate);
|
|
DirectHandle<WeakCell> weak_cell2 =
|
|
FinalizationRegistryRegister(finalization_registry, js_object, isolate);
|
|
|
|
// Like TestWeakCellNullify1 but nullify the WeakCells in opposite order.
|
|
NullifyWeakCell(weak_cell2, isolate);
|
|
CHECK_EQ(finalization_registry->active_cells(), *weak_cell1);
|
|
CHECK(IsUndefined(weak_cell1->prev(), isolate));
|
|
CHECK(IsUndefined(weak_cell1->next(), isolate));
|
|
CHECK_EQ(finalization_registry->cleared_cells(), *weak_cell2);
|
|
CHECK(IsUndefined(weak_cell2->prev(), isolate));
|
|
CHECK(IsUndefined(weak_cell2->next(), isolate));
|
|
|
|
NullifyWeakCell(weak_cell1, isolate);
|
|
CHECK(IsUndefined(finalization_registry->active_cells(), isolate));
|
|
CHECK_EQ(finalization_registry->cleared_cells(), *weak_cell1);
|
|
CHECK_EQ(weak_cell1->next(), *weak_cell2);
|
|
CHECK(IsUndefined(weak_cell1->prev(), isolate));
|
|
CHECK_EQ(weak_cell2->prev(), *weak_cell1);
|
|
CHECK(IsUndefined(weak_cell2->next(), isolate));
|
|
}
|
|
|
|
TEST(TestJSFinalizationRegistryPopClearedCellHoldings1) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
Factory* factory = isolate->factory();
|
|
HandleScope outer_scope(isolate);
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry =
|
|
ConstructJSFinalizationRegistry(isolate);
|
|
DirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
DirectHandle<Object> undefined =
|
|
direct_handle(ReadOnlyRoots(isolate).undefined_value(), isolate);
|
|
|
|
DirectHandle<Object> holdings1 =
|
|
factory->NewStringFromAsciiChecked("holdings1");
|
|
DirectHandle<WeakCell> weak_cell1 = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, holdings1, undefined, isolate);
|
|
DirectHandle<Object> holdings2 =
|
|
factory->NewStringFromAsciiChecked("holdings2");
|
|
DirectHandle<WeakCell> weak_cell2 = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, holdings2, undefined, isolate);
|
|
DirectHandle<Object> holdings3 =
|
|
factory->NewStringFromAsciiChecked("holdings3");
|
|
DirectHandle<WeakCell> weak_cell3 = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, holdings3, undefined, isolate);
|
|
|
|
NullifyWeakCell(weak_cell2, isolate);
|
|
NullifyWeakCell(weak_cell3, isolate);
|
|
|
|
CHECK(finalization_registry->NeedsCleanup());
|
|
Tagged<Object> cleared1 =
|
|
PopClearedCellHoldings(finalization_registry, isolate);
|
|
CHECK_EQ(cleared1, *holdings3);
|
|
CHECK(IsUndefined(weak_cell3->prev(), isolate));
|
|
CHECK(IsUndefined(weak_cell3->next(), isolate));
|
|
|
|
CHECK(finalization_registry->NeedsCleanup());
|
|
Tagged<Object> cleared2 =
|
|
PopClearedCellHoldings(finalization_registry, isolate);
|
|
CHECK_EQ(cleared2, *holdings2);
|
|
CHECK(IsUndefined(weak_cell2->prev(), isolate));
|
|
CHECK(IsUndefined(weak_cell2->next(), isolate));
|
|
|
|
CHECK(!finalization_registry->NeedsCleanup());
|
|
|
|
NullifyWeakCell(weak_cell1, isolate);
|
|
|
|
CHECK(finalization_registry->NeedsCleanup());
|
|
Tagged<Object> cleared3 =
|
|
PopClearedCellHoldings(finalization_registry, isolate);
|
|
CHECK_EQ(cleared3, *holdings1);
|
|
CHECK(IsUndefined(weak_cell1->prev(), isolate));
|
|
CHECK(IsUndefined(weak_cell1->next(), isolate));
|
|
|
|
CHECK(!finalization_registry->NeedsCleanup());
|
|
CHECK(IsUndefined(finalization_registry->active_cells(), isolate));
|
|
CHECK(IsUndefined(finalization_registry->cleared_cells(), isolate));
|
|
}
|
|
|
|
TEST(TestJSFinalizationRegistryPopClearedCellHoldings2) {
|
|
// Test that when all WeakCells for a key are popped, the key is removed from
|
|
// the key map.
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
Factory* factory = isolate->factory();
|
|
HandleScope outer_scope(isolate);
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry =
|
|
ConstructJSFinalizationRegistry(isolate);
|
|
DirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
DirectHandle<JSObject> token1 = CreateKey("token1", isolate);
|
|
|
|
DirectHandle<Object> holdings1 =
|
|
factory->NewStringFromAsciiChecked("holdings1");
|
|
DirectHandle<WeakCell> weak_cell1 = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, holdings1, token1, isolate);
|
|
DirectHandle<Object> holdings2 =
|
|
factory->NewStringFromAsciiChecked("holdings2");
|
|
DirectHandle<WeakCell> weak_cell2 = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, holdings2, token1, isolate);
|
|
|
|
NullifyWeakCell(weak_cell1, isolate);
|
|
NullifyWeakCell(weak_cell2, isolate);
|
|
|
|
// Nullifying doesn't affect the key chains (just moves WeakCells from
|
|
// active_cells to cleared_cells).
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 2, *weak_cell2,
|
|
*weak_cell1);
|
|
}
|
|
|
|
Tagged<Object> cleared1 =
|
|
PopClearedCellHoldings(finalization_registry, isolate);
|
|
CHECK_EQ(cleared1, *holdings2);
|
|
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 1, *weak_cell1);
|
|
}
|
|
|
|
Tagged<Object> cleared2 =
|
|
PopClearedCellHoldings(finalization_registry, isolate);
|
|
CHECK_EQ(cleared2, *holdings1);
|
|
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 0);
|
|
}
|
|
}
|
|
|
|
TEST(TestUnregisterActiveCells) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
HandleScope outer_scope(isolate);
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry =
|
|
ConstructJSFinalizationRegistry(isolate);
|
|
DirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
|
|
DirectHandle<JSObject> token1 = CreateKey("token1", isolate);
|
|
DirectHandle<JSObject> token2 = CreateKey("token2", isolate);
|
|
DirectHandle<Object> undefined =
|
|
direct_handle(ReadOnlyRoots(isolate).undefined_value(), isolate);
|
|
|
|
DirectHandle<WeakCell> weak_cell1a = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token1, isolate);
|
|
DirectHandle<WeakCell> weak_cell1b = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token1, isolate);
|
|
|
|
DirectHandle<WeakCell> weak_cell2a = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token2, isolate);
|
|
DirectHandle<WeakCell> weak_cell2b = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token2, isolate);
|
|
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 4,
|
|
*weak_cell2b, *weak_cell2a, *weak_cell1b, *weak_cell1a);
|
|
VerifyWeakCellChain(isolate, finalization_registry->cleared_cells(), 0);
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 2, *weak_cell1b,
|
|
*weak_cell1a);
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token2, 2, *weak_cell2b,
|
|
*weak_cell2a);
|
|
}
|
|
|
|
JSFinalizationRegistry::Unregister(finalization_registry, token1, isolate);
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 0);
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token2, 2, *weak_cell2b,
|
|
*weak_cell2a);
|
|
}
|
|
|
|
// Both weak_cell1a and weak_cell1b removed from active_cells.
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 2,
|
|
*weak_cell2b, *weak_cell2a);
|
|
VerifyWeakCellChain(isolate, finalization_registry->cleared_cells(), 0);
|
|
}
|
|
|
|
TEST(TestUnregisterActiveAndClearedCells) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
HandleScope outer_scope(isolate);
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry =
|
|
ConstructJSFinalizationRegistry(isolate);
|
|
DirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
|
|
DirectHandle<JSObject> token1 = CreateKey("token1", isolate);
|
|
DirectHandle<JSObject> token2 = CreateKey("token2", isolate);
|
|
DirectHandle<Object> undefined =
|
|
direct_handle(ReadOnlyRoots(isolate).undefined_value(), isolate);
|
|
|
|
DirectHandle<WeakCell> weak_cell1a = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token1, isolate);
|
|
DirectHandle<WeakCell> weak_cell1b = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token1, isolate);
|
|
|
|
DirectHandle<WeakCell> weak_cell2a = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token2, isolate);
|
|
DirectHandle<WeakCell> weak_cell2b = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token2, isolate);
|
|
|
|
NullifyWeakCell(weak_cell2a, isolate);
|
|
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 3,
|
|
*weak_cell2b, *weak_cell1b, *weak_cell1a);
|
|
VerifyWeakCellChain(isolate, finalization_registry->cleared_cells(), 1,
|
|
*weak_cell2a);
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 2, *weak_cell1b,
|
|
*weak_cell1a);
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token2, 2, *weak_cell2b,
|
|
*weak_cell2a);
|
|
}
|
|
|
|
JSFinalizationRegistry::Unregister(finalization_registry, token2, isolate);
|
|
|
|
// Both weak_cell2a and weak_cell2b removed.
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 2,
|
|
*weak_cell1b, *weak_cell1a);
|
|
VerifyWeakCellChain(isolate, finalization_registry->cleared_cells(), 0);
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 2, *weak_cell1b,
|
|
*weak_cell1a);
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token2, 0);
|
|
}
|
|
}
|
|
|
|
TEST(TestWeakCellUnregisterTwice) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
HandleScope outer_scope(isolate);
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry =
|
|
ConstructJSFinalizationRegistry(isolate);
|
|
DirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
|
|
DirectHandle<JSObject> token1 = CreateKey("token1", isolate);
|
|
DirectHandle<Object> undefined =
|
|
direct_handle(ReadOnlyRoots(isolate).undefined_value(), isolate);
|
|
|
|
DirectHandle<WeakCell> weak_cell1 = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token1, isolate);
|
|
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 1,
|
|
*weak_cell1);
|
|
VerifyWeakCellChain(isolate, finalization_registry->cleared_cells(), 0);
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 1, *weak_cell1);
|
|
}
|
|
|
|
JSFinalizationRegistry::Unregister(finalization_registry, token1, isolate);
|
|
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 0);
|
|
VerifyWeakCellChain(isolate, finalization_registry->cleared_cells(), 0);
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 0);
|
|
}
|
|
|
|
JSFinalizationRegistry::Unregister(finalization_registry, token1, isolate);
|
|
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 0);
|
|
VerifyWeakCellChain(isolate, finalization_registry->cleared_cells(), 0);
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 0);
|
|
}
|
|
}
|
|
|
|
TEST(TestWeakCellUnregisterPopped) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
Factory* factory = isolate->factory();
|
|
HandleScope outer_scope(isolate);
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry =
|
|
ConstructJSFinalizationRegistry(isolate);
|
|
DirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
DirectHandle<JSObject> token1 = CreateKey("token1", isolate);
|
|
DirectHandle<Object> holdings1 =
|
|
factory->NewStringFromAsciiChecked("holdings1");
|
|
DirectHandle<WeakCell> weak_cell1 = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, holdings1, token1, isolate);
|
|
|
|
NullifyWeakCell(weak_cell1, isolate);
|
|
|
|
CHECK(finalization_registry->NeedsCleanup());
|
|
Tagged<Object> cleared1 =
|
|
PopClearedCellHoldings(finalization_registry, isolate);
|
|
CHECK_EQ(cleared1, *holdings1);
|
|
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 0);
|
|
VerifyWeakCellChain(isolate, finalization_registry->cleared_cells(), 0);
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 0);
|
|
}
|
|
|
|
JSFinalizationRegistry::Unregister(finalization_registry, token1, isolate);
|
|
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 0);
|
|
VerifyWeakCellChain(isolate, finalization_registry->cleared_cells(), 0);
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 0);
|
|
}
|
|
}
|
|
|
|
TEST(TestWeakCellUnregisterNonexistentKey) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
HandleScope outer_scope(isolate);
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry =
|
|
ConstructJSFinalizationRegistry(isolate);
|
|
DirectHandle<JSObject> token1 = CreateKey("token1", isolate);
|
|
|
|
JSFinalizationRegistry::Unregister(finalization_registry, token1, isolate);
|
|
}
|
|
|
|
TEST(TestJSWeakRef) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
i::DisableConservativeStackScanningScopeForTesting no_stack_scanning(
|
|
CcTest::heap());
|
|
HandleScope outer_scope(isolate);
|
|
IndirectHandle<JSWeakRef> weak_ref;
|
|
{
|
|
HandleScope inner_scope(isolate);
|
|
|
|
IndirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
// This doesn't add the target into the KeepDuringJob set.
|
|
IndirectHandle<JSWeakRef> inner_weak_ref =
|
|
ConstructJSWeakRef(js_object, isolate);
|
|
|
|
heap::InvokeMajorGC(CcTest::heap());
|
|
CHECK(!IsUndefined(inner_weak_ref->target(), isolate));
|
|
|
|
weak_ref = inner_scope.CloseAndEscape(inner_weak_ref);
|
|
}
|
|
|
|
CHECK(!IsUndefined(weak_ref->target(), isolate));
|
|
|
|
heap::InvokeMajorGC(CcTest::heap());
|
|
|
|
CHECK(IsUndefined(weak_ref->target(), isolate));
|
|
}
|
|
|
|
TEST(TestJSWeakRefIncrementalMarking) {
|
|
if (!v8_flags.incremental_marking) {
|
|
return;
|
|
}
|
|
ManualGCScope manual_gc_scope;
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
Heap* heap = isolate->heap();
|
|
i::DisableConservativeStackScanningScopeForTesting no_stack_scanning(heap);
|
|
HandleScope outer_scope(isolate);
|
|
IndirectHandle<JSWeakRef> weak_ref;
|
|
{
|
|
HandleScope inner_scope(isolate);
|
|
|
|
IndirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
// This doesn't add the target into the KeepDuringJob set.
|
|
IndirectHandle<JSWeakRef> inner_weak_ref =
|
|
ConstructJSWeakRef(js_object, isolate);
|
|
|
|
heap::SimulateIncrementalMarking(heap, true);
|
|
heap::InvokeMajorGC(heap);
|
|
CHECK(!IsUndefined(inner_weak_ref->target(), isolate));
|
|
|
|
weak_ref = inner_scope.CloseAndEscape(inner_weak_ref);
|
|
}
|
|
|
|
CHECK(!IsUndefined(weak_ref->target(), isolate));
|
|
|
|
heap::SimulateIncrementalMarking(heap, true);
|
|
heap::InvokeMajorGC(heap);
|
|
|
|
CHECK(IsUndefined(weak_ref->target(), isolate));
|
|
}
|
|
|
|
TEST(TestJSWeakRefKeepDuringJob) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
i::DisableConservativeStackScanningScopeForTesting no_stack_scanning(
|
|
CcTest::heap());
|
|
|
|
HandleScope outer_scope(isolate);
|
|
IndirectHandle<JSWeakRef> weak_ref = MakeWeakRefAndKeepDuringJob(isolate);
|
|
CHECK(!IsUndefined(weak_ref->target(), isolate));
|
|
heap::InvokeMajorGC(CcTest::heap());
|
|
CHECK(!IsUndefined(weak_ref->target(), isolate));
|
|
|
|
// Clears the KeepDuringJob set.
|
|
context->GetIsolate()->ClearKeptObjects();
|
|
heap::InvokeMajorGC(CcTest::heap());
|
|
CHECK(IsUndefined(weak_ref->target(), isolate));
|
|
|
|
weak_ref = MakeWeakRefAndKeepDuringJob(isolate);
|
|
CHECK(!IsUndefined(weak_ref->target(), isolate));
|
|
heap::InvokeMajorGC(CcTest::heap());
|
|
CHECK(!IsUndefined(weak_ref->target(), isolate));
|
|
|
|
// ClearKeptObjects should be called by PerformMicrotasksCheckpoint.
|
|
CcTest::isolate()->PerformMicrotaskCheckpoint();
|
|
heap::InvokeMajorGC(CcTest::heap());
|
|
CHECK(IsUndefined(weak_ref->target(), isolate));
|
|
|
|
weak_ref = MakeWeakRefAndKeepDuringJob(isolate);
|
|
CHECK(!IsUndefined(weak_ref->target(), isolate));
|
|
heap::InvokeMajorGC(CcTest::heap());
|
|
CHECK(!IsUndefined(weak_ref->target(), isolate));
|
|
|
|
// ClearKeptObjects should be called by MicrotasksScope::PerformCheckpoint.
|
|
v8::MicrotasksScope::PerformCheckpoint(CcTest::isolate());
|
|
heap::InvokeMajorGC(CcTest::heap());
|
|
CHECK(IsUndefined(weak_ref->target(), isolate));
|
|
}
|
|
|
|
TEST(TestJSWeakRefKeepDuringJobIncrementalMarking) {
|
|
if (!v8_flags.incremental_marking) {
|
|
return;
|
|
}
|
|
ManualGCScope manual_gc_scope;
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
Heap* heap = isolate->heap();
|
|
i::DisableConservativeStackScanningScopeForTesting no_stack_scanning(heap);
|
|
HandleScope outer_scope(isolate);
|
|
IndirectHandle<JSWeakRef> weak_ref = MakeWeakRefAndKeepDuringJob(isolate);
|
|
|
|
CHECK(!IsUndefined(weak_ref->target(), isolate));
|
|
|
|
heap::SimulateIncrementalMarking(heap, true);
|
|
heap::InvokeMajorGC(heap);
|
|
|
|
CHECK(!IsUndefined(weak_ref->target(), isolate));
|
|
|
|
// Clears the KeepDuringJob set.
|
|
context->GetIsolate()->ClearKeptObjects();
|
|
heap::SimulateIncrementalMarking(heap, true);
|
|
heap::InvokeMajorGC(heap);
|
|
|
|
CHECK(IsUndefined(weak_ref->target(), isolate));
|
|
}
|
|
|
|
TEST(TestRemoveUnregisterToken) {
|
|
CcTest::InitializeVM();
|
|
LocalContext context;
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
HandleScope outer_scope(isolate);
|
|
DirectHandle<JSFinalizationRegistry> finalization_registry =
|
|
ConstructJSFinalizationRegistry(isolate);
|
|
DirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
|
|
DirectHandle<JSObject> token1 = CreateKey("token1", isolate);
|
|
DirectHandle<JSObject> token2 = CreateKey("token2", isolate);
|
|
DirectHandle<HeapObject> undefined =
|
|
direct_handle(ReadOnlyRoots(isolate).undefined_value(), isolate);
|
|
|
|
DirectHandle<WeakCell> weak_cell1a = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token1, isolate);
|
|
DirectHandle<WeakCell> weak_cell1b = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token1, isolate);
|
|
|
|
DirectHandle<WeakCell> weak_cell2a = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token2, isolate);
|
|
DirectHandle<WeakCell> weak_cell2b = FinalizationRegistryRegister(
|
|
finalization_registry, js_object, undefined, token2, isolate);
|
|
|
|
NullifyWeakCell(weak_cell2a, isolate);
|
|
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 3,
|
|
*weak_cell2b, *weak_cell1b, *weak_cell1a);
|
|
VerifyWeakCellChain(isolate, finalization_registry->cleared_cells(), 1,
|
|
*weak_cell2a);
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 2, *weak_cell1b,
|
|
*weak_cell1a);
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token2, 2, *weak_cell2b,
|
|
*weak_cell2a);
|
|
}
|
|
|
|
finalization_registry->RemoveUnregisterToken(
|
|
Cast<JSReceiver>(*token2), isolate,
|
|
JSFinalizationRegistry::kKeepMatchedCellsInRegistry,
|
|
[](Tagged<HeapObject>, ObjectSlot, Tagged<Object>) {});
|
|
|
|
// Both weak_cell2a and weak_cell2b remain on the weak cell chains.
|
|
VerifyWeakCellChain(isolate, finalization_registry->active_cells(), 3,
|
|
*weak_cell2b, *weak_cell1b, *weak_cell1a);
|
|
VerifyWeakCellChain(isolate, finalization_registry->cleared_cells(), 1,
|
|
*weak_cell2a);
|
|
|
|
// But both weak_cell2a and weak_cell2b are removed from the key chain.
|
|
{
|
|
Tagged<SimpleNumberDictionary> key_map =
|
|
Cast<SimpleNumberDictionary>(finalization_registry->key_map());
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token1, 2, *weak_cell1b,
|
|
*weak_cell1a);
|
|
VerifyWeakCellKeyChain(isolate, key_map, *token2, 0);
|
|
}
|
|
}
|
|
|
|
TEST(JSWeakRefScavengedInWorklist) {
|
|
if (!v8_flags.incremental_marking || v8_flags.single_generation) {
|
|
return;
|
|
}
|
|
|
|
ManualGCScope manual_gc_scope;
|
|
CcTest::InitializeVM();
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
Heap* heap = isolate->heap();
|
|
i::DisableConservativeStackScanningScopeForTesting no_stack_scanning(heap);
|
|
|
|
{
|
|
HandleScope outer_scope(isolate);
|
|
IndirectHandle<JSWeakRef> weak_ref;
|
|
|
|
// Make a WeakRef that points to a target, both of which become unreachable.
|
|
{
|
|
HandleScope inner_scope(isolate);
|
|
IndirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
IndirectHandle<JSWeakRef> inner_weak_ref =
|
|
ConstructJSWeakRef(js_object, isolate);
|
|
CHECK(HeapLayout::InYoungGeneration(*js_object));
|
|
CHECK(HeapLayout::InYoungGeneration(*inner_weak_ref));
|
|
|
|
weak_ref = inner_scope.CloseAndEscape(inner_weak_ref);
|
|
}
|
|
|
|
// Store weak_ref in Global such that it is part of the root set when
|
|
// starting incremental marking.
|
|
v8::Global<Value> global_weak_ref(CcTest::isolate(),
|
|
Utils::ToLocal(Cast<Object>(weak_ref)));
|
|
|
|
// Do marking. This puts the WeakRef above into the js_weak_refs worklist
|
|
// since its target isn't marked.
|
|
CHECK(
|
|
heap->mark_compact_collector()->weak_objects()->js_weak_refs.IsEmpty());
|
|
heap::SimulateIncrementalMarking(heap, true);
|
|
heap->mark_compact_collector()->local_weak_objects()->Publish();
|
|
CHECK(!heap->mark_compact_collector()
|
|
->weak_objects()
|
|
->js_weak_refs.IsEmpty());
|
|
}
|
|
|
|
// Now collect both weak_ref and its target. The worklist should be empty.
|
|
heap::InvokeMinorGC(heap);
|
|
CHECK(heap->mark_compact_collector()->weak_objects()->js_weak_refs.IsEmpty());
|
|
|
|
// The mark-compactor shouldn't see zapped WeakRefs in the worklist.
|
|
heap::InvokeMajorGC(heap);
|
|
}
|
|
|
|
TEST(JSWeakRefTenuredInWorklist) {
|
|
if (!v8_flags.incremental_marking || v8_flags.single_generation ||
|
|
v8_flags.separate_gc_phases) {
|
|
return;
|
|
}
|
|
|
|
ManualGCScope manual_gc_scope;
|
|
CcTest::InitializeVM();
|
|
Isolate* isolate = CcTest::i_isolate();
|
|
Heap* heap = isolate->heap();
|
|
i::DisableConservativeStackScanningScopeForTesting no_stack_scanning(heap);
|
|
|
|
HandleScope outer_scope(isolate);
|
|
IndirectHandle<JSWeakRef> weak_ref;
|
|
|
|
// Make a WeakRef that points to a target. The target becomes unreachable.
|
|
{
|
|
HandleScope inner_scope(isolate);
|
|
IndirectHandle<JSObject> js_object =
|
|
isolate->factory()->NewJSObject(isolate->object_function());
|
|
IndirectHandle<JSWeakRef> inner_weak_ref =
|
|
ConstructJSWeakRef(js_object, isolate);
|
|
CHECK(HeapLayout::InYoungGeneration(*js_object));
|
|
CHECK(HeapLayout::InYoungGeneration(*inner_weak_ref));
|
|
|
|
weak_ref = inner_scope.CloseAndEscape(inner_weak_ref);
|
|
}
|
|
// Store weak_ref such that it is part of the root set when starting
|
|
// incremental marking.
|
|
v8::Global<Value> global_weak_ref(CcTest::isolate(),
|
|
Utils::ToLocal(Cast<Object>(weak_ref)));
|
|
Address old_weak_ref_location = weak_ref->address();
|
|
|
|
// Do marking. This puts the WeakRef above into the js_weak_refs worklist
|
|
// since its target isn't marked.
|
|
CHECK(heap->mark_compact_collector()->weak_objects()->js_weak_refs.IsEmpty());
|
|
heap::SimulateIncrementalMarking(heap, true);
|
|
heap->mark_compact_collector()->local_weak_objects()->Publish();
|
|
CHECK(
|
|
!heap->mark_compact_collector()->weak_objects()->js_weak_refs.IsEmpty());
|
|
|
|
// Now collect weak_ref's target. We still have a Handle to weak_ref, so it is
|
|
// moved and remains on the worklist.
|
|
heap::InvokeMinorGC(heap);
|
|
Address new_weak_ref_location = weak_ref->address();
|
|
CHECK_NE(old_weak_ref_location, new_weak_ref_location);
|
|
CHECK(
|
|
!heap->mark_compact_collector()->weak_objects()->js_weak_refs.IsEmpty());
|
|
|
|
// The mark-compactor should see the moved WeakRef in the worklist.
|
|
heap::InvokeMajorGC(heap);
|
|
CHECK(heap->mark_compact_collector()->weak_objects()->js_weak_refs.IsEmpty());
|
|
CHECK(IsUndefined(weak_ref->target(), isolate));
|
|
}
|
|
|
|
TEST(UnregisterTokenHeapVerifier) {
|
|
if (!v8_flags.incremental_marking) return;
|
|
ManualGCScope manual_gc_scope;
|
|
#ifdef VERIFY_HEAP
|
|
v8_flags.verify_heap = true;
|
|
#endif
|
|
|
|
CcTest::InitializeVM();
|
|
v8::Isolate* isolate = CcTest::isolate();
|
|
Heap* heap = CcTest::heap();
|
|
i::DisableConservativeStackScanningScopeForTesting no_stack_scanning(heap);
|
|
v8::HandleScope outer_scope(isolate);
|
|
|
|
{
|
|
// Make a new FinalizationRegistry and register two objects with the same
|
|
// unregister token that's unreachable after the IIFE returns.
|
|
v8::HandleScope scope(isolate);
|
|
CompileRun(
|
|
"var token = {}; "
|
|
"var registry = new FinalizationRegistry(function () {}); "
|
|
"(function () { "
|
|
" let o1 = {}; "
|
|
" let o2 = {}; "
|
|
" registry.register(o1, {}, token); "
|
|
" registry.register(o2, {}, token); "
|
|
"})();");
|
|
}
|
|
|
|
// GC so the WeakCell corresponding to o is moved from the active_cells to
|
|
// cleared_cells.
|
|
heap::InvokeMajorGC(heap);
|
|
heap::InvokeMajorGC(heap);
|
|
|
|
{
|
|
// Override the unregister token to make the original object collectible.
|
|
v8::HandleScope scope(isolate);
|
|
CompileRun("token = 0;");
|
|
}
|
|
|
|
heap::SimulateIncrementalMarking(heap, true);
|
|
|
|
// Pump message loop to run the finalizer task, then the incremental marking
|
|
// task. The finalizer task will pop the WeakCell from the cleared list. This
|
|
// should make the unregister_token slot undefined. That slot is iterated as a
|
|
// custom weak pointer, so if it is not made undefined, the verifier as part
|
|
// of the incremental marking task will crash.
|
|
EmptyMessageQueues(isolate);
|
|
}
|
|
|
|
TEST(UnregisteredAndUnclearedCellHeapVerifier) {
|
|
if (!v8_flags.incremental_marking) return;
|
|
ManualGCScope manual_gc_scope;
|
|
#ifdef VERIFY_HEAP
|
|
v8_flags.verify_heap = true;
|
|
#endif
|
|
|
|
CcTest::InitializeVM();
|
|
v8::Isolate* isolate = CcTest::isolate();
|
|
Heap* heap = CcTest::heap();
|
|
i::DisableConservativeStackScanningScopeForTesting no_stack_scanning(heap);
|
|
v8::HandleScope outer_scope(isolate);
|
|
|
|
{
|
|
// Make a new FinalizationRegistry and register an object with a token.
|
|
v8::HandleScope scope(isolate);
|
|
CompileRun(
|
|
"var token = {}; "
|
|
"var registry = new FinalizationRegistry(function () {}); "
|
|
"registry.register({}, undefined, token);");
|
|
}
|
|
|
|
// Start incremental marking to activate the marking barrier.
|
|
heap::SimulateIncrementalMarking(heap, false);
|
|
|
|
{
|
|
// Make a WeakCell list with length >1, then unregister with the token to
|
|
// the WeakCell from the registry. The linked list manipulation keeps the
|
|
// unregistered WeakCell alive (i.e. not put into cleared_cells) due to the
|
|
// marking barrier from incremental marking. Then make the original token
|
|
// collectible.
|
|
v8::HandleScope scope(isolate);
|
|
CompileRun(
|
|
"registry.register({}); "
|
|
"registry.unregister(token); "
|
|
"token = 0;");
|
|
}
|
|
|
|
// Trigger GC.
|
|
heap::InvokeMajorGC(heap);
|
|
heap::InvokeMajorGC(heap);
|
|
|
|
// Pump message loop to run the finalizer task, then the incremental marking
|
|
// task. The verifier will verify that live WeakCells don't point to dead
|
|
// unregister tokens.
|
|
EmptyMessageQueues(isolate);
|
|
}
|
|
|
|
} // namespace internal
|
|
} // namespace v8
|