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

chrome/browser/extensions/api/app/app_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.
 
 #include "chrome/browser/extensions/api/app/app_api.h"
 
 #include "base/json/json_writer.h"
 #include "base/string_number_conversions.h"
 #include "base/time.h"
 #include "base/utf_string_conversions.h"
@@ skipping 103 matching lines @@
 
   AppNotificationManager* manager =
       profile()->GetExtensionService()->app_notification_manager();
   manager->ClearAll(id);
   return true;
 }
 
 // static.
 void AppEventRouter::DispatchOnLaunchedEvent(
     Profile* profile, const Extension* extension) {
+  scoped_ptr<ListValue> arguments(new ListValue());
   profile->GetExtensionEventRouter()->DispatchEventToExtension(
-      extension->id(), kOnLaunchedEvent, "[]", NULL, GURL());
+      extension->id(), kOnLaunchedEvent, arguments.Pass(), NULL, GURL());
 }
 
 // static.
 void AppEventRouter::DispatchOnLaunchedEventWithFileEntry(
     Profile* profile, const Extension* extension, const string16& action,
     const std::string& file_system_id, const std::string& base_name) {
-  ListValue args;
+  scoped_ptr<ListValue> args(new ListValue());
   DictionaryValue* launch_data = new DictionaryValue();
   DictionaryValue* intent = new DictionaryValue();
   intent->SetString("action", UTF16ToUTF8(action));
   intent->SetString("type", "chrome-extension://fileentry");
   launch_data->Set("intent", intent);
-  args.Append(launch_data);
+  args->Append(launch_data);
   DictionaryValue* intent_data = new DictionaryValue();
   intent_data->SetString("format", "fileEntry");
   intent_data->SetString("fileSystemId", file_system_id);
   intent_data->SetString("baseName", base_name);
-  args.Append(intent_data);
-  std::string json_args;
-  base::JSONWriter::Write(&args, &json_args);
+  args->Append(intent_data);
   profile->GetExtensionEventRouter()->DispatchEventToExtension(
-      extension->id(), kOnLaunchedEvent, json_args, NULL, GURL());
+      extension->id(), kOnLaunchedEvent, args.Pass(), NULL, GURL());
 }
 
 // static.
 void AppEventRouter::DispatchOnLaunchedEventWithWebIntent(
     Profile* profile, const Extension* extension,
     const webkit_glue::WebIntentData web_intent_data) {
-  ListValue args;
+  scoped_ptr<ListValue> args(new ListValue());
   DictionaryValue* launch_data = new DictionaryValue();
   DictionaryValue* intent = new DictionaryValue();
   intent->SetString("action", UTF16ToUTF8(web_intent_data.action));
   intent->SetString("type", UTF16ToUTF8(web_intent_data.type));
   launch_data->Set("intent", intent);
-  args.Append(launch_data);
+  args->Append(launch_data);
   DictionaryValue* intent_data;
   switch (web_intent_data.data_type) {
     case webkit_glue::WebIntentData::SERIALIZED:
       intent_data = new DictionaryValue();
       intent_data->SetString("format", "serialized");
       intent_data->SetString("data", UTF16ToUTF8(web_intent_data.data));
-      args.Append(intent_data);
+      args->Append(intent_data);
       break;
     case webkit_glue::WebIntentData::UNSERIALIZED:
       intent->SetString("data", UTF16ToUTF8(web_intent_data.unserialized_data));
       break;
     case webkit_glue::WebIntentData::BLOB:
       intent_data = new DictionaryValue();
       intent_data->SetString("format", "blob");
       intent_data->SetString("blobFileName", web_intent_data.blob_file.value());
       intent_data->SetString("blobLength",
                              base::Int64ToString(web_intent_data.blob_length));
-      args.Append(intent_data);
+      args->Append(intent_data);
       break;
     default:
       NOTREACHED();
       break;
   }
-  std::string json_args;
-  base::JSONWriter::Write(&args, &json_args);
   profile->GetExtensionEventRouter()->DispatchEventToExtension(
-      extension->id(), kOnLaunchedEvent, json_args, NULL, GURL());
+      extension->id(), kOnLaunchedEvent, args.Pass(), NULL, GURL());
 }
 
 }  // namespace extensions