Release all DataSources before the main thread goes away.

chrome/browser/dom_ui/chrome_url_data_manager.cc

Index: chrome/browser/dom_ui/chrome_url_data_manager.cc
diff --git a/chrome/browser/dom_ui/chrome_url_data_manager.cc b/chrome/browser/dom_ui/chrome_url_data_manager.cc
index d0ca42c894b292754412e1544d8efa84566a095a..9c64f908a32edbf830c45803a9c43914e0494ce0 100644
--- a/chrome/browser/dom_ui/chrome_url_data_manager.cc
+++ b/chrome/browser/dom_ui/chrome_url_data_manager.cc
@@ -207,6 +207,15 @@ void ChromeURLDataManager::AddDataSource(scoped_refptr<DataSource> source) {
   data_sources_[source->source_name()] = source;
 }
 
+void ChromeURLDataManager::RemoveAllDataSources() {
+  for (DataSourceMap::iterator i = data_sources_.begin();

[eroman, 2011/01/25 23:47:35]

why not simply data_sources_.clear() ?

[ahendrickson, 2011/01/26 04:34:38]

I wanted to call Release() rather than destruct the items directly.

+       i != data_sources_.end();
+       i = data_sources_.begin()) {
+    (*i).second = NULL;  // Calls Release().
+    data_sources_.erase(i);
+  }
+}
+
 void ChromeURLDataManager::AddFileSource(const std::string& source_name,
                                          const FilePath& file_path) {
   DCHECK(file_sources_.count(source_name) == 0);