Commit 767c700c authored by Yue Li's avatar Yue Li Committed by Commit Bot

Dismiss the opt-in dialog if the Assistant is disabled

If the user disable the Assistant when the opt-in dialog is showing up,
the dialog should be dismissed.

Bug: b/152283171
Test: Manual Test
Change-Id: Iecb9a309f839fd6dbbcb80e82597376658a2ce3f
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2121036Reviewed-by: default avatarXiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Yue Li <updowndota@chromium.org>
Cr-Commit-Position: refs/heads/master@{#753829}
parent 84262dfa
......@@ -275,6 +275,12 @@ void AssistantOptInFlowScreenHandler::OnDialogClosed() {
}
}
void AssistantOptInFlowScreenHandler::OnAssistantSettingsEnabled(bool enabled) {
// Close the opt-in screen is the Assistant is disabled.
if (!enabled)
HandleFlowFinished();
}
void AssistantOptInFlowScreenHandler::OnAssistantStatusChanged(
ash::mojom::AssistantState state) {
if (state != ash::mojom::AssistantState::NOT_READY) {
......
......@@ -96,6 +96,7 @@ class AssistantOptInFlowScreenHandler
void Initialize() override;
// ash::AssistantStateObserver:
void OnAssistantSettingsEnabled(bool enabled) override;
void OnAssistantStatusChanged(ash::mojom::AssistantState state) override;
// Connect to assistant settings manager.
......
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