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 <vector>
 
@@ skipping 62 matching lines @@
       break;
 
     default:
       NOTREACHED();
   }
 }
 
 void ExtensionCookiesEventRouter::CookieChanged(
     Profile* profile,
     ChromeCookieDetails* details) {
-  ListValue args;
+  scoped_ptr<ListValue> args(new ListValue());
   DictionaryValue* dict = new DictionaryValue();
   dict->SetBoolean(keys::kRemovedKey, details->removed);
 
   scoped_ptr<Cookie> cookie(
       cookies_helpers::CreateCookie(*details->cookie,
           cookies_helpers::GetStoreIdFromProfile(profile)));
   dict->Set(keys::kCookieKey, cookie->ToValue().release());
 
   // Map the interal cause to an external string.
   std::string cause;
@@ skipping 16 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.Pass(), cookie_domain);
 }
 
-void ExtensionCookiesEventRouter::DispatchEvent(Profile* profile,
-                                                const char* event_name,
-                                                const std::string& json_args,
-                                                GURL& cookie_domain) {
+void ExtensionCookiesEventRouter::DispatchEvent(
+    Profile* profile, const char* event_name, scoped_ptr<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.Pass(), profile, cookie_domain,
+        EventFilteringInfo());
   }
 }
 
 bool CookiesFunction::ParseUrl(const std::string& url_string, GURL* url,
                                bool check_host_permissions) {
   *url = GURL(url_string);
   if (!url->is_valid()) {
     error_ = ExtensionErrorUtils::FormatErrorMessage(
         keys::kInvalidUrlError, url_string);
     return false;
@@ skipping 392 matching lines @@
   }
   results_ = GetAllCookieStores::Results::Create(cookie_stores);
   return true;
 }
 
 void GetAllCookieStoresFunction::Run() {
   SendResponse(RunImpl());
 }
 
 }  // namespace extensions