Skip to content

deps: V8: backport 5177b10891e6#64631

Open
avivkeller wants to merge 2 commits into
nodejs:mainfrom
avivkeller:v8-backport-8123081
Open

deps: V8: backport 5177b10891e6#64631
avivkeller wants to merge 2 commits into
nodejs:mainfrom
avivkeller:v8-backport-8123081

Conversation

@avivkeller

@avivkeller avivkeller commented Jul 20, 2026

Copy link
Copy Markdown
Member

Original commit message:

fix(inspector): hold on to promises

Keep `m_evaluationResult` strong for evaluations
until the promise settles or the request is cancelled.

Bug: 536271637
Change-Id: If21cc4aa0ba6bb2e2722d5ee73eb7744a0ead207
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/8123081
Commit-Queue: Simon Zünd <szuend@chromium.org>
Reviewed-by: Simon Zünd <szuend@chromium.org>
Reviewed-by: Kim-Anh Tran <kimanh@chromium.org>
Cr-Commit-Position: refs/heads/main@{#108874}

Refs: v8/v8@5177b10
Co-authored-by: avivkeller me@aviv.sh

@avivkeller avivkeller added the blocked PRs that are blocked by other issues or PRs. label Jul 20, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/gyp
  • @nodejs/security-wg
  • @nodejs/v8-update

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. v8 engine Issues and PRs related to the V8 dependency. labels Jul 20, 2026
@avivkeller avivkeller linked an issue Jul 20, 2026 that may be closed by this pull request
@avivkeller
avivkeller force-pushed the v8-backport-8123081 branch from 3f332e4 to 2ec3540 Compare July 20, 2026 16:32
@legendecas

Copy link
Copy Markdown
Member

CI failure seems related to #64633.

@avivkeller avivkeller added the commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. label Jul 27, 2026
@avivkeller
avivkeller force-pushed the v8-backport-8123081 branch from 2ec3540 to eb2eb4f Compare July 27, 2026 05:23
@avivkeller avivkeller added repl Issues and PRs related to the REPL subsystem. and removed blocked PRs that are blocked by other issues or PRs. labels Jul 27, 2026
@avivkeller

Copy link
Copy Markdown
Member Author

cc @nodejs/repl @nodejs/v8 Ready for review, the upstream change has landed

@avivkeller
avivkeller force-pushed the v8-backport-8123081 branch from eb2eb4f to c7fb243 Compare July 27, 2026 05:26
@avivkeller

avivkeller commented Jul 27, 2026

Copy link
Copy Markdown
Member Author

Why didn't git node include the changeset 🤔 I'll re-write it manually.

@avivkeller avivkeller changed the title deps: V8: backport 2d837cbc1ca2 deps: V8: backport If21cc4aa0ba6 Jul 27, 2026
@avivkeller avivkeller changed the title deps: V8: backport If21cc4aa0ba6 deps: V8: backport 5177b10891e6 Jul 27, 2026
Original commit message:

    fix(inspector): hold on to promises

    Keep `m_evaluationResult` strong for evaluations
    until the promise settles or the request is cancelled.

    Bug: 536271637
    Change-Id: If21cc4aa0ba6bb2e2722d5ee73eb7744a0ead207
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/8123081
    Commit-Queue: Simon Zünd <szuend@chromium.org>
    Reviewed-by: Simon Zünd <szuend@chromium.org>
    Reviewed-by: Kim-Anh Tran <kimanh@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#108874}

Refs: v8/v8@5177b10
Co-authored-by: avivkeller <me@aviv.sh>
Signed-off-by: avivkeller <me@aviv.sh>
@avivkeller
avivkeller force-pushed the v8-backport-8123081 branch from c7fb243 to 1c8abc1 Compare July 27, 2026 05:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Issues and PRs related to build files or the CI. commit-queue-rebase Add this label to allow the Commit Queue to land a PR in several commits. needs-ci PRs that need a full CI run. repl Issues and PRs related to the REPL subsystem. v8 engine Issues and PRs related to the V8 dependency.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

test/parallel/test-repl-user-error-handler is flaky

3 participants