Index: chrome/browser/autocomplete/autocomplete.h |
diff --git a/chrome/browser/autocomplete/autocomplete.h b/chrome/browser/autocomplete/autocomplete.h |
index 54bd9ff791e8f1c35c95ed0c20033222654531bf..e7766ddf639513b4189df17b84610398d9b71063 100644 |
--- a/chrome/browser/autocomplete/autocomplete.h |
+++ b/chrome/browser/autocomplete/autocomplete.h |
@@ -538,11 +538,11 @@ class AutocompleteProvider |
// NOTE: Remember to call OnProviderUpdate() if matches_ is updated. |
virtual void DeleteMatch(const AutocompleteMatch& match) {} |
- static void set_max_matches(size_t max_matches) { |
- max_matches_ = max_matches; |
- } |
- |
- static size_t max_matches() { return max_matches_; } |
+ // A suggested upper bound for how many matches a provider should return. |
+ // TODO(pkasting): http://b/1111299 , http://b/933133 This should go away once |
+ // we have good relevance heuristics; the controller should handle all |
+ // culling. |
+ static const size_t kMaxMatches; |
protected: |
friend class base::RefCountedThreadSafe<AutocompleteProvider>; |
@@ -576,13 +576,7 @@ class AutocompleteProvider |
const char* name_; |
private: |
- // A suggested upper bound for how many matches a provider should return. |
- // TODO(pkasting): http://b/1111299 , http://b/933133 This should go away once |
- // we have good relevance heuristics; the controller should handle all |
- // culling. |
- static size_t max_matches_; |
- |
- DISALLOW_EVIL_CONSTRUCTORS(AutocompleteProvider); |
+ DISALLOW_COPY_AND_ASSIGN(AutocompleteProvider); |
}; |
typedef AutocompleteProvider::ACProviderListener ACProviderListener; |
@@ -628,11 +622,6 @@ class AutocompleteResult { |
bool is_history_what_you_typed_match; |
}; |
- static void set_max_matches(size_t max_matches) { |
- max_matches_ = max_matches; |
- } |
- static size_t max_matches() { return max_matches_; } |
- |
AutocompleteResult(); |
// operator=() by another name. |
@@ -682,12 +671,12 @@ class AutocompleteResult { |
void Validate() const; |
#endif |
- private: |
// Max number of matches we'll show from the various providers. We may end |
// up showing an additional shortcut for Destinations->History, see |
// AddHistoryContentsShortcut. |
- static size_t max_matches_; |
+ static const size_t kMaxMatches; |
+ private: |
ACMatches matches_; |
const_iterator default_match_; |
@@ -701,7 +690,7 @@ class AutocompleteResult { |
// 'foo'" result, and this will contain "http://foo/". |
GURL alternate_nav_url_; |
- DISALLOW_EVIL_CONSTRUCTORS(AutocompleteResult); |
+ DISALLOW_COPY_AND_ASSIGN(AutocompleteResult); |
}; |
// AutocompleteController ----------------------------------------------------- |
@@ -865,7 +854,7 @@ class AutocompleteController : public ACProviderListener { |
// fast or continuously the user is typing. |
base::RepeatingTimer<AutocompleteController> update_delay_timer_; |
- DISALLOW_EVIL_CONSTRUCTORS(AutocompleteController); |
+ DISALLOW_COPY_AND_ASSIGN(AutocompleteController); |
}; |
// AutocompleteLog ------------------------------------------------------------ |