Browse Source

fix: do not DCHECK non-const methods (#20833)

Shelley Vohr 5 years ago
parent
commit
5d00494f8d
2 changed files with 5 additions and 2 deletions
  1. 3 1
      shell/renderer/atom_renderer_client.cc
  2. 2 1
      shell/renderer/web_worker_observer.cc

+ 3 - 1
shell/renderer/atom_renderer_client.cc

@@ -121,7 +121,9 @@ void AtomRendererClient::DidCreateScriptContext(
     node::tracing::TraceEventHelper::SetAgent(node::CreateAgent());
 
   // Setup node environment for each window.
-  DCHECK(node::InitializeContext(renderer_context));
+  bool initialized = node::InitializeContext(renderer_context);
+  CHECK(initialized);
+
   node::Environment* env =
       node_bindings_->CreateEnvironment(renderer_context, nullptr, true);
 

+ 2 - 1
shell/renderer/web_worker_observer.cc

@@ -48,7 +48,8 @@ void WebWorkerObserver::ContextCreated(v8::Local<v8::Context> worker_context) {
   node_bindings_->PrepareMessageLoop();
 
   // Setup node environment for each window.
-  DCHECK(node::InitializeContext(worker_context));
+  bool initialized = node::InitializeContext(worker_context);
+  CHECK(initialized);
   node::Environment* env =
       node_bindings_->CreateEnvironment(worker_context, nullptr, true);