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

chrome/browser/extensions/extension_event_router_forwarder.cc

-// Copyright (c) 2011 The Chromium Authors. All rights reserved.
+// 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.
 
 #include "chrome/browser/extensions/extension_event_router_forwarder.h"
 
 #include "base/bind.h"
+#include "base/values.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/extensions/extension_event_router.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "googleurl/src/gurl.h"
 
 using content::BrowserThread;
 
 ExtensionEventRouterForwarder::ExtensionEventRouterForwarder() {
 }
 
 ExtensionEventRouterForwarder::~ExtensionEventRouterForwarder() {
 }
 
 void ExtensionEventRouterForwarder::BroadcastEventToRenderers(
     const std::string& event_name,
-    const std::string& event_args,
+    base::ListValue* event_args,
     const GURL& event_url) {
   HandleEvent("", event_name, event_args, 0, true, event_url);
 }
 
 void ExtensionEventRouterForwarder::DispatchEventToRenderers(
     const std::string& event_name,
-    const std::string& event_args,
+    base::ListValue* event_args,
     void* profile,
     bool use_profile_to_restrict_events,
     const GURL& event_url) {
   if (!profile)
     return;
   HandleEvent("", event_name, event_args, profile,
               use_profile_to_restrict_events, event_url);
 }
 
 void ExtensionEventRouterForwarder::BroadcastEventToExtension(
     const std::string& extension_id,
     const std::string& event_name,
-    const std::string& event_args,
+    base::ListValue* event_args,
     const GURL& event_url) {
   HandleEvent(extension_id, event_name, event_args, 0, true, event_url);
 }
 
 void ExtensionEventRouterForwarder::DispatchEventToExtension(
     const std::string& extension_id,
     const std::string& event_name,
-    const std::string& event_args,
+    base::ListValue* event_args,
     void* profile,
     bool use_profile_to_restrict_events,
     const GURL& event_url) {
   if (!profile)
     return;
   HandleEvent(extension_id, event_name, event_args, profile,
               use_profile_to_restrict_events, event_url);
 }
 
 void ExtensionEventRouterForwarder::HandleEvent(
     const std::string& extension_id,
     const std::string& event_name,
-    const std::string& event_args,
+    base::ListValue* event_args,
     void* profile_ptr,
     bool use_profile_to_restrict_events,
     const GURL& event_url) {
   if (!BrowserThread::CurrentlyOn(BrowserThread::UI)) {
     BrowserThread::PostTask(
         BrowserThread::UI, FROM_HERE,
         base::Bind(&ExtensionEventRouterForwarder::HandleEvent, this,
                    extension_id, event_name, event_args, profile_ptr,
                    use_profile_to_restrict_events, event_url));
     return;
@@ skipping 20 matching lines @@
           profiles[i], extension_id, event_name, event_args,
           use_profile_to_restrict_events ? profiles[i] : NULL, event_url);
     }
   }
 }
 
 void ExtensionEventRouterForwarder::CallExtensionEventRouter(
     Profile* profile,
     const std::string& extension_id,
     const std::string& event_name,
-    const std::string& event_args,
+    base::ListValue* event_args,
     Profile* restrict_to_profile,
     const GURL& event_url) {
   // We may not have an extension in cases like chromeos login
   // (crosbug.com/12856), chrome_frame_net_tests.exe which reuses the chrome
   // browser single process framework.
   if (!profile->GetExtensionEventRouter())
     return;
 
   if (extension_id.empty()) {
     profile->GetExtensionEventRouter()->
         DispatchEventToRenderers(
             event_name, event_args, restrict_to_profile, event_url);
   } else {
     profile->GetExtensionEventRouter()->
         DispatchEventToExtension(
             extension_id,
             event_name, event_args, restrict_to_profile, event_url);
   }
 }