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

chrome/browser/search_engines/template_url_service.cc

Index: chrome/browser/search_engines/template_url_service.cc
diff --git a/chrome/browser/search_engines/template_url_service.cc b/chrome/browser/search_engines/template_url_service.cc
index 912f9b1078b754b9746399600ef56171f7066e4d..8af4bf41987be63236c4c422db1ed1c0c56ddfde 100644
--- a/chrome/browser/search_engines/template_url_service.cc
+++ b/chrome/browser/search_engines/template_url_service.cc
@@ -306,11 +306,14 @@ TemplateURLService::TemplateURLService(const Initializer* initializers,
 }
 
 TemplateURLService::~TemplateURLService() {
+}
+
+void TemplateURLService::Shutdown() {
   if (load_handle_) {

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

Add a comment about why it's important to do this in Shutdown() instead of the
destructor.

     DCHECK(service_.get());
     service_->CancelRequest(load_handle_);
   }
-
+  service_ = NULL;
   STLDeleteElements(&template_urls_);
 }