Skip to content

Commit 0e5b6f9

Browse files
codebyterezcbenz
authored andcommitted
Upgrade to node v9.3.0 (#11507)
* update submodule refs for node v9.3.0 * Define "llvm_version" for Node.js build * NODE_MODULE_CONTEXT_AWARE_BUILTIN -> NODE_BUILTIN_MODULE_CONTEXT_AWARE * update NodePlatform to MultiIsolatePlatform * fix linting error * update node ref * REVIEW: Explicitly register builtin modules nodejs/node#16565 * update libcc ref * switch libcc to c62 * REVIEW: Address node api changes - Always start the inspector agent for nodejs/node#17085 - Set the tracing controller for node nodejs/node#15538 - Isolate data creation now requires plaform nodejs/node#16700
1 parent 31eb5e2 commit 0e5b6f9

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

44 files changed

+137
-98
lines changed

atom/app/node_main.cc

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@
1212
#include "atom/common/api/atom_bindings.h"
1313
#include "atom/common/crash_reporter/crash_reporter.h"
1414
#include "atom/common/native_mate_converters/string16_converter.h"
15+
#include "atom/common/node_bindings.h"
1516
#include "base/command_line.h"
1617
#include "base/feature_list.h"
1718
#include "base/task_scheduler/task_scheduler.h"
@@ -50,14 +51,16 @@ int NodeMain(int argc, char *argv[]) {
5051
// Initialize gin::IsolateHolder.
5152
JavascriptEnvironment gin_env;
5253

54+
// Explicitly register electron's builtin modules.
55+
NodeBindings::RegisterBuiltinModules();
56+
5357
int exec_argc;
5458
const char** exec_argv;
5559
node::Init(&argc, const_cast<const char**>(argv), &exec_argc, &exec_argv);
5660

57-
node::IsolateData isolate_data(gin_env.isolate(), loop);
5861
node::Environment* env = node::CreateEnvironment(
59-
&isolate_data, gin_env.context(), argc, argv,
60-
exec_argc, exec_argv);
62+
node::CreateIsolateData(gin_env.isolate(), loop, gin_env.platform()),
63+
gin_env.context(), argc, argv, exec_argc, exec_argv);
6164

6265
// Enable support for v8 inspector.
6366
NodeDebugger node_debugger(env);
@@ -79,6 +82,7 @@ int NodeMain(int argc, char *argv[]) {
7982
bool more;
8083
do {
8184
more = uv_run(env->event_loop(), UV_RUN_ONCE);
85+
gin_env.platform()->DrainBackgroundTasks(env->isolate());
8286
if (more == false) {
8387
node::EmitBeforeExit(env);
8488

@@ -92,6 +96,8 @@ int NodeMain(int argc, char *argv[]) {
9296

9397
exit_code = node::EmitExit(env);
9498
node::RunAtExit(env);
99+
gin_env.platform()->DrainBackgroundTasks(env->isolate());
100+
gin_env.platform()->CancelPendingDelayedTasks(env->isolate());
95101

96102
node::FreeEnvironment(env);
97103
}

atom/browser/api/atom_api_app.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1342,4 +1342,4 @@ void Initialize(v8::Local<v8::Object> exports, v8::Local<v8::Value> unused,
13421342

13431343
} // namespace
13441344

1345-
NODE_MODULE_CONTEXT_AWARE_BUILTIN(atom_browser_app, Initialize)
1345+
NODE_BUILTIN_MODULE_CONTEXT_AWARE(atom_browser_app, Initialize)

atom/browser/api/atom_api_auto_updater.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -150,4 +150,4 @@ void Initialize(v8::Local<v8::Object> exports, v8::Local<v8::Value> unused,
150150

151151
} // namespace
152152

153-
NODE_MODULE_CONTEXT_AWARE_BUILTIN(atom_browser_auto_updater, Initialize)
153+
NODE_BUILTIN_MODULE_CONTEXT_AWARE(atom_browser_auto_updater, Initialize)

atom/browser/api/atom_api_browser_view.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,4 +162,4 @@ void Initialize(v8::Local<v8::Object> exports,
162162

163163
} // namespace
164164

165-
NODE_MODULE_CONTEXT_AWARE_BUILTIN(atom_browser_browser_view, Initialize)
165+
NODE_BUILTIN_MODULE_CONTEXT_AWARE(atom_browser_browser_view, Initialize)

atom/browser/api/atom_api_browser_window.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1332,4 +1332,4 @@ void Initialize(v8::Local<v8::Object> exports, v8::Local<v8::Value> unused,
13321332

13331333
} // namespace
13341334

1335-
NODE_MODULE_CONTEXT_AWARE_BUILTIN(atom_browser_window, Initialize)
1335+
NODE_BUILTIN_MODULE_CONTEXT_AWARE(atom_browser_window, Initialize)

atom/browser/api/atom_api_content_tracing.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,4 +73,4 @@ void Initialize(v8::Local<v8::Object> exports, v8::Local<v8::Value> unused,
7373

7474
} // namespace
7575

76-
NODE_MODULE_CONTEXT_AWARE_BUILTIN(atom_browser_content_tracing, Initialize)
76+
NODE_BUILTIN_MODULE_CONTEXT_AWARE(atom_browser_content_tracing, Initialize)

atom/browser/api/atom_api_debugger.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -178,4 +178,4 @@ void Initialize(v8::Local<v8::Object> exports, v8::Local<v8::Value> unused,
178178

179179
} // namespace
180180

181-
NODE_MODULE_CONTEXT_AWARE_BUILTIN(atom_browser_debugger, Initialize);
181+
NODE_BUILTIN_MODULE_CONTEXT_AWARE(atom_browser_debugger, Initialize);

atom/browser/api/atom_api_desktop_capturer.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,4 +114,4 @@ void Initialize(v8::Local<v8::Object> exports, v8::Local<v8::Value> unused,
114114

115115
} // namespace
116116

117-
NODE_MODULE_CONTEXT_AWARE_BUILTIN(atom_browser_desktop_capturer, Initialize);
117+
NODE_BUILTIN_MODULE_CONTEXT_AWARE(atom_browser_desktop_capturer, Initialize);

atom/browser/api/atom_api_dialog.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -140,4 +140,4 @@ void Initialize(v8::Local<v8::Object> exports, v8::Local<v8::Value> unused,
140140

141141
} // namespace
142142

143-
NODE_MODULE_CONTEXT_AWARE_BUILTIN(atom_browser_dialog, Initialize)
143+
NODE_BUILTIN_MODULE_CONTEXT_AWARE(atom_browser_dialog, Initialize)

atom/browser/api/atom_api_download_item.cc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -236,4 +236,4 @@ void Initialize(v8::Local<v8::Object> exports, v8::Local<v8::Value> unused,
236236

237237
} // namespace
238238

239-
NODE_MODULE_CONTEXT_AWARE_BUILTIN(atom_browser_download_item, Initialize);
239+
NODE_BUILTIN_MODULE_CONTEXT_AWARE(atom_browser_download_item, Initialize);

0 commit comments

Comments
 (0)