Skip to content

Commit

Permalink
Add V8 worker shared spaces to allowlist
Browse files Browse the repository at this point in the history
An omission from an earlier CL which added shared_space and
shared_large_object_space to v8/main. This CL adds the same spaces to
v8/workers.

Bug: v8:13267, chromium:1365608
Fixed: chromium:1365608
Cq-Include-Trybots: luci.chromium.try:win_optional_gpu_tests_rel
Change-Id: Ifecf2e76cd6a555c085afa76c6a38fc5b3458b5e
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/3905053
Reviewed-by: Siddhartha S <ssid@chromium.org>
Commit-Queue: Shu-yu Guo <syg@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1048902}
  • Loading branch information
syg authored and Chromium LUCI CQ committed Sep 20, 2022
1 parent 8eb622e commit 4c133c1
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions base/trace_event/memory_infra_background_allowlist.cc
Original file line number Diff line number Diff line change
Expand Up @@ -259,6 +259,8 @@ constexpr auto kAllocatorDumpNameAllowlist = base::MakeFixedFlatSet<
"v8/workers/heap/new_space/isolate_0x?",
"v8/workers/heap/old_space/isolate_0x?",
"v8/workers/heap/read_only_space/isolate_0x?",
"v8/workers/heap/shared_large_object_space/isolate_0x?",
"v8/workers/heap/shared_space/isolate_0x?",
"v8/workers/malloc/isolate_0x?",
"v8/workers/zapped_for_debug/isolate_0x?",
"site_storage/index_db/db_0x?",
Expand Down

0 comments on commit 4c133c1

Please sign in to comment.