Commit 8126ced3 authored by Ahmed Mehfooz's avatar Ahmed Mehfooz Committed by Commit Bot

Explicitly SetCanActivate for quick settings widget

Need this to ensure the quick settings widget is always activated
when focus is transferred from the message center widget.

Bug: 1048127
Change-Id: I6d142ea37bf270712d609024a0d0e09bd5a5d295
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2036274
Commit-Queue: Ahmed Mehfooz <amehfooz@chromium.org>
Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
Reviewed-by: default avatarTim Song <tengs@chromium.org>
Cr-Commit-Position: refs/heads/master@{#738359}
parent bb200018
......@@ -275,6 +275,8 @@ bool UnifiedSystemTray::FocusQuickSettings(bool reverse) {
return false;
views::Widget* quick_settings_widget = bubble_->GetBubbleWidget();
quick_settings_widget->widget_delegate()->SetCanActivate(true);
Shell::Get()->focus_cycler()->FocusWidget(quick_settings_widget);
// Focus an individual element in quick settings if chrome vox is
......@@ -444,7 +446,6 @@ void UnifiedSystemTray::ShowBubbleInternal(bool show_by_click) {
if (features::IsUnifiedMessageCenterRefactorEnabled()) {
message_center_bubble_ = std::make_unique<UnifiedMessageCenterBubble>(this);
message_center_bubble_->ShowBubble();
FocusQuickSettings(false /*reverse*/);
}
SetIsActive(true);
......
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