components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc - Issue 2194203002: Make ContentSuggestionsService recognize new/removed categories

Keyboard Shortcuts

	File
u :	up to issue
j / k :	jump to file after / before current file
J / K :	jump to next file with a comment after / before current file
	Side-by-side diff
i :	toggle intra-line diffs
e :	expand all comments
c :	collapse all comments
s :	toggle showing all comments
n / p :	next / previous diff chunk or comment
N / P :	next / previous comment
<Up> / <Down> :	next / previous line

	Issue
u :	up to list of issues
j / k :	jump to patch after / before current patch
o / <Enter> :	open current patch in side-by-side view
i :	open current patch in unified diff view

	Issue List
j / k :	jump to issue after / before current issue
o / <Enter> :	open current issue

Unified Diff: components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc

Issue 2194203002: Make ContentSuggestionsService recognize new/removed categories (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@renaming

Patch Set: Put the status-change notification back in RegisterProvider() Created 4 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

Index: components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc

diff --git a/components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc b/components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc

index 2cc197268d71c0297a2ae0743e65fef4d84b5d1d..1220ab09b2d9f929df5f394ef7b678fee9732bd9 100644

--- a/components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc

+++ b/components/ntp_snippets/offline_pages/offline_page_suggestions_provider.cc

@@ -128,7 +128,7 @@ void OfflinePageSuggestionsProvider::OnOfflinePagesLoaded(

break;

}

- observer_->OnNewSuggestions(provided_category_, std::move(suggestions));

+ observer_->OnNewSuggestions(this, provided_category_, std::move(suggestions));

}

void OfflinePageSuggestionsProvider::NotifyStatusChanged(

@@ -139,7 +139,7 @@ void OfflinePageSuggestionsProvider::NotifyStatusChanged(

if (!observer_)

return;

- observer_->OnCategoryStatusChanged(provided_category_, new_status);

+ observer_->OnCategoryStatusChanged(this, provided_category_, new_status);

}

} // namespace ntp_snippets

« components/ntp_snippets/content_suggestions_service.cc ('K') | « components/ntp_snippets/ntp_snippets_service.cc ('k') | no next file » | no next file with comments »