Don't autocomplete searches of >1 word if they've only been visited once and the user has not yet...

chrome/browser/autocomplete/shortcuts_provider.cc

 // Copyright (c) 2011 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "chrome/browser/autocomplete/shortcuts_provider.h"
 
 #include <algorithm>
 #include <cmath>
 #include <map>
 #include <vector>
@@ skipping 54 matching lines @@
     return urls_.find(match.destination_url) != urls_.end();
   }
  private:
   // Lifetime of the object is less than the lifetime of passed |urls|, so
   // it is safe to store reference.
   const std::set<GURL>& urls_;
 };
 
 }  // namespace
 
-// A match needs to score at least 1200 to be default, so set the max below
-// this. For ease of unit testing, make it divisible by 4 (since some tests
-// check for half or quarter of the max score).
-const int ShortcutsProvider::kMaxScore = 1196;
+// For ease of unit testing, make the clamp value divisible by 4 (since some
+// tests check for half or quarter of the max score).
+// static
+const int ShortcutsProvider::kMaxScore =
+    (AutocompleteResult::kLowestDefaultScore - 1) & ~3;

[sky, 2011/07/15 23:01:34]

Wow, that's obscure (the & ~3)

 
 ShortcutsProvider::ShortcutsProvider(ACProviderListener* listener,
                                      Profile* profile)
     : AutocompleteProvider(listener, profile, "ShortcutsProvider"),
       languages_(profile_->GetPrefs()->GetString(prefs::kAcceptLanguages)) {
   notification_registrar_.Add(this, chrome::NOTIFICATION_OMNIBOX_OPENED_URL,
                               Source<Profile>(profile));
   notification_registrar_.Add(this, chrome::NOTIFICATION_HISTORY_URLS_DELETED,
                               Source<Profile>(profile));
 }
@@ skipping 304 matching lines @@
   StripMatchMarkersFromClassifications(&description_class);
 }
 
 ShortcutsProvider::Shortcut::Shortcut()
     : last_access_time(base::Time::Now()),
       number_of_hits(0) {
 }
 
 ShortcutsProvider::Shortcut::~Shortcut() {
 }