Manage ChromeToMobile URLFetcher lifetimes with a ScopedVector.

Manage ChromeToMobile URLFetcher lifetimes with a ScopedVector.

TBR=sky@chromium.org,rsimha@chromium.org
BUG=146826
TEST=Automated tests no longer fail with leaked URLFetchers.

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=155417

[msw, 2012-09-07 15:46:09]

Please take a look; thanks!

[erikwright (departed), 2012-09-07 15:50:56]

On 2012/09/07 15:46:09, msw wrote:
> Please take a look; thanks!

In general, this seems like it ought to improve the situation, assuming that the
ChromeToMobileService is itself being destroyed before the profile goes away.

Is that the case?

[msw, 2012-09-07 15:52:53]

On 2012/09/07 15:50:56, erikwright wrote:
> On 2012/09/07 15:46:09, msw wrote:
> > Please take a look; thanks!
> 
> In general, this seems like it ought to improve the situation, assuming that
the
> ChromeToMobileService is itself being destroyed before the profile goes away.
> 
> Is that the case?

ChromeToMobileService is a ProfileKeyedService, so you're correct, that is the
case.

[erikwright (departed), 2012-09-07 15:59:45]

LGTM.

[msw, 2012-09-07 16:04:01]

TBR'ed sky and rsimha, sending to the CQ to green up the tree.

[commit-bot: I haz the power, 2012-09-07 16:06:40]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/msw@chromium.org/10914147/1

[commit-bot: I haz the power, 2012-09-07 17:45:56]

Change committed as 155417

[rsimha_google_not_chromium, 2012-09-07 21:17:28]

LGTM. Thanks for the fix!