Implement ClearHistory for all ntp_snippets providers

components/ntp_snippets/ntp_snippets_service.cc

Index: components/ntp_snippets/ntp_snippets_service.cc
diff --git a/components/ntp_snippets/ntp_snippets_service.cc b/components/ntp_snippets/ntp_snippets_service.cc
index dbc71396623bd40ad0a68ef98ba47fa854d98b6e..1fb9fd5f642e8e52449eeadcc33210efd8144715 100644
--- a/components/ntp_snippets/ntp_snippets_service.cc
+++ b/components/ntp_snippets/ntp_snippets_service.cc
@@ -339,7 +339,13 @@ void NTPSnippetsService::ClearHistory(
     base::Time begin,
     base::Time end,
     const base::Callback<bool(const GURL& url)>& filter) {
-  // TODO(vitaliii): Implement. See crbug.com/641321.
+  // Note that both time range and the filter are ignored and all suggestions
+  // are removed, because it is not known which history entries were used for
+  // the suggestions personalization.
+  if (!ready())
+    nuke_after_load_ = true;
+  else
+    NukeAllSnippets();

[Marc Treib, 2016/09/01 14:11:20]

Since this four-line block now exists twice, we could put it into its own helper
method. Not sure what a good name would be though...

[vitaliii, 2016/09/01 14:45:15]

Acknowledged.

 }
 
 void NTPSnippetsService::ClearCachedSuggestions(Category category) {