Commit ff7325bf authored by kkania@chromium.org's avatar kkania@chromium.org

Fix leak in testing_automation_provider.cc by postponing creating a dictionary.

BUG=none
TEST=none


Review URL: http://codereview.chromium.org/8888026

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@113716 0039d316-1c4b-4281-b951-d872f2087c98
parent f9b6a59c
...@@ -6300,10 +6300,10 @@ void TestingAutomationProvider::GetViews( ...@@ -6300,10 +6300,10 @@ void TestingAutomationProvider::GetViews(
for (iter = extension_mgr->begin(); iter != extension_mgr->end(); for (iter = extension_mgr->begin(); iter != extension_mgr->end();
++iter) { ++iter) {
ExtensionHost* host = *iter; ExtensionHost* host = *iter;
DictionaryValue* dict = new DictionaryValue();
AutomationId id = automation_util::GetIdForExtensionView(host); AutomationId id = automation_util::GetIdForExtensionView(host);
if (!id.is_valid()) if (!id.is_valid())
continue; continue;
DictionaryValue* dict = new DictionaryValue();
dict->Set("auto_id", id.ToValue()); dict->Set("auto_id", id.ToValue());
dict->SetString("extension_id", host->extension_id()); dict->SetString("extension_id", host->extension_id());
view_list->Append(dict); view_list->Append(dict);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment