From 529274f61adf138f7bad57a9bcce49a94ef98677 Mon Sep 17 00:00:00 2001 From: Anna Henningsen Date: Fri, 15 Feb 2019 19:49:42 +0100 Subject: [PATCH] src: remove inspector main_thread_request_ field This is redundant to the platform notification mechanism, and the handle may not be cleaned up util we attempt to close the loop. Refs: https://github.com/nodejs/node/pull/26089 Refs: https://github.com/nodejs/node/pull/26006 --- src/inspector/main_thread_interface.cc | 6 ------ src/inspector/main_thread_interface.h | 1 - 2 files changed, 7 deletions(-) diff --git a/src/inspector/main_thread_interface.cc b/src/inspector/main_thread_interface.cc index 1bcf65134fb48a..25b6270e412464 100644 --- a/src/inspector/main_thread_interface.cc +++ b/src/inspector/main_thread_interface.cc @@ -224,11 +224,6 @@ MainThreadInterface::MainThreadInterface(Agent* agent, uv_loop_t* loop, v8::Platform* platform) : agent_(agent), isolate_(isolate), platform_(platform) { - main_thread_request_.reset(new AsyncAndInterface(uv_async_t(), this)); - CHECK_EQ(0, uv_async_init(loop, &main_thread_request_->first, - DispatchMessagesAsyncCallback)); - // Inspector uv_async_t should not prevent main loop shutdown. - uv_unref(reinterpret_cast(&main_thread_request_->first)); } MainThreadInterface::~MainThreadInterface() { @@ -253,7 +248,6 @@ void MainThreadInterface::Post(std::unique_ptr request) { bool needs_notify = requests_.empty(); requests_.push_back(std::move(request)); if (needs_notify) { - CHECK_EQ(0, uv_async_send(&main_thread_request_->first)); if (isolate_ != nullptr && platform_ != nullptr) { std::shared_ptr taskrunner = platform_->GetForegroundTaskRunner(isolate_); diff --git a/src/inspector/main_thread_interface.h b/src/inspector/main_thread_interface.h index a7d9f8a3c939d8..1c644144cca991 100644 --- a/src/inspector/main_thread_interface.h +++ b/src/inspector/main_thread_interface.h @@ -105,7 +105,6 @@ class MainThreadInterface { Agent* const agent_; v8::Isolate* const isolate_; v8::Platform* const platform_; - DeleteFnPtr main_thread_request_; std::shared_ptr handle_; std::unordered_map> managed_objects_; };