Commit 1456af65 authored by Maggie Cai's avatar Maggie Cai Committed by Chromium LUCI CQ

[IntentHandling] Show intent picker icon before loading the bubble.

Currently when we try to automatically show bubble for intent picker, we
will wait until the content of the bubble loaded before showing the
intent picker icon. This CL updates the code to show the icon
immediately when we find out there is app in the picker.

BUG=853604

Change-Id: I32ed2b3e1368c2d6ff8f7032ffbea3fa820c5cf5
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2643424
Commit-Queue: Maggie Cai <mxcai@chromium.org>
Reviewed-by: default avatarDominick Ng <dominickn@chromium.org>
Cr-Commit-Position: refs/heads/master@{#846007}
parent 99da1c99
......@@ -237,13 +237,14 @@ void MaybeShowIntentPickerChromeOs(
return;
}
IntentPickerTabHelper::SetShouldShowIcon(web_contents, true);
IntentPickerAutoDisplayService* ui_auto_display_service =
IntentPickerAutoDisplayService::Get(
Profile::FromBrowserContext(web_contents->GetBrowserContext()));
if (GetPickerShowState(navigation_handle, apps) ==
PickerShowState::kOmnibox) {
IntentPickerTabHelper::SetShouldShowIcon(web_contents, true);
return;
}
......
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