If a test aborted (e.g. ASSERT_EQ() failure) with a TemplateURLFetcher running, shutdown would go b…

If a test aborted (e.g. ASSERT_EQ() failure) with a TemplateURLFetcher running, shutdown would go badly, leading to a DCHECK failure.  In the real browser this isn't an issue because the URLFetchers are shut down correctly, but in the TestingProfile we don't have all the same shutdown machinery so resetting the TemplateURLFetcher explicitly avoids the potential problem.

BUG=none
TEST=none
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=126229

[Peter Kasting, 2012-03-10 01:39:20]

Will, this is the belated review for a fix we discussed in email almost a month
ago (see thread "URLFetcherImpl threading requirements and shutdown?").

[willchan no longer on Chromium, 2012-03-12 20:34:39]

lgtm