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

Unified Diff: ui/app_list/app_list_features.cc

Issue 2972243002: Fix app list item indexing bug. (Closed)
Patch Set: oshima's comments. Created 3 years, 5 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
« no previous file with comments | « ui/app_list/app_list_features.h ('k') | ui/app_list/test/app_list_test_view_delegate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: ui/app_list/app_list_features.cc
diff --git a/ui/app_list/app_list_features.cc b/ui/app_list/app_list_features.cc
index 55973ae281edcb8870f5eb066ce35c6174396ef3..1563921adb4d46756207b94c06b58a9cec100764 100644
--- a/ui/app_list/app_list_features.cc
+++ b/ui/app_list/app_list_features.cc
@@ -43,9 +43,8 @@ bool IsSearchResultsNewDesignEnabled() {
}
bool IsPlayStoreAppSearchEnabled() {
- static const bool enabled =
- base::FeatureList::IsEnabled(kEnablePlayStoreAppSearch);
- return enabled;
+ // Not using local static variable to allow tests to change this value.
+ return base::FeatureList::IsEnabled(kEnablePlayStoreAppSearch);
}
int AnswerCardMaxWidth() {
« no previous file with comments | « ui/app_list/app_list_features.h ('k') | ui/app_list/test/app_list_test_view_delegate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698