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

Unified Diff: components/ntp_snippets/remote/json_request.cc

Issue 2666823003: 📰 Remove the NTPSuggestionsFetchMore feature flag (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
« no previous file with comments | « components/ntp_snippets/features.cc ('k') | components/ntp_snippets/remote/remote_suggestions_fetcher.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/ntp_snippets/remote/json_request.cc
diff --git a/components/ntp_snippets/remote/json_request.cc b/components/ntp_snippets/remote/json_request.cc
index 9c24b7f84955c89f41e2797715ef22fd5db96b7e..8e522936d5d79ddb200a36198fd93ebe493e0939 100644
--- a/components/ntp_snippets/remote/json_request.cc
+++ b/components/ntp_snippets/remote/json_request.cc
@@ -141,7 +141,7 @@ CategoryInfo BuildArticleCategoryInfo(
ContentSuggestionsCardLayout::FULL_CARD,
// TODO(dgn): merge has_more_action and has_reload_action when we remove
// the kFetchMoreFeature flag. See https://crbug.com/667752
- /*has_more_action=*/base::FeatureList::IsEnabled(kFetchMoreFeature),
+ /*has_more_action=*/true,
/*has_reload_action=*/true,
/*has_view_all_action=*/false,
/*show_if_empty=*/true,
@@ -154,8 +154,7 @@ CategoryInfo BuildRemoteCategoryInfo(const base::string16& title,
title, ContentSuggestionsCardLayout::FULL_CARD,
// TODO(dgn): merge has_more_action and has_reload_action when we remove
// the kFetchMoreFeature flag. See https://crbug.com/667752
- /*has_more_action=*/allow_fetching_more_results &&
- base::FeatureList::IsEnabled(kFetchMoreFeature),
+ /*has_more_action=*/allow_fetching_more_results,
/*has_reload_action=*/allow_fetching_more_results,
/*has_view_all_action=*/false,
/*show_if_empty=*/false,
« no previous file with comments | « components/ntp_snippets/features.cc ('k') | components/ntp_snippets/remote/remote_suggestions_fetcher.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698