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

Unified Diff: components/ntp_snippets/physical_web_pages/physical_web_page_suggestions_provider.cc

Issue 2744253004: NTP: clang-format (Closed)
Patch Set: rebase Created 3 years, 9 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: components/ntp_snippets/physical_web_pages/physical_web_page_suggestions_provider.cc
diff --git a/components/ntp_snippets/physical_web_pages/physical_web_page_suggestions_provider.cc b/components/ntp_snippets/physical_web_pages/physical_web_page_suggestions_provider.cc
index 5398b22d65cf5e36eec8c2b99d5574a68736294c..b8382c0caa002e07aad20dee6439435cafb3e5eb 100644
--- a/components/ntp_snippets/physical_web_pages/physical_web_page_suggestions_provider.cc
+++ b/components/ntp_snippets/physical_web_pages/physical_web_page_suggestions_provider.cc
@@ -50,8 +50,7 @@ bool CompareByDistance(const physical_web::Metadata& left,
return left.distance_estimate < right.distance_estimate;
}
-void FilterOutByGroupId(
- physical_web::MetadataList& page_metadata_list) {
+void FilterOutByGroupId(physical_web::MetadataList& page_metadata_list) {
// |std::unique| only removes duplicates that immediately follow each other.
// Thus, first, we have to sort by group_id and distance and only then remove
// duplicates.
@@ -69,18 +68,17 @@ void FilterOutByGroupId(
// Each empty group_id must be treated as unique, so we do not apply
// std::unique to them at all.
- auto nonempty_group_id_begin = std::find_if(
- page_metadata_list.begin(), page_metadata_list.end(),
- [](const physical_web::Metadata& page) {
- return !page.group_id.empty();
- });
+ auto nonempty_group_id_begin =
+ std::find_if(page_metadata_list.begin(), page_metadata_list.end(),
+ [](const physical_web::Metadata& page) {
+ return !page.group_id.empty();
+ });
- auto new_end = std::unique(
- nonempty_group_id_begin, page_metadata_list.end(),
- [](const physical_web::Metadata& left,
- const physical_web::Metadata& right) {
- return left.group_id == right.group_id;
- });
+ auto new_end = std::unique(nonempty_group_id_begin, page_metadata_list.end(),
+ [](const physical_web::Metadata& left,
+ const physical_web::Metadata& right) {
+ return left.group_id == right.group_id;
+ });
page_metadata_list.erase(new_end, page_metadata_list.end());
}
@@ -98,8 +96,8 @@ PhysicalWebPageSuggestionsProvider::PhysicalWebPageSuggestionsProvider(
physical_web_data_source_(physical_web_data_source),
pref_service_(pref_service) {
observer->OnCategoryStatusChanged(this, provided_category_, category_status_);
- physical_web_data_source_->RegisterListener(this,
- physical_web::BACKGROUND_INTERMITTENT);
+ physical_web_data_source_->RegisterListener(
+ this, physical_web::BACKGROUND_INTERMITTENT);
// TODO(vitaliii): Rewrite initial fetch once crbug.com/667754 is resolved.
FetchPhysicalWebPages();
}

Powered by Google App Engine
This is Rietveld 408576698