Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1156)

Unified Diff: mojo/edk/embedder/embedder.cc

Issue 1496663002: EDK: Make mojo::platform::TaskRunner's PostTask() take an std::function<void()>. (Closed) Base URL: https://github.com/domokit/mojo.git@master
Patch Set: Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « mojo/edk/base_edk/platform_task_runner_impl.cc ('k') | mojo/edk/embedder/embedder_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: mojo/edk/embedder/embedder.cc
diff --git a/mojo/edk/embedder/embedder.cc b/mojo/edk/embedder/embedder.cc
index b058b3685e9de28e4f98fa36efb4c381abe3f004..255eafc2903a80203909c6bb5d94255741dec8f5 100644
--- a/mojo/edk/embedder/embedder.cc
+++ b/mojo/edk/embedder/embedder.cc
@@ -64,21 +64,6 @@ system::ChannelId MakeChannelId() {
return static_cast<system::ChannelId>(-new_counter_value);
}
-// Note: Called on the I/O thread.
-void ShutdownIPCSupportHelper() {
- // Save these before they get nuked by |ShutdownChannelOnIOThread()|.
- RefPtr<TaskRunner> delegate_thread_task_runner(
- internal::g_ipc_support->delegate_thread_task_runner());
- ProcessDelegate* process_delegate =
- internal::g_ipc_support->process_delegate();
-
- ShutdownIPCSupportOnIOThread();
-
- delegate_thread_task_runner->PostTask(
- base::Bind(&ProcessDelegate::OnShutdownComplete,
- base::Unretained(process_delegate)));
-}
-
} // namespace
Configuration* GetConfiguration() {
@@ -167,8 +152,18 @@ void ShutdownIPCSupportOnIOThread() {
void ShutdownIPCSupport() {
DCHECK(internal::g_ipc_support);
- internal::g_ipc_support->io_thread_task_runner()->PostTask(
- base::Bind(&ShutdownIPCSupportHelper));
+ internal::g_ipc_support->io_thread_task_runner()->PostTask([]() {
+ // Save these before they get nuked by |ShutdownChannelOnIOThread()|.
+ RefPtr<TaskRunner> delegate_thread_task_runner(
+ internal::g_ipc_support->delegate_thread_task_runner());
+ ProcessDelegate* process_delegate =
+ internal::g_ipc_support->process_delegate();
+
+ ShutdownIPCSupportOnIOThread();
+
+ delegate_thread_task_runner->PostTask(
+ [process_delegate]() { process_delegate->OnShutdownComplete(); });
+ });
}
ScopedMessagePipeHandle ConnectToSlave(
« no previous file with comments | « mojo/edk/base_edk/platform_task_runner_impl.cc ('k') | mojo/edk/embedder/embedder_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698