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

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

Issue 2570673004: Remove host restrictions from snippets (Closed)
Patch Set: Remove host restrictions from snippets 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 988dc19ce21135d458c7272408c78501d1934be9..87f7939159b1f360d490815572aa975ed9f96468 100644
--- a/chrome/browser/ui/webui/snippets_internals_message_handler.cc
+++ b/chrome/browser/ui/webui/snippets_internals_message_handler.cc
@@ -182,23 +182,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 (!ntp_snippets_service_)
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());
-
- ntp_snippets_service_->FetchSnippetsFromHosts(hosts,
- /*interactive_request=*/true);
+ ntp_snippets_service_->FetchSnippetsForAllCategories();
}
void SnippetsInternalsMessageHandler::HandleClearCachedSuggestions(
@@ -378,7 +369,7 @@ void SnippetsInternalsMessageHandler::SendContentSuggestions() {
const std::string& status =
ntp_snippets_service_->snippets_fetcher()->last_status();
if (!status.empty())
- SendString("hosts-status", "Finished: " + status);
+ SendString("remote-status", "Finished: " + status);
}
base::DictionaryValue result;

Powered by Google App Engine
This is Rietveld 408576698