Fix leak in JsSyncManagerObserver

sync/internal_api/js_sync_manager_observer.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 "sync/internal_api/js_sync_manager_observer.h"
 
 #include <cstddef>
 
 #include "base/location.h"
 #include "base/logging.h"
@@ skipping 113 matching lines @@
 
 void JsSyncManagerObserver::OnInitializationComplete(
     const WeakHandle<JsBackend>& js_backend,
     bool success, syncer::ModelTypeSet restored_types) {
   if (!event_handler_.IsInitialized()) {
     return;
   }
   // Ignore the |js_backend| argument; it's not really convertible to
   // JSON anyway.
 
-  DictionaryValue* details = new DictionaryValue();
-  details->Set("restoredTypes", ModelTypeSetToValue(restored_types));
+  DictionaryValue details;
+  details.Set("restoredTypes", ModelTypeSetToValue(restored_types));
 
-  HandleJsEvent(FROM_HERE, "onInitializationComplete", JsEventDetails(details));
+  HandleJsEvent(FROM_HERE,
+                "onInitializationComplete",
+                JsEventDetails(&details));
 }
 
 void JsSyncManagerObserver::OnStopSyncingPermanently() {
   if (!event_handler_.IsInitialized()) {
     return;
   }
   HandleJsEvent(FROM_HERE, "onStopSyncingPermanently", JsEventDetails());
 }
 
 void JsSyncManagerObserver::HandleJsEvent(
     const tracked_objects::Location& from_here,
     const std::string& name, const JsEventDetails& details) {
   if (!event_handler_.IsInitialized()) {
     NOTREACHED();
     return;
   }
   event_handler_.Call(from_here,
                       &JsEventHandler::HandleJsEvent, name, details);
 }
 
 }  // namespace syncer