Cancel pending requests for TemplateURLs in Shutdown, instead of dtor.

chrome/browser/search_engines/template_url_service.h

Index: chrome/browser/search_engines/template_url_service.h
diff --git a/chrome/browser/search_engines/template_url_service.h b/chrome/browser/search_engines/template_url_service.h
index 643dd0ae095fc7e1f181dce038c7a214d1ecf831..7024f43a1fc2eff79962c534c6b9b752f9985aa2 100644
--- a/chrome/browser/search_engines/template_url_service.h
+++ b/chrome/browser/search_engines/template_url_service.h
@@ -84,6 +84,9 @@ class TemplateURLService : public WebDataServiceConsumer,
   TemplateURLService(const Initializer* initializers, const int count);
   virtual ~TemplateURLService();
 
+  // ProfileKeyedService implementation.
+  virtual void Shutdown() OVERRIDE;

[Peter Kasting, 2013/04/12 22:32:17]

Please declare this just above Observe() so it is with the other overrides in
the same order as the parent class list.

.cc file order should be kept in sync.

+
   // Generates a suitable keyword for the specified url, which must be valid.
   // This is guaranteed not to return an empty string, since TemplateURLs should
   // never have an empty keyword.