Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(74)

Unified Diff: ios/chrome/browser/ui/contextual_search/touch_to_search_permissions_mediator_unittest.mm

Issue 2627663002: Remove Finch config control for contextual search. (Closed)
Patch Set: Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: ios/chrome/browser/ui/contextual_search/touch_to_search_permissions_mediator_unittest.mm
diff --git a/ios/chrome/browser/ui/contextual_search/touch_to_search_permissions_mediator_unittest.mm b/ios/chrome/browser/ui/contextual_search/touch_to_search_permissions_mediator_unittest.mm
index 0459922081f48deed50bd14d6e7351d4e9c3a9ce..73bee7cf4b9b487aff17f32c51d38224ede9e4fb 100644
--- a/ios/chrome/browser/ui/contextual_search/touch_to_search_permissions_mediator_unittest.mm
+++ b/ios/chrome/browser/ui/contextual_search/touch_to_search_permissions_mediator_unittest.mm
@@ -555,11 +555,14 @@ TEST(TouchToSearchPermissionsAvailabilityTest, CommandLinePermissions) {
}
TEST(TouchToSearchPermissionsAvailabilityTest, FieldTrial) {
+ // Field trial support is not currently supported, so it is expected
+ // that under all field trial configs, the feature will remain disabled.
+ // If field trial support is added back in, this test should be updated.
const struct {
const std::string trial_group_name;
bool expect_available;
} tests[] = {
- {"Enabled", true},
+ {"Enabled", false},
{"Disabled", false},
{"Control", false},
{"Spadoinkle", false},

Powered by Google App Engine
This is Rietveld 408576698