chromium/src
0
Fork 0
Code Activity

NULL g_manage_passwords_bubble_ variable only when there is no next

Browse Source 
password bubble already open.

Bug: 399131922
Change-Id: I353f79d68a60685dae8f9959d17d862eabdff4a7
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6310316
Reviewed-by: Maria Kazinova <kazinova@google.com>
Commit-Queue: Vasilii Sukhanov <vasilii@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1425828}
This commit is contained in:
Vasilii Sukhanov
2025-02-27 10:12:32 -08:00
committed by Chromium LUCI CQ
parent 8c6cdd27db
commit dd20082c99
1 changed files with 7 additions and 2 deletions

9
chrome/browser/ui/views/passwords/password_bubble_view_base.cc

@ -105,8 +105,13 @@ void PasswordBubbleViewBase::ShowBubble(content::WebContents* web_contents,
views::BubbleDialogDelegateView::CreateBubble(g_manage_passwords_bubble_);
g_manage_passwords_bubble_->ShowForReason(reason);
g_manage_passwords_bubble_->RegisterWindowClosingCallback(
base::BindOnce([]() { g_manage_passwords_bubble_ = nullptr; }));
g_manage_passwords_bubble_->RegisterWindowClosingCallback(base::BindOnce(
[](PasswordBubbleViewBase* closing_bubble) {
if (closing_bubble == g_manage_passwords_bubble_) {
g_manage_passwords_bubble_ = nullptr;
}
},
bubble));
if (features::IsToolbarPinningEnabled()) {
auto* passwords_action_item = actions::ActionManager::Get().FindAction(