diff --git a/atom/common/node_bindings.cc b/atom/common/node_bindings.cc index 2207046d8e02..d1db83b70850 100644 --- a/atom/common/node_bindings.cc +++ b/atom/common/node_bindings.cc @@ -169,10 +169,11 @@ node::Environment* NodeBindings::CreateEnvironment( context->GetIsolate(), uv_default_loop(), context, args.size(), c_argv.get(), 0, nullptr); - // Node turns off AutorunMicrotasks, but we need it in web pages to match the - // behavior of Chrome. - if (!is_browser_) - context->GetIsolate()->SetAutorunMicrotasks(true); + // Node uses the deprecated SetAutorunMicrotasks(false) mode, we should switch + // to use the scoped policy to match blink's behavior. + if (!is_browser_) { + context->GetIsolate()->SetMicrotasksPolicy(v8::MicrotasksPolicy::kScoped); + } mate::Dictionary process(context->GetIsolate(), env->process_object()); process.Set("type", process_type); diff --git a/spec/chromium-spec.js b/spec/chromium-spec.js index feeffb041c51..991da316ec97 100644 --- a/spec/chromium-spec.js +++ b/spec/chromium-spec.js @@ -496,4 +496,26 @@ describe('chromium feature', function () { }) }) }) + + describe('fetch', function () { + it('does not crash', function (done) { + const server = http.createServer(function (req, res) { + res.end('test') + server.close() + }) + server.listen(0, '127.0.0.1', function () { + const port = server.address().port + fetch(`http://127.0.0.1:${port}`).then((res) => { + return res.body.getReader() + }).then((reader) => { + reader.read().then((r) => { + reader.cancel() + done() + }) + }).catch(function (e) { + done(e) + }) + }) + }) + }) })