diff options
| author | rlarocque@chromium.org <rlarocque@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-07-30 23:09:36 +0000 |
|---|---|---|
| committer | rlarocque@chromium.org <rlarocque@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-07-30 23:09:36 +0000 |
| commit | 32e757aa2d481facde67b3a4010c4c65b71bd10b (patch) | |
| tree | 6ca74d5ff87f85a26d6686a4e359819cda0d543c /sync | |
| parent | ef7b89304eacc94ae75cb75becf193e79fa91b72 (diff) | |
| download | chromium_src-32e757aa2d481facde67b3a4010c4c65b71bd10b.zip chromium_src-32e757aa2d481facde67b3a4010c4c65b71bd10b.tar.gz chromium_src-32e757aa2d481facde67b3a4010c4c65b71bd10b.tar.bz2 | |
Fix leak in JsSyncManagerObserver
This leak was introduced in r148926.
BUG=129825,139635
Review URL: https://chromiumcodereview.appspot.com/10821091
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@149062 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'sync')
| -rw-r--r-- | sync/internal_api/js_sync_manager_observer.cc | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/sync/internal_api/js_sync_manager_observer.cc b/sync/internal_api/js_sync_manager_observer.cc index 078e1e9..f017107 100644 --- a/sync/internal_api/js_sync_manager_observer.cc +++ b/sync/internal_api/js_sync_manager_observer.cc @@ -131,10 +131,12 @@ void JsSyncManagerObserver::OnInitializationComplete( // 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() { |