[NTP Snippets] Unschedule fetches when the service should be disabled

chrome/browser/ui/webui/snippets_internals_message_handler.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 "chrome/browser/ui/webui/snippets_internals_message_handler.h"
 
 #include <memory>
 #include <set>
 #include <vector>
 
@@ skipping 103 matching lines @@
   DCHECK_EQ(0u, args->GetSize());
 
   ntp_snippets_service_->ClearSnippets();
 }
 
 void SnippetsInternalsMessageHandler::HandleClearDiscarded(
     const base::ListValue* args) {
   DCHECK_EQ(0u, args->GetSize());
 
   ntp_snippets_service_->ClearDiscardedSnippets();
+  ntp_snippets_service_->FetchSnippets();

[Marc Treib, 2016/06/07 08:55:39]

Hm. So this is the only call site for ClearDiscardedSnippets?
Now that I see it, maybe it's actually better not to fetch automatically here?
Less "magic" in the debug UI... WDYT?

[dgn, 2016/06/07 11:22:46]

I think so too. And when debugging the ntp, after clearing a refresh is required
to pull the new snippets anyway, so it doesn't affect it. Removed this then.

   SendDiscardedSnippets();
 }
 
 void SnippetsInternalsMessageHandler::HandleDownload(
     const base::ListValue* args) {
   DCHECK_EQ(1u, args->GetSize());
 
   SendString("hosts-status", std::string());
 
   std::string hosts_string;
@@ skipping 90 matching lines @@
 }
 
 void SnippetsInternalsMessageHandler::SendString(const std::string& name,
                                                  const std::string& value) {
   base::StringValue string_name(name);
   base::StringValue string_value(value);
 
   web_ui()->CallJavascriptFunction("chrome.SnippetsInternals.receiveProperty",
                                    string_name, string_value);
 }