Omnibox: Speed Up SearchProviderTest

chrome/browser/autocomplete/search_provider.cc

Index: chrome/browser/autocomplete/search_provider.cc
diff --git a/chrome/browser/autocomplete/search_provider.cc b/chrome/browser/autocomplete/search_provider.cc
index 1e3739453299eb34ea9e1c8e374380aa7dfc65b8..683b7c40603240ad318f1111ec0e1b53d6817188 100644
--- a/chrome/browser/autocomplete/search_provider.cc
+++ b/chrome/browser/autocomplete/search_provider.cc
@@ -121,6 +121,8 @@ const TemplateURL* SearchProvider::Providers::GetKeywordProviderURL() const {
 const int SearchProvider::kDefaultProviderURLFetcherID = 1;
 // static
 const int SearchProvider::kKeywordProviderURLFetcherID = 2;
+// static
+int SearchProvider::kMinimumTimeBetweenSuggestQueriesMs = 100;

[msw, 2013/04/03 23:28:48]

optional nit: technically definition order should match declaration order, so I
guess this would belong at line 500, just before
SearchProvider::~SearchProvider()... but this does seem like a better spot.

[Mark P, 2013/04/03 23:58:39]

Acknowledged.  Left as is.

 
 SearchProvider::SearchProvider(AutocompleteProviderListener* listener,
                                Profile* profile)
@@ -562,9 +564,8 @@ void SearchProvider::StartOrStopSuggestQuery(bool minimal_changes) {
   if (input_.matches_requested() != AutocompleteInput::ALL_MATCHES)
     return;
 
-  // To avoid flooding the suggest server, don't send a query until at least 100
-  // ms since the last query.
-  const int kMinimumTimeBetweenSuggestQueriesMs = 100;
+  // To avoid flooding the suggest server, don't send a query until at
+  // least 100 ms since the last query.
   base::TimeTicks next_suggest_time(time_suggest_request_sent_ +
       TimeDelta::FromMilliseconds(kMinimumTimeBetweenSuggestQueriesMs));
   base::TimeTicks now(base::TimeTicks::Now());