Skip to content

deps: V8: cherry-pick ba2b3f0f93#63839

Closed
bnoordhuis wants to merge 1 commit into
nodejs:mainfrom
bnoordhuis:cherry-pick-v8-commit
Closed

deps: V8: cherry-pick ba2b3f0f93#63839
bnoordhuis wants to merge 1 commit into
nodejs:mainfrom
bnoordhuis:cherry-pick-v8-commit

Conversation

@bnoordhuis

Copy link
Copy Markdown
Member

Original commit message:

[api] Add ArrayBuffer::CopyArrayBufferBytes

Make the not uncommon case of copying bytes from one ArrayBuffer to
another easier and less surprising, in particular when SharedArrayBuffer
semantics are involved.

Change-Id: I2a2bc5cba4e8c3b2d88cfcb2df0ef7e2021a8f6f
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/7735151
Auto-Submit: Ben Noordhuis <info@bnoordhuis.nl>
Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
Reviewed-by: Olivier Flückiger <olivf@chromium.org>
Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
Cr-Commit-Position: refs/heads/main@{#107890}

Original commit message:

    [api] Add ArrayBuffer::CopyArrayBufferBytes

    Make the not uncommon case of copying bytes from one ArrayBuffer to
    another easier and less surprising, in particular when SharedArrayBuffer
    semantics are involved.

    Change-Id: I2a2bc5cba4e8c3b2d88cfcb2df0ef7e2021a8f6f
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/7735151
    Auto-Submit: Ben Noordhuis <info@bnoordhuis.nl>
    Commit-Queue: Michael Lippautz <mlippautz@chromium.org>
    Reviewed-by: Olivier Flückiger <olivf@chromium.org>
    Reviewed-by: Michael Lippautz <mlippautz@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#107890}

Signed-off-by: Ben Noordhuis <info@bnoordhuis.nl>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

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

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. v8 engine Issues and PRs related to the V8 dependency. labels Jun 10, 2026
@bnoordhuis

Copy link
Copy Markdown
Member Author

I just discovered @ronag also included this commit as part of #63828. I'll go ahead and close this then.

@bnoordhuis bnoordhuis closed this Jun 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-ci PRs that need a full CI run. v8 engine Issues and PRs related to the V8 dependency.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants