MD Settings: Use FireWebUIListener() helper wherever possible.

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 90 matching lines @@
   model_util_->GetChildNodeList(
       parent_node, start, count, /*include_quota_nodes=*/false, children.get());
 
   base::DictionaryValue args;
   if (parent == tree_model->GetRoot())
     args.Set(kId, base::MakeUnique<base::Value>());
   else
     args.SetString(kId, model_util_->GetTreeNodeId(parent_node));
   args.SetInteger(kStart, start);
   args.Set(kChildren, std::move(children));
-  CallJavascriptFunction("cr.webUIListenerCallback",
-                         base::Value("onTreeItemAdded"), args);
+  FireWebUIListener("on.TreeItemAdded", args);

[dpapad, 2017/05/19 19:39:43]

Whoops. Something wrong with my regex capturing. I'll re-confirm manually that
all changes are sane.

[dpapad, 2017/05/19 20:35:35]

Fixed. This was the only occurrence I found.

 }
 
 void CookiesViewHandler::TreeNodesRemoved(ui::TreeModel* model,
                                           ui::TreeModelNode* parent,
                                           int start,
                                           int count) {
   // Skip if there is a batch update in progress.
   if (batch_update_)
     return;
 
   CookiesTreeModel* tree_model = static_cast<CookiesTreeModel*>(model);
 
   base::DictionaryValue args;
   if (parent == tree_model->GetRoot())
     args.Set(kId, base::MakeUnique<base::Value>());
   else
     args.SetString(kId, model_util_->GetTreeNodeId(tree_model->AsNode(parent)));
   args.SetInteger(kStart, start);
   args.SetInteger(kCount, count);
-  CallJavascriptFunction("cr.webUIListenerCallback",
-                         base::Value("onTreeItemRemoved"), args);
+  FireWebUIListener("onTreeItemRemoved", args);
 }
 
 void CookiesViewHandler::TreeModelBeginBatch(CookiesTreeModel* model) {
   DCHECK(!batch_update_);  // There should be no nested batch begin.
   batch_update_ = true;
 }
 
 void CookiesViewHandler::TreeModelEndBatch(CookiesTreeModel* model) {
   DCHECK(batch_update_);
   batch_update_ = false;
@@ skipping 134 matching lines @@
     args.Set(kId, base::MakeUnique<base::Value>());
   else
     args.SetString(kId, model_util_->GetTreeNodeId(parent));
   args.Set(kChildren, std::move(children));
 
   ResolveJavascriptCallback(base::Value(callback_id_), args);
   callback_id_.clear();
 }
 
 }  // namespace settings