Commit 354a0e4b authored by Tommy Li's avatar Tommy Li Committed by Commit Bot

[omnibox] Add base::Feature kClobberIsZeroSuggestEntrypoint.

This CL just adds a base::Feature to guard the "clobber" as ZeroSuggest
entrypoint.

Adds it to about:flags too.

Bug: 1106096
Change-Id: I63f3dbdb60a986dd0fde58d8da3007603d8f2349
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2327844Reviewed-by: default avatarJustin Donnelly <jdonnelly@chromium.org>
Commit-Queue: Tommy Li <tommycli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#793214}
parent bc51503f
...@@ -3612,6 +3612,11 @@ const FeatureEntry kFeatureEntries[] = { ...@@ -3612,6 +3612,11 @@ const FeatureEntry kFeatureEntries[] = {
FEATURE_VALUE_TYPE(omnibox::kOmniboxTabSwitchSuggestions)}, FEATURE_VALUE_TYPE(omnibox::kOmniboxTabSwitchSuggestions)},
#endif // defined(OS_ANDROID) #endif // defined(OS_ANDROID)
{"omnibox-clobber-is-zero-suggest-entrypoint",
flag_descriptions::kOmniboxClobberIsZeroSuggestEntrypointName,
flag_descriptions::kOmniboxClobberIsZeroSuggestEntrypointDescription,
kOsAll, FEATURE_VALUE_TYPE(omnibox::kClobberIsZeroSuggestEntrypoint)},
{"omnibox-on-device-head-suggestions-incognito", {"omnibox-on-device-head-suggestions-incognito",
flag_descriptions::kOmniboxOnDeviceHeadSuggestionsIncognitoName, flag_descriptions::kOmniboxOnDeviceHeadSuggestionsIncognitoName,
flag_descriptions::kOmniboxOnDeviceHeadSuggestionsIncognitoDescription, flag_descriptions::kOmniboxOnDeviceHeadSuggestionsIncognitoDescription,
......
...@@ -3186,6 +3186,11 @@ ...@@ -3186,6 +3186,11 @@
"owners": [ "manukh", "chrome-omnibox-team@google.com" ], "owners": [ "manukh", "chrome-omnibox-team@google.com" ],
"expiry_milestone": 85 "expiry_milestone": 85
}, },
{
"name": "omnibox-clobber-is-zero-suggest-entrypoint",
"owners": [ "tommycli", "chrome-omnibox-team@google.com" ],
"expiry_milestone": 95
},
{ {
"name": "omnibox-compact-suggestions", "name": "omnibox-compact-suggestions",
"owners": [ "ender", "chrome-omnibox-team@google.com" ], "owners": [ "ender", "chrome-omnibox-team@google.com" ],
......
...@@ -1442,6 +1442,15 @@ const char kOmniboxAutocompleteTitlesDescription[] = ...@@ -1442,6 +1442,15 @@ const char kOmniboxAutocompleteTitlesDescription[] =
"Allows autocompleting bookmark, history, and document suggestions when the" "Allows autocompleting bookmark, history, and document suggestions when the"
" user input is a prefix of their titles, as opposed to their URLs."; " user input is a prefix of their titles, as opposed to their URLs.";
const char kOmniboxClobberIsZeroSuggestEntrypointName[] =
"Omnibox Clobber is ZeroSuggest Entrypoint";
const char kOmniboxClobberIsZeroSuggestEntrypointDescription[] =
"If enabled, when the user clears the whole omnibox text (i.e. via "
"Backspace), Chrome will request ZeroSuggest suggestions. Note, this flag "
"merely adds a new ZeroSuggest entrypoint. ZeroSuggest still must be "
"enabled on the proper page classification (either by default or via a "
"separate flag), or else this flag will do nothing.";
const char kOmniboxCompactSuggestionsName[] = "Omnibox: Compact suggestions"; const char kOmniboxCompactSuggestionsName[] = "Omnibox: Compact suggestions";
const char kOmniboxCompactSuggestionsDescription[] = const char kOmniboxCompactSuggestionsDescription[] =
"Conserve the space for Omnibox Suggestions by slightly reducing their " "Conserve the space for Omnibox Suggestions by slightly reducing their "
......
...@@ -846,6 +846,9 @@ extern const char kOmniboxAssistantVoiceSearchDescription[]; ...@@ -846,6 +846,9 @@ extern const char kOmniboxAssistantVoiceSearchDescription[];
extern const char kOmniboxAutocompleteTitlesName[]; extern const char kOmniboxAutocompleteTitlesName[];
extern const char kOmniboxAutocompleteTitlesDescription[]; extern const char kOmniboxAutocompleteTitlesDescription[];
extern const char kOmniboxClobberIsZeroSuggestEntrypointName[];
extern const char kOmniboxClobberIsZeroSuggestEntrypointDescription[];
extern const char kOmniboxCompactSuggestionsName[]; extern const char kOmniboxCompactSuggestionsName[];
extern const char kOmniboxCompactSuggestionsDescription[]; extern const char kOmniboxCompactSuggestionsDescription[];
......
...@@ -216,6 +216,14 @@ const base::Feature kOmniboxMaxURLMatches{ ...@@ -216,6 +216,14 @@ const base::Feature kOmniboxMaxURLMatches{
const base::Feature kDynamicMaxAutocomplete{"OmniboxDynamicMaxAutocomplete", const base::Feature kDynamicMaxAutocomplete{"OmniboxDynamicMaxAutocomplete",
base::FEATURE_DISABLED_BY_DEFAULT}; base::FEATURE_DISABLED_BY_DEFAULT};
// If enabled, when the user clears the whole omnibox text (i.e. via Backspace),
// Chrome will request ZeroSuggest suggestions. Note, this flag merely adds a
// new ZeroSuggest entrypoint. ZeroSuggest still must be enabled on the proper
// page classification (either by default or via a separate flag), or else this
// flag will do nothing.
const base::Feature kClobberIsZeroSuggestEntrypoint{
"OmniboxClobberIsZeroSuggestEntrypoint", base::FEATURE_DISABLED_BY_DEFAULT};
// If enabled, ranks the local zero-prefix suggestions based on frecency // If enabled, ranks the local zero-prefix suggestions based on frecency
// (combined frequency and recency). // (combined frequency and recency).
const base::Feature kOmniboxLocalZeroSuggestFrecencyRanking{ const base::Feature kOmniboxLocalZeroSuggestFrecencyRanking{
......
...@@ -54,6 +54,7 @@ extern const base::Feature kOmniboxMaxURLMatches; ...@@ -54,6 +54,7 @@ extern const base::Feature kOmniboxMaxURLMatches;
extern const base::Feature kDynamicMaxAutocomplete; extern const base::Feature kDynamicMaxAutocomplete;
// On-Focus Suggestions a.k.a. ZeroSuggest. // On-Focus Suggestions a.k.a. ZeroSuggest.
extern const base::Feature kClobberIsZeroSuggestEntrypoint;
extern const base::Feature kOmniboxLocalZeroSuggestAgeThreshold; extern const base::Feature kOmniboxLocalZeroSuggestAgeThreshold;
extern const base::Feature kOmniboxLocalZeroSuggestFrecencyRanking; extern const base::Feature kOmniboxLocalZeroSuggestFrecencyRanking;
extern const base::Feature kOnFocusSuggestions; extern const base::Feature kOnFocusSuggestions;
......
...@@ -41251,6 +41251,8 @@ from previous Chrome versions. ...@@ -41251,6 +41251,8 @@ from previous Chrome versions.
<int value="-593536514" label="ExperimentalTabController:enabled"/> <int value="-593536514" label="ExperimentalTabController:enabled"/>
<int value="-591874964" label="LiteVideo:enabled"/> <int value="-591874964" label="LiteVideo:enabled"/>
<int value="-589096918" label="ash-enable-fullscreen-app-list"/> <int value="-589096918" label="ash-enable-fullscreen-app-list"/>
<int value="-588669613"
label="OmniboxClobberIsZeroSuggestEntrypoint:enabled"/>
<int value="-582870536" label="BluetoothNextHandsfreeProfile:enabled"/> <int value="-582870536" label="BluetoothNextHandsfreeProfile:enabled"/>
<int value="-579192400" label="disable-input-view"/> <int value="-579192400" label="disable-input-view"/>
<int value="-577982497" label="CupsPrintersUiOverhaul:enabled"/> <int value="-577982497" label="CupsPrintersUiOverhaul:enabled"/>
...@@ -42213,6 +42215,8 @@ from previous Chrome versions. ...@@ -42213,6 +42215,8 @@ from previous Chrome versions.
<int value="480677983" label="WebXRARDOMOverlay:disabled"/> <int value="480677983" label="WebXRARDOMOverlay:disabled"/>
<int value="481506759" label="ServiceWorkerScriptStreaming:enabled"/> <int value="481506759" label="ServiceWorkerScriptStreaming:enabled"/>
<int value="482232604" label="PercentBasedScrolling:enabled"/> <int value="482232604" label="PercentBasedScrolling:enabled"/>
<int value="482429100"
label="OmniboxClobberIsZeroSuggestEntrypoint:disabled"/>
<int value="485957747" label="finch-seed-ignore-pending-download"/> <int value="485957747" label="finch-seed-ignore-pending-download"/>
<int value="487810392" label="EnablePalmOnToolTypePalm:enabled"/> <int value="487810392" label="EnablePalmOnToolTypePalm:enabled"/>
<int value="491334698" label="ConditionalTabStripAndroid:enabled"/> <int value="491334698" label="ConditionalTabStripAndroid:enabled"/>
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