Commit cddd4424 authored by Xiyuan Xia's avatar Xiyuan Xia Committed by Commit Bot

cros: Fix LoginDisplayHost double deletion

We could have two DeleteSoon scheduled at the same time and cause
double deletion. Change LoginDisplayHostMojo::OnFinalize to use
ShutdownDisplayHost() so that they will be consolidated into one.

Bug: 900615
Change-Id: I2f2db9369079adc7d4cf24bb9ef65b15ebc79320
Reviewed-on: https://chromium-review.googlesource.com/c/1313740Reviewed-by: default avatarAchuith Bhandarkar <achuith@chromium.org>
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#604942}
parent 5f78b3f5
......@@ -150,7 +150,7 @@ void LoginDisplayHostMojo::OnFinalize() {
if (dialog_)
dialog_->Close();
base::ThreadTaskRunnerHandle::Get()->DeleteSoon(FROM_HERE, this);
ShutdownDisplayHost();
}
void LoginDisplayHostMojo::SetStatusAreaVisible(bool visible) {
......
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