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

chrome/browser/extensions/api/cookies/cookies_api.cc

 // Copyright (c) 2012 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.
 
 // Implements the Chrome Extensions Cookies API.
 
 #include "chrome/browser/extensions/api/cookies/cookies_api.h"
 
 #include "base/bind.h"
 #include "base/json/json_writer.h"
@@ skipping 46 matching lines @@
       break;
 
     default:
       NOTREACHED();
   }
 }
 
 void ExtensionCookiesEventRouter::CookieChanged(
     Profile* profile,
     ChromeCookieDetails* details) {
-  ListValue args;
+  ListValue* args = new ListValue();
   DictionaryValue* dict = new DictionaryValue();
   dict->SetBoolean(keys::kRemovedKey, details->removed);
   dict->Set(
       keys::kCookieKey,
       cookies_helpers::CreateCookieValue(*details->cookie,
           cookies_helpers::GetStoreIdFromProfile(profile)));
 
   // Map the interal cause to an external string.
   std::string cause;
   switch (details->cause) {
@@ skipping 15 matching lines @@
 
     case net::CookieMonster::Delegate::CHANGE_COOKIE_EXPIRED_OVERWRITE:
       cause = keys::kExpiredOverwriteChangeCause;
       break;
 
     default:
       NOTREACHED();
   }
   dict->SetString(keys::kCauseKey, cause);
 
-  args.Append(dict);
+  args->Append(dict);
 
-  std::string json_args;
-  base::JSONWriter::Write(&args, &json_args);
   GURL cookie_domain =
       cookies_helpers::GetURLFromCanonicalCookie(*details->cookie);
-  DispatchEvent(profile, keys::kOnChanged, json_args, cookie_domain);
+  DispatchEvent(profile, keys::kOnChanged, args, cookie_domain);
 }
 
 void ExtensionCookiesEventRouter::DispatchEvent(Profile* profile,
                                                 const char* event_name,
-                                                const std::string& json_args,
+                                                ListValue* event_args,
                                                 GURL& cookie_domain) {
   if (profile && profile->GetExtensionEventRouter()) {
     profile->GetExtensionEventRouter()->DispatchEventToRenderers(
-        event_name, json_args, profile, cookie_domain, EventFilteringInfo());
+        event_name, event_args, profile, cookie_domain, EventFilteringInfo());
   }
 }
 
 bool CookiesFunction::ParseUrl(const DictionaryValue* details, GURL* url,
                                bool check_host_permissions) {
   DCHECK(details && url);
   std::string url_string;
   // Get the URL string or return false.
   EXTENSION_FUNCTION_VALIDATE(details->GetString(keys::kUrlKey, &url_string));
   *url = GURL(url_string);
@@ skipping 409 matching lines @@
   }
   result_.reset(cookie_store_list);
   return true;
 }
 
 void GetAllCookieStoresFunction::Run() {
   SendResponse(RunImpl());
 }
 
 }  // namespace extensions