From e02e296bd58fb156dbd7fa61919bbb3584a5aee1 Mon Sep 17 00:00:00 2001 From: Trevor Norris Date: Fri, 10 Jul 2015 13:55:14 -0600 Subject: [PATCH] node: remove bad fn call and check These two lines exist because of a screw up on my part while combining MakeCallback() and MakeDomainCallback(). The reason it never broke core tests is because any paths it would have broken were rerouted to AsyncWrap::MakeCallback(). The only case that node::MakeCallback() handles anymore is setImmediate(). Fix: a1da024 "node, async-wrap: remove MakeDomainCallback" --- src/node.cc | 2 -- 1 file changed, 2 deletions(-) diff --git a/src/node.cc b/src/node.cc index ec8907a6d56123..c103a31433e04e 100644 --- a/src/node.cc +++ b/src/node.cc @@ -1091,8 +1091,6 @@ Handle MakeCallback(Environment* env, return Undefined(env->isolate()); } } - env->tick_callback_function()->Call(process, 0, nullptr); - CHECK_EQ(env->context(), env->isolate()->GetCurrentContext()); if (try_catch.HasCaught()) { return Undefined(env->isolate());