Commit 78e81299 authored by Ce Chen's avatar Ce Chen Committed by Commit Bot

[omnibox] Launch On Device Head Suggest Incognito for all clients.

Bug: 925072
Change-Id: Ib86cd7d6f99a8489e81f679f9f10265d02574a1d
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2525962Reviewed-by: default avatarJustin Donnelly <jdonnelly@chromium.org>
Commit-Queue: Ce Chen <cch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#825612}
parent 88b1d7e4
...@@ -121,34 +121,8 @@ TEST_F(OnDeviceHeadProviderTest, TestIfIncognitoIsAllowed) { ...@@ -121,34 +121,8 @@ TEST_F(OnDeviceHeadProviderTest, TestIfIncognitoIsAllowed) {
EXPECT_CALL(*client_.get(), SearchSuggestEnabled()) EXPECT_CALL(*client_.get(), SearchSuggestEnabled())
.WillRepeatedly(Return(true)); .WillRepeatedly(Return(true));
// By default incognito request will be accepted on mobile, rejected on // By default incognito request will be accepted.
// desktop.
{ {
base::test::ScopedFeatureList scoped_feature_list;
scoped_feature_list.InitAndEnableFeature(
omnibox::kOnDeviceHeadProviderNonIncognito);
#if defined(OS_ANDROID) || defined(OS_IOS)
ASSERT_TRUE(IsOnDeviceHeadProviderAllowed(input));
#else
ASSERT_FALSE(IsOnDeviceHeadProviderAllowed(input));
#endif
}
// Now enable for incognito.
{
base::test::ScopedFeatureList scoped_feature_list;
scoped_feature_list.InitAndEnableFeature(
omnibox::kOnDeviceHeadProviderIncognito);
ASSERT_TRUE(IsOnDeviceHeadProviderAllowed(input));
}
// Test enable for both.
{
base::test::ScopedFeatureList scoped_feature_list;
scoped_feature_list.InitWithFeatures(
{omnibox::kOnDeviceHeadProviderNonIncognito,
omnibox::kOnDeviceHeadProviderIncognito},
{});
ASSERT_TRUE(IsOnDeviceHeadProviderAllowed(input)); ASSERT_TRUE(IsOnDeviceHeadProviderAllowed(input));
} }
......
...@@ -251,7 +251,7 @@ const base::Feature kReactiveZeroSuggestionsOnNTPRealbox{ ...@@ -251,7 +251,7 @@ const base::Feature kReactiveZeroSuggestionsOnNTPRealbox{
// NonIncognito will only controls behaviors under incognito / non-incognito // NonIncognito will only controls behaviors under incognito / non-incognito
// mode respectively. // mode respectively.
const base::Feature kOnDeviceHeadProviderIncognito{ const base::Feature kOnDeviceHeadProviderIncognito{
"OmniboxOnDeviceHeadProviderIncognito", enabled_by_default_android_ios}; "OmniboxOnDeviceHeadProviderIncognito", base::FEATURE_ENABLED_BY_DEFAULT};
const base::Feature kOnDeviceHeadProviderNonIncognito{ const base::Feature kOnDeviceHeadProviderNonIncognito{
"OmniboxOnDeviceHeadProviderNonIncognito", enabled_by_default_android_ios}; "OmniboxOnDeviceHeadProviderNonIncognito", enabled_by_default_android_ios};
......
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