Refactor extension event distribution to use Values instead of JSON strings.

chrome/browser/bookmarks/bookmark_manager_extension_api.cc

Index: chrome/browser/bookmarks/bookmark_manager_extension_api.cc
diff --git a/chrome/browser/bookmarks/bookmark_manager_extension_api.cc b/chrome/browser/bookmarks/bookmark_manager_extension_api.cc
index 4679096644c041f6a3b14c3ad7af4a9429e51431..d8887795496c4a4130c47065b9b7850576ffd0c8 100644
--- a/chrome/browser/bookmarks/bookmark_manager_extension_api.cc
+++ b/chrome/browser/bookmarks/bookmark_manager_extension_api.cc
@@ -166,14 +166,12 @@ BookmarkManagerExtensionEventRouter::~BookmarkManagerExtensionEventRouter() {
 }
 
 void BookmarkManagerExtensionEventRouter::DispatchEvent(const char* event_name,
-                                                        const ListValue* args) {
+                                                        ListValue* args) {
   if (!profile_->GetExtensionEventRouter())
     return;
 
-  std::string json_args;
-  base::JSONWriter::Write(args, &json_args);
   profile_->GetExtensionEventRouter()->DispatchEventToRenderers(
-      event_name, json_args, NULL, GURL(), extensions::EventFilteringInfo());
+      event_name, args, NULL, GURL(), extensions::EventFilteringInfo());
 }
 
 void BookmarkManagerExtensionEventRouter::DispatchDragEvent(
@@ -181,9 +179,9 @@ void BookmarkManagerExtensionEventRouter::DispatchDragEvent(
   if (data.size() == 0)
     return;
 
-  ListValue args;
-  BookmarkNodeDataToJSON(profile_, data, &args);
-  DispatchEvent(event_name, &args);
+  ListValue* args = new ListValue();

[miket_OOO, 2012/07/10 22:33:19]

I talked to Antony and confirmed that this apparent new leak is fixed (though
the way it leaked, and the way it's fixed, it more complex than it appears
here).

+  BookmarkNodeDataToJSON(profile_, data, args);
+  DispatchEvent(event_name, args);
 }
 
 void BookmarkManagerExtensionEventRouter::OnDragEnter(