Fix Omnibox search provider's confusing internal variable semantics

chrome/browser/autocomplete/search_provider.h

Index: chrome/browser/autocomplete/search_provider.h
diff --git a/chrome/browser/autocomplete/search_provider.h b/chrome/browser/autocomplete/search_provider.h
index 50631579c5f06660607687e6e643e86361431ce1..1885f2dd9f2f24e94320e96b18f593ded6c2e21c 100644
--- a/chrome/browser/autocomplete/search_provider.h
+++ b/chrome/browser/autocomplete/search_provider.h
@@ -335,8 +335,12 @@ class SearchProvider : public AutocompleteProvider,
   HistoryResults keyword_history_results_;
   HistoryResults default_history_results_;
 
+  // Indicates that |timer_| is still running.  When it triggers, we'll
+  // send requests to the suggest server.
+  bool in_timer_phase_;

[msw, 2012/08/21 18:30:27]

Remove this.

[Mark P, 2012/08/21 19:13:41]

Done.

+
   // Number of suggest results that haven't yet arrived. If greater than 0 it
-  // indicates either |timer_| or one of the URLFetchers is still running.
+  // indicates one of the URLFetchers is still running.
   int suggest_results_pending_;
 
   // A timer to start a query to the suggest server after the user has stopped