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

chrome/browser/extensions/api/alarms/alarm_manager.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/alarms/alarm_manager.h"
 
 #include "base/bind.h"
 #include "base/json/json_writer.h"
 #include "base/message_loop.h"
 #include "base/time.h"
@@ skipping 20 matching lines @@
 // The minimum period between polling for alarms to run.
 const base::TimeDelta kDefaultMinPollPeriod = base::TimeDelta::FromMinutes(5);
 
 class DefaultAlarmDelegate : public AlarmManager::Delegate {
  public:
   explicit DefaultAlarmDelegate(Profile* profile) : profile_(profile) {}
   virtual ~DefaultAlarmDelegate() {}
 
   virtual void OnAlarm(const std::string& extension_id,
                        const Alarm& alarm) {
-    ListValue args;
-    std::string json_args;
-    args.Append(alarm.js_alarm->ToValue().release());
-    base::JSONWriter::Write(&args, &json_args);
+    ListValue* args = new ListValue();
+    args->Append(alarm.js_alarm->ToValue().release());
     ExtensionSystem::Get(profile_)->event_router()->DispatchEventToExtension(

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

I'm guessing I'll see later on how DispatchEventToExtension() takes ownership of
args.

-        extension_id, kOnAlarmEvent, json_args, NULL, GURL());
+        extension_id, kOnAlarmEvent, args, NULL, GURL());
   }
 
  private:
   Profile* profile_;
 };
 
 // Creates a TimeDelta from a delay as specified in the API.
 base::TimeDelta TimeDeltaFromDelay(double delay_in_minutes) {
   return base::TimeDelta::FromMicroseconds(
       delay_in_minutes * base::Time::kMicrosecondsPerMinute);
@@ skipping 322 matching lines @@
   if (create_info.period_in_minutes.get()) {
     js_alarm->period_in_minutes.reset(
         new double(*create_info.period_in_minutes));
   }
 }
 
 Alarm::~Alarm() {
 }
 
 }  // namespace extensions