| 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();
|
| }
|
|
|