Skip to content

deps: update V8 to 10.1 #42657

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
targos wants to merge 21 commits into from
Closed

deps: update V8 to 10.1 #42657

Changes from 1 commit
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
a5cecb3
deps: update V8 to 10.1.124.6
targos Apr 12, 2022
f4df3c5
build: reset embedder string to "-node.0"
targos Apr 12, 2022
9be688c
src: update NODE_MODULE_VERSION to 108
targos Apr 8, 2022
4f4d0a5
deps: V8: un-cherry-pick bd019bd
refack Mar 27, 2019
1db7e5b
deps: V8: forward declaration of `Rtl*FunctionTable`
refack May 22, 2019
4451f9e
deps: fix V8 build issue with inline methods
gengjiawen Oct 14, 2020
f5e32e9
deps: disable trap handler for Windows cross-compiler
targos Oct 20, 2021
b253a3a
deps: silence V8's warning on CompileFunction
targos Nov 22, 2021
3d77c09
deps: V8: fix v8-cppgc.h for MSVC
gengjiawen Mar 17, 2022
1a07ddf
deps: make V8 compilable with older glibc
targos Mar 18, 2022
3d27658
deps: V8: cherry-pick 87ce4f5d98a5
targos Apr 5, 2022
f97c5c9
deps: V8: cherry-pick a2cae2180a7a
targos Apr 9, 2022
15e00ca
deps: V8: cherry-pick ca2a787a0b49
targos Apr 12, 2022
32d70cd
deps: V8: cherry-pick 4c29cf1b7885
targos Apr 12, 2022
5a7a632
deps: V8: cherry-pick ad21d212fc14
targos Apr 12, 2022
9aabac8
deps: V8: cherry-pick semver-major commits from 10.2
targos Apr 12, 2022
a398c24
tools,test: fix V8 initialization order
camillobruni Dec 9, 2021
bdefa66
tools: update V8 gypfiles for 10.1
targos Dec 15, 2021
827bdb7
test: adapt test-v8-serdes for V8 9.9
targos Feb 24, 2022
da62d9e
test: adapt test-worker-debug for V8 10.0
targos Feb 25, 2022
6c5b764
test: adapt message tests for V8 10.2
targos Apr 5, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
test: adapt test-worker-debug for V8 10.0 
V8 is removing callFrame.url.

Refs: https://chromium-review.googlesource.com/c/v8/v8/+/3345001
  • Loading branch information
@targos
targos committed Apr 12, 2022
commit da62d9e0e75ac9c70ee27a4701486c4e4fa8e7bf
4 changes: 1 addition & 3 deletions test/parallel/test-worker-debug.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ common.skipIfInspectorDisabled();
const assert = require('assert');
const EventEmitter = require('events');
const { Session } = require('inspector');
const { pathToFileURL } = require('url');
const {
Worker, isMainThread, parentPort, workerData
} = require('worker_threads');
Expand Down Expand Up @@ -106,7 +105,6 @@ class WorkerSession extends EventEmitter {
this.post(command);
const notification = await notificationPromise;
const callFrame = notification.params.callFrames[0];
assert.strictEqual(callFrame.url, pathToFileURL(script).toString());
assert.strictEqual(callFrame.location.lineNumber, line);
}

Expand Down Expand Up @@ -153,7 +151,7 @@ async function testBasicWorkerDebug(session, post) {
await workerSession.waitForBreakAfterCommand(
'Runtime.runIfWaitingForDebugger', __filename, 1);
await workerSession.waitForBreakAfterCommand(
'Debugger.resume', __filename, 26); // V8 line number is zero-based
'Debugger.resume', __filename, 25); // V8 line number is zero-based
const msg = await consolePromise;
assert.strictEqual(msg, workerMessage);
workerSession.post('Debugger.resume');
Expand Down