Site Settings Desktop: Implement individual cookie removal and RemoveAll.

chrome/browser/ui/webui/settings/settings_cookies_view_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/settings/settings_cookies_view_handler.h"
 
 #include <string>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 159 matching lines @@
   base::string16 query;
   if (!args->GetString(0, &query))
     return;
 
   EnsureCookiesTreeModelCreated();
 
   cookies_tree_model_->UpdateSearchResults(query);
 }
 
 void CookiesViewHandler::RemoveAll(const base::ListValue* args) {
+  CHECK_EQ(1U, args->GetSize());
+  CHECK(args->GetString(0, &callback_id_));
+
   EnsureCookiesTreeModelCreated();
   cookies_tree_model_->DeleteAllStoredObjects();
 }
 
 void CookiesViewHandler::Remove(const base::ListValue* args) {
   std::string node_path;
   if (!args->GetString(0, &node_path))
     return;
 
   EnsureCookiesTreeModelCreated();
 
   const CookieTreeNode* node = model_util_->GetTreeNodeFromPath(
       cookies_tree_model_->GetRoot(), node_path);
   if (node)
     cookies_tree_model_->DeleteCookieNode(const_cast<CookieTreeNode*>(node));
 }
 
 void CookiesViewHandler::LoadChildren(const base::ListValue* args) {
+  CHECK_LT(0U, args->GetSize());
+  CHECK(args->GetString(0, &callback_id_));
+
   std::string node_path;
-  if (!args->GetString(0, &node_path))
+  if (!args->GetString(1, &node_path))
     return;
 
   EnsureCookiesTreeModelCreated();
 
   const CookieTreeNode* node = model_util_->GetTreeNodeFromPath(
       cookies_tree_model_->GetRoot(), node_path);
   if (node)
     SendChildren(node);
 }
 
 void CookiesViewHandler::SendChildren(const CookieTreeNode* parent) {
   AllowJavascript();
 
   std::unique_ptr<base::ListValue> children(new base::ListValue);
   model_util_->GetChildNodeList(parent, 0, parent->child_count(),
                                 children.get());
 
   base::ListValue args;
   if (parent == cookies_tree_model_->GetRoot())
     args.Append(base::Value::CreateNullValue());
   else
     args.AppendString(model_util_->GetTreeNodeId(parent));
   args.Append(std::move(children));
 
-  CallJavascriptFunction("cr.webUIListenerCallback",
-                         base::StringValue("loadChildren"),
-                         args);
+  ResolveJavascriptCallback(base::StringValue(callback_id_), args);
+  callback_id_ = "";
 }
 
 void CookiesViewHandler::ReloadCookies(const base::ListValue* args) {
+  CHECK_EQ(1U, args->GetSize());
+  CHECK(args->GetString(0, &callback_id_));
+
   cookies_tree_model_.reset();
 
   EnsureCookiesTreeModelCreated();
 }
 
 }  // namespace settings