Commit c87ad80e authored by Hidehiko Abe's avatar Hidehiko Abe Committed by Commit Bot

lacros: Fix DCHECK failure.

In base::Process::WaitForExitWithTimeout, it is checked whether
base sync primitives is allowed. Send to the thread with the trait.

Bug: 1103798
Test: Ran locally, and no DCHECK error on lacros-chrome termination.
Change-Id: Ie1d15d1ee949a6f7c06726ccb14c90b38fea96ad
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2299947Reviewed-by: default avatarJames Cook <jamescook@chromium.org>
Commit-Queue: Hidehiko Abe <hidehiko@chromium.org>
Cr-Commit-Position: refs/heads/master@{#788888}
parent 776c27ba
...@@ -241,7 +241,7 @@ void LacrosManager::OnMojoDisconnected() { ...@@ -241,7 +241,7 @@ void LacrosManager::OnMojoDisconnected() {
lacros_chrome_service_.reset(); lacros_chrome_service_.reset();
ash_chrome_service_ = nullptr; ash_chrome_service_ = nullptr;
base::ThreadPool::PostTaskAndReply( base::ThreadPool::PostTaskAndReply(
FROM_HERE, {base::MayBlock()}, FROM_HERE, {base::WithBaseSyncPrimitives()},
base::BindOnce(&TerminateLacrosChrome, std::move(lacros_process_)), base::BindOnce(&TerminateLacrosChrome, std::move(lacros_process_)),
base::BindOnce(&LacrosManager::OnLacrosChromeTerminated, base::BindOnce(&LacrosManager::OnLacrosChromeTerminated,
weak_factory_.GetWeakPtr())); weak_factory_.GetWeakPtr()));
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment