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

Unified Diff: chrome/browser/ui/webui/snippets_internals_message_handler.cc

Issue 2570673004: Remove host restrictions from snippets (Closed)
Patch Set: rebase Created 4 years 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: chrome/browser/ui/webui/snippets_internals_message_handler.cc
diff --git a/chrome/browser/ui/webui/snippets_internals_message_handler.cc b/chrome/browser/ui/webui/snippets_internals_message_handler.cc
index a0cd45ffad9ae9c018dcce10e4b590cabd2975d1..c32b36c4355fb5317082bdb2a46a42dcebaacfdd 100644
--- a/chrome/browser/ui/webui/snippets_internals_message_handler.cc
+++ b/chrome/browser/ui/webui/snippets_internals_message_handler.cc
@@ -188,24 +188,14 @@ void SnippetsInternalsMessageHandler::HandleRefreshContent(
void SnippetsInternalsMessageHandler::HandleDownload(
const base::ListValue* args) {
- DCHECK_EQ(1u, args->GetSize());
+ DCHECK_EQ(0u, args->GetSize());
- SendString("hosts-status", std::string());
+ SendString("remote-status", std::string());
if (!remote_suggestions_provider_)
return;
- std::string hosts_string;
- if (!args->GetString(0, &hosts_string))
- return;
-
- std::vector<std::string> hosts_vector = base::SplitString(
- hosts_string, " ", base::TRIM_WHITESPACE, base::SPLIT_WANT_NONEMPTY);
- std::set<std::string> hosts(hosts_vector.begin(), hosts_vector.end());
-
- remote_suggestions_provider_->FetchSnippetsFromHosts(
- hosts,
- /*interactive_request=*/true);
+ remote_suggestions_provider_->FetchSnippetsForAllCategories();
}
void SnippetsInternalsMessageHandler::HandleClearCachedSuggestions(
@@ -404,7 +394,7 @@ void SnippetsInternalsMessageHandler::SendContentSuggestions() {
const std::string& status =
remote_suggestions_provider_->snippets_fetcher()->last_status();
if (!status.empty())
- SendString("hosts-status", "Finished: " + status);
+ SendString("remote-status", "Finished: " + status);
}
base::DictionaryValue result;
« no previous file with comments | « chrome/browser/resources/snippets_internals.js ('k') | components/ntp_snippets/remote/ntp_snippets_fetcher.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698