Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3903)

Unified Diff: chrome/browser/autocomplete/search_provider.cc

Issue 13461012: Omnibox: Speed Up SearchProviderTest (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: . Created 7 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
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..3bbf082ad3c00c26008b9a46c50167511727bb75 100644
--- a/chrome/browser/autocomplete/search_provider.cc
+++ b/chrome/browser/autocomplete/search_provider.cc
@@ -121,6 +121,10 @@ const TemplateURL* SearchProvider::Providers::GetKeywordProviderURL() const {
const int SearchProvider::kDefaultProviderURLFetcherID = 1;
// static
const int SearchProvider::kKeywordProviderURLFetcherID = 2;
+// static
+// To avoid flooding the suggest server, don't send a query until at least 100
msw 2013/04/03 21:46:08 nit: keep this comment with its use in StartOrStop
Mark P 2013/04/03 22:51:40 Done. Moved back to old location.
+// ms since the last query.
+int SearchProvider::kMinimumTimeBetweenSuggestQueriesMs = 100;
SearchProvider::SearchProvider(AutocompleteProviderListener* listener,
Profile* profile)
@@ -562,9 +566,6 @@ 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;
base::TimeTicks next_suggest_time(time_suggest_request_sent_ +
TimeDelta::FromMilliseconds(kMinimumTimeBetweenSuggestQueriesMs));
base::TimeTicks now(base::TimeTicks::Now());

Powered by Google App Engine
This is Rietveld 408576698