Skip to content

Commit

Permalink
deps: V8: cherry-pick 30861a39323d
Browse files Browse the repository at this point in the history
Original commit message:

    [debug] Re-enable internal [[Scopes]] property

    We received feedback that the [[Scopes]] property has some legitimate
    use-cases not covered by the Scopes View during pause.

    We re-enable the feature for now and will remove the flag in a
    follow-up.

    [email protected]

    Bug: chromium:1365858
    Change-Id: Ibf279ae6c4f5ae492d03e9b4ee7316f6500508d9
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/4099385
    Auto-Submit: Simon Zünd <[email protected]>
    Commit-Queue: Benedikt Meurer <[email protected]>
    Commit-Queue: Simon Zünd <[email protected]>
    Reviewed-by: Benedikt Meurer <[email protected]>
    Cr-Commit-Position: refs/heads/main@{#84800}

Refs: v8/v8@30861a3
PR-URL: #45851
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
AaronFriel authored and RafaelGSS committed Jan 2, 2023
1 parent 6f1a180 commit 669cdb2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion deps/v8/src/flags/flag-definitions.h
Original file line number Diff line number Diff line change
Expand Up @@ -1717,7 +1717,7 @@ DEFINE_BOOL(experimental_reuse_locals_blocklists, false,
DEFINE_BOOL(
live_edit_top_frame, true,
"enable support for live-editing the top-most function on the stack")
DEFINE_BOOL(experimental_remove_internal_scopes_property, true,
DEFINE_BOOL(experimental_remove_internal_scopes_property, false,
"don't report the artificial [[Scopes]] property for functions")

// disassembler
Expand Down

0 comments on commit 669cdb2

Please sign in to comment.