Remove host restrictions from snippets

components/ntp_snippets/remote/ntp_snippets_fetcher.cc

 // Copyright 2016 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "components/ntp_snippets/remote/ntp_snippets_fetcher.h"
 
 #include <algorithm>
 #include <cstdlib>
 #include <utility>
 
@@ skipping 698 matching lines @@
 
       auto content_restricts = base::MakeUnique<base::ListValue>();
       for (const auto* metadata : {"TITLE", "SNIPPET", "THUMBNAIL"}) {
         auto entry = base::MakeUnique<base::DictionaryValue>();
         entry->SetString("type", "METADATA");
         entry->SetString("value", metadata);
         content_restricts->Append(std::move(entry));
       }
 
       auto content_selectors = base::MakeUnique<base::ListValue>();
-      for (const auto& host : params_.hosts) {
-        auto entry = base::MakeUnique<base::DictionaryValue>();
-        entry->SetString("type", "HOST_RESTRICT");
-        entry->SetString("value", host);
-        content_selectors->Append(std::move(entry));
-      }
 
       auto local_scoring_params = base::MakeUnique<base::DictionaryValue>();
       local_scoring_params->Set("content_params", std::move(content_params));
       local_scoring_params->Set("content_restricts",
                                 std::move(content_restricts));
       local_scoring_params->Set("content_selectors",
                                 std::move(content_selectors));

[Marc Treib, 2016/12/13 13:27:49]

I think this can now be removed?

[sfiera, 2016/12/13 13:41:57]

Done.

 
       auto global_scoring_params = base::MakeUnique<base::DictionaryValue>();
       global_scoring_params->SetInteger("num_to_return",
                                         params_.count_to_fetch);
       global_scoring_params->SetInteger("sort_type", 1);
 
       auto advanced = base::MakeUnique<base::DictionaryValue>();
       advanced->Set("local_scoring_params", std::move(local_scoring_params));
       advanced->Set("global_scoring_params", std::move(global_scoring_params));
 
       request->SetString("response_detail_level", "STANDARD");
       request->Set("advanced_options", std::move(advanced));
       if (!obfuscated_gaia_id_.empty()) {
         request->SetString("obfuscated_gaia_id", obfuscated_gaia_id_);
       }
       if (!user_locale.empty()) {
         request->SetString("user_locale", user_locale);
       }
       break;
     }
 
     case NTPSnippetsFetcher::CHROME_CONTENT_SUGGESTIONS_API: {
       if (!user_locale.empty()) {
         request->SetString("uiLanguage", user_locale);
       }
 
-      auto regular_hosts = base::MakeUnique<base::ListValue>();
-      for (const auto& host : params_.hosts) {
-        regular_hosts->AppendString(host);
-      }
-      request->Set("regularlyVisitedHostNames", std::move(regular_hosts));
       request->SetString("priority", params_.interactive_request
                                          ? "USER_ACTION"
                                          : "BACKGROUND_PREFETCH");
 
       auto excluded = base::MakeUnique<base::ListValue>();
       for (const auto& id : params_.excluded_ids) {
         excluded->AppendString(id);
         if (excluded->GetSize() >= kMaxExcludedIds) {
           break;
         }
@@ skipping 337 matching lines @@
   NOTREACHED();
   return false;
 }
 
 bool NTPSnippetsFetcher::NeedsAuthentication() const {
   return (personalization_ == Personalization::kPersonal ||
           personalization_ == Personalization::kBoth);
 }
 
 }  // namespace ntp_snippets