remove src_expose_maybeinitializecontext_to_allow_existing_contexts.patch

codebytere · codebytere · commit 848570ddce09 · 2019-09-05T10:43:23.000-07:00
nodejs/node#28544
diff --git a/patches/node/.patches b/patches/node/.patches
@@ -32,7 +32,6 @@ chore_allow_the_node_entrypoint_to_be_a_builtin_module.patch
 inherit_electron_crashpad_pipe_name_in_child_process.patch
 fixme_revert_crypto_add_support_for_rsa-pss_keys.patch
 chore_re-add_compileandcall_this_should_be_added_as_a_helper_in.patch
-src_expose_maybeinitializecontext_to_allow_existing_contexts.patch
 fix_extern_the_nativemoduleenv_and_options_parser_for_debug_builds.patch
 chore_read_nobrowserglobals_from_global_not_process.patch
 chore_split_createenvironment_into_createenvironment_and.patch
diff --git a/patches/node/src_expose_maybeinitializecontext_to_allow_existing_contexts.patch b/patches/node/src_expose_maybeinitializecontext_to_allow_existing_contexts.patch
diff --git a/shell/renderer/atom_renderer_client.cc b/shell/renderer/atom_renderer_client.cc
@@ -119,11 +119,12 @@ void AtomRendererClient::DidCreateScriptContext(
     node::tracing::TraceEventHelper::SetAgent(node::CreateAgent());
 
   // Setup node environment for each window.
-  v8::Local<v8::Context> context =
-      node::MaybeInitializeContext(renderer_context);
-  DCHECK(!context.IsEmpty());
+  bool success = node::InitializeContext(renderer_context);
+  DCHECK(success);
+
   node::Environment* env =
-      node_bindings_->CreateEnvironment(context, nullptr, true);
+      node_bindings_->CreateEnvironment(renderer_context, nullptr, true);
+
   // If we have disabled the site instance overrides we should prevent loading
   // any non-context aware native module
   if (command_line->HasSwitch(switches::kDisableElectronSiteInstanceOverrides))
diff --git a/shell/renderer/web_worker_observer.cc b/shell/renderer/web_worker_observer.cc
@@ -48,10 +48,11 @@ void WebWorkerObserver::ContextCreated(v8::Local<v8::Context> worker_context) {
   node_bindings_->PrepareMessageLoop();
 
   // Setup node environment for each window.
-  v8::Local<v8::Context> context = node::MaybeInitializeContext(worker_context);
-  DCHECK(!context.IsEmpty());
+  bool success = node::InitializeContext(worker_context);
+  DCHECK(success);
+
   node::Environment* env =
-      node_bindings_->CreateEnvironment(context, nullptr, true);
+      node_bindings_->CreateEnvironment(worker_context, nullptr, true);
 
   // Add Electron extended APIs.
   electron_bindings_->BindTo(env->isolate(), env->process_object());
