Commit ce2cdcf7 authored by manuk's avatar manuk Committed by Commit Bot

[omnibox] Add a flag setting the unfocused omnibox background white.

- This CL adds a chrome://flags flag for crrev.com/c/1474281.
- Additionally, in about_flags.cc and flag-metadata.json, it orders
omnibox-ui-use-generic-search-engine-icon below
omnibox-ui-show-suffix-on-all-search-suggestions, as in
omnibox_field_trial.cc.

Bug: 931836
Change-Id: If5a1a104c1bece5396c54630a55cb4190ed872cb
Reviewed-on: https://chromium-review.googlesource.com/c/1487363Reviewed-by: default avatarElly Fong-Jones <ellyjones@chromium.org>
Reviewed-by: default avatarKevin Bailey <krb@chromium.org>
Commit-Queue: manuk hovanesian <manukh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#635544}
parent 0683adf7
...@@ -2994,12 +2994,6 @@ const FeatureEntry kFeatureEntries[] = { ...@@ -2994,12 +2994,6 @@ const FeatureEntry kFeatureEntries[] = {
kOsDesktop, kOsDesktop,
FEATURE_VALUE_TYPE(omnibox::kUIExperimentBlueTitlesOnPageSuggestions)}, FEATURE_VALUE_TYPE(omnibox::kUIExperimentBlueTitlesOnPageSuggestions)},
{"omnibox-ui-use-generic-search-engine-icon",
flag_descriptions::kOmniboxUIUseGenericSearchEngineIconName,
flag_descriptions::kOmniboxUIUseGenericSearchEngineIconDescription,
kOsDesktop,
FEATURE_VALUE_TYPE(omnibox::kUIExperimentUseGenericSearchEngineIcon)},
{"omnibox-ui-show-suffix-on-all-search-suggestions", {"omnibox-ui-show-suffix-on-all-search-suggestions",
flag_descriptions::kOmniboxUIShowSuffixOnAllSearchSuggestionsName, flag_descriptions::kOmniboxUIShowSuffixOnAllSearchSuggestionsName,
flag_descriptions::kOmniboxUIShowSuffixOnAllSearchSuggestionsDescription, flag_descriptions::kOmniboxUIShowSuffixOnAllSearchSuggestionsDescription,
...@@ -3007,6 +3001,17 @@ const FeatureEntry kFeatureEntries[] = { ...@@ -3007,6 +3001,17 @@ const FeatureEntry kFeatureEntries[] = {
FEATURE_VALUE_TYPE( FEATURE_VALUE_TYPE(
omnibox::kUIExperimentShowSuffixOnAllSearchSuggestions)}, omnibox::kUIExperimentShowSuffixOnAllSearchSuggestions)},
{"omnibox-ui-white-background-on-blur",
flag_descriptions::kOmniboxUIWhiteBackgroundOnBlurName,
flag_descriptions::kOmniboxUIWhiteBackgroundOnBlurDescription, kOsDesktop,
FEATURE_VALUE_TYPE(omnibox::kUIExperimentWhiteBackgroundOnBlur)},
{"omnibox-ui-use-generic-search-engine-icon",
flag_descriptions::kOmniboxUIUseGenericSearchEngineIconName,
flag_descriptions::kOmniboxUIUseGenericSearchEngineIconDescription,
kOsDesktop,
FEATURE_VALUE_TYPE(omnibox::kUIExperimentUseGenericSearchEngineIcon)},
{"omnibox-material-design-weather-icons", {"omnibox-material-design-weather-icons",
flag_descriptions::kOmniboxMaterialDesignWeatherIconsName, flag_descriptions::kOmniboxMaterialDesignWeatherIconsName,
flag_descriptions::kOmniboxMaterialDesignWeatherIconsDescription, flag_descriptions::kOmniboxMaterialDesignWeatherIconsDescription,
......
...@@ -2572,12 +2572,17 @@ ...@@ -2572,12 +2572,17 @@
"expiry_milestone": 80 "expiry_milestone": 80
}, },
{ {
"name": "omnibox-ui-use-generic-search-engine-icon", "name": "omnibox-ui-show-suffix-on-all-search-suggestions",
"owners": [ "tommycli", "chrome-omnibox-team@google.com" ], "owners": [ "tommycli", "chrome-omnibox-team@google.com" ],
"expiry_milestone": 80 "expiry_milestone": 80
}, },
{ {
"name": "omnibox-ui-show-suffix-on-all-search-suggestions", "name": "omnibox-ui-white-background-on-blur",
"owners": [ "tommycli", "chrome-omnibox-team@google.com" ],
"expiry_milestone": 80
},
{
"name": "omnibox-ui-use-generic-search-engine-icon",
"owners": [ "tommycli", "chrome-omnibox-team@google.com" ], "owners": [ "tommycli", "chrome-omnibox-team@google.com" ],
"expiry_milestone": 80 "expiry_milestone": 80
}, },
......
...@@ -1422,6 +1422,11 @@ const char kOmniboxUIVerticalMarginName[] = "Omnibox UI Vertical Margin"; ...@@ -1422,6 +1422,11 @@ const char kOmniboxUIVerticalMarginName[] = "Omnibox UI Vertical Margin";
const char kOmniboxUIVerticalMarginDescription[] = const char kOmniboxUIVerticalMarginDescription[] =
"Changes the vertical margin in the Omnibox UI."; "Changes the vertical margin in the Omnibox UI.";
const char kOmniboxUIWhiteBackgroundOnBlurName[] =
"Omnibox UI White Background On Blur";
const char kOmniboxUIWhiteBackgroundOnBlurDescription[] =
"Set the omnibox background white when it's unfocused.";
const char kOmniboxVoiceSearchAlwaysVisibleName[] = const char kOmniboxVoiceSearchAlwaysVisibleName[] =
"Omnibox Voice Search Always Visible"; "Omnibox Voice Search Always Visible";
const char kOmniboxVoiceSearchAlwaysVisibleDescription[] = const char kOmniboxVoiceSearchAlwaysVisibleDescription[] =
......
...@@ -859,6 +859,9 @@ extern const char kOmniboxUIUseGenericSearchEngineIconDescription[]; ...@@ -859,6 +859,9 @@ extern const char kOmniboxUIUseGenericSearchEngineIconDescription[];
extern const char kOmniboxUIVerticalMarginName[]; extern const char kOmniboxUIVerticalMarginName[];
extern const char kOmniboxUIVerticalMarginDescription[]; extern const char kOmniboxUIVerticalMarginDescription[];
extern const char kOmniboxUIWhiteBackgroundOnBlurName[];
extern const char kOmniboxUIWhiteBackgroundOnBlurDescription[];
extern const char kOmniboxVoiceSearchAlwaysVisibleName[]; extern const char kOmniboxVoiceSearchAlwaysVisibleName[];
extern const char kOmniboxVoiceSearchAlwaysVisibleDescription[]; extern const char kOmniboxVoiceSearchAlwaysVisibleDescription[];
......
...@@ -31969,6 +31969,8 @@ from previous Chrome versions. ...@@ -31969,6 +31969,8 @@ from previous Chrome versions.
<int value="-192389983" label="NoStatePrefetch:enabled"/> <int value="-192389983" label="NoStatePrefetch:enabled"/>
<int value="-191256027" label="AvoidFlashBetweenNavigation:disabled"/> <int value="-191256027" label="AvoidFlashBetweenNavigation:disabled"/>
<int value="-185162926" label="IncreaseInputAudioBufferSize:enabled"/> <int value="-185162926" label="IncreaseInputAudioBufferSize:enabled"/>
<int value="-184091779"
label="OmniboxUIExperimentWhiteBackgroundOnBlur:enabled"/>
<int value="-183246373" label="enable-multilingual-spellchecker"/> <int value="-183246373" label="enable-multilingual-spellchecker"/>
<int value="-181590721" <int value="-181590721"
label="AutofillEnforceMinRequiredFieldsForHeuristics:enabled"/> label="AutofillEnforceMinRequiredFieldsForHeuristics:enabled"/>
...@@ -33033,6 +33035,8 @@ from previous Chrome versions. ...@@ -33033,6 +33035,8 @@ from previous Chrome versions.
<int value="1614309501" label="DataSaverLiteModeRebranding:enabled"/> <int value="1614309501" label="DataSaverLiteModeRebranding:enabled"/>
<int value="1614596813" label="CloseButtonsInactiveTabs:disabled"/> <int value="1614596813" label="CloseButtonsInactiveTabs:disabled"/>
<int value="1615988672" label="GrantNotificationsToDSE:enabled"/> <int value="1615988672" label="GrantNotificationsToDSE:enabled"/>
<int value="1616938915"
label="OmniboxUIExperimentWhiteBackgroundOnBlur:disabled"/>
<int value="1617187093" label="enable-improved-a2hs"/> <int value="1617187093" label="enable-improved-a2hs"/>
<int value="1618633341" label="MashOopViz:enabled"/> <int value="1618633341" label="MashOopViz:enabled"/>
<int value="1621298798" label="VrBrowserKeyboard:enabled"/> <int value="1621298798" label="VrBrowserKeyboard: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