Omnibox: HistoryContents Shouldn't *Always* Clear Results on Stop()

chrome/browser/autocomplete/history_contents_provider.cc

Index: chrome/browser/autocomplete/history_contents_provider.cc
diff --git a/chrome/browser/autocomplete/history_contents_provider.cc b/chrome/browser/autocomplete/history_contents_provider.cc
index 190d0a3afc85bbd3186f5ca0caeda03bd8bdfa81..3e097232de74dc4cff341146b2d4295693f41560 100644
--- a/chrome/browser/autocomplete/history_contents_provider.cc
+++ b/chrome/browser/autocomplete/history_contents_provider.cc
@@ -152,9 +152,11 @@ void HistoryContentsProvider::Stop(bool clear_cached_results) {
   request_consumer_.CancelAllRequests();
 
   // Clear the results. We swap in an empty one as the easy way to clear it.
-  history::QueryResults empty_results;
-  results_.Swap(&empty_results);
-  have_results_ = false;
+  if (clear_cached_results) {
+    history::QueryResults empty_results;
+    results_.Swap(&empty_results);
+    have_results_ = false;
+  }
 }
 
 HistoryContentsProvider::~HistoryContentsProvider() {