Commit 684514a5 authored by Michael Hansen's avatar Michael Hansen Committed by Commit Bot

[Nearby] Remove extra controller clear.

A recent CL [1] fixed the possibility of a crash in the event that
OnClose() is called multiple times. This removes logic where we clear
the controller which no longer has an effect.

1) https://chromium-review.googlesource.com/c/chromium/src/+/2548373

Bug: 1150176
Change-Id: If1f650f41c42ed64961f6fa312c0eb755def4b91
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2550528Reviewed-by: default avatarJames Vecore <vecore@google.com>
Commit-Queue: Michael Hansen <hansenmichael@google.com>
Cr-Commit-Position: refs/heads/master@{#829309}
parent f433130a
...@@ -132,5 +132,4 @@ void NearbyShareAction::OnClosing( ...@@ -132,5 +132,4 @@ void NearbyShareAction::OnClosing(
nearby_ui_->RemoveObserver(this); nearby_ui_->RemoveObserver(this);
nearby_ui_ = nullptr; nearby_ui_ = nullptr;
} }
controller_ = nullptr;
} }
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