Switch the WebstoreDataFetcherDelegate::OnWebstoreResponseParseSuccess interface to scoped_ptr.

Switch the WebstoreDataFetcherDelegate::OnWebstoreResponseParseSuccess interface to scoped_ptr.

Fixes memory leaks.

BUG=313695
R=benwells@chromium.org
TBR=satorux@chromium.org

Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=232877

[earthdok, 2013-10-31 14:54:37]

PTAL

benwells@ - c/b/extensions
satorux@ - c/b/chromeos

[benwells, 2013-11-01 00:01:33]

On 2013/10/31 14:54:37, earthdok wrote:
> PTAL
> 
> benwells@ - c/b/extensions
> satorux@ - c/b/chromeos

I'm getting 'old chunk mismatch'. Can you reupload?

[earthdok, 2013-11-01 01:07:47]

On 2013/11/01 00:01:33, benwells wrote:
> On 2013/10/31 14:54:37, earthdok wrote:
> > PTAL
> > 
> > benwells@ - c/b/extensions
> > satorux@ - c/b/chromeos
> 
> I'm getting 'old chunk mismatch'. Can you reupload?

Done.
Please tick the CQ if you approve.

[benwells, 2013-11-01 06:01:01]

c/b/e lgtm. Were there actual leaks here or is this being proactive for future
leaks?

https://codereview.chromium.org/51113014/diff/80001/chrome/browser/extensions...
File chrome/browser/extensions/webstore_data_fetcher.cc (right):

https://codereview.chromium.org/51113014/diff/80001/chrome/browser/extensions...
chrome/browser/extensions/webstore_data_fetcher.cc:57:
static_cast<base::DictionaryValue*>(parsed_json.release())));
fun times :)

[satorux1, 2013-11-01 06:02:53]

c/b/chromeos lgtm

[commit-bot: I haz the power, 2013-11-01 06:04:11]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/earthdok@chromium.org/51113014/80001

[commit-bot: I haz the power, 2013-11-01 06:41:34]

Sorry for I got bad news for ya.
Compile failed with a clobber build on linux_chromeos_clang.
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=linux_chro...
Your code is likely broken or HEAD is junk. Please ensure your
code is not broken then alert the build sheriffs.
Look at the try server FAQ for more details.

[earthdok, 2013-11-01 07:49:51]

On 2013/11/01 06:01:01, benwells wrote:
> c/b/e lgtm. Were there actual leaks here or is this being proactive for future
> leaks?

There were. See the bug linked in the description.

[commit-bot: I haz the power, 2013-11-04 17:42:49]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/earthdok@chromium.org/51113014/360001

[commit-bot: I haz the power, 2013-11-04 22:44:44]

Retried try job too often on win7_aura for step(s) browser_tests
http://build.chromium.org/p/tryserver.chromium/buildstatus?builder=win7_aura&...

[commit-bot: I haz the power, 2013-11-04 23:32:34]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/earthdok@chromium.org/51113014/360001

[commit-bot: I haz the power, 2013-11-05 01:35:10]

Change committed as 232877