Omnibox SearchProvider Experiment Client Implementation.

chrome/browser/autocomplete/autocomplete.cc

 // Copyright (c) 2012 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/autocomplete.h"
 
 #include <algorithm>
 #include <iterator>
 #include <set>
 
@@ skipping 701 matching lines @@
                  matches_.end());
 
   // Sort and trim to the most relevant kMaxMatches matches.
   const size_t num_matches = std::min(kMaxMatches, matches_.size());
   std::partial_sort(matches_.begin(), matches_.begin() + num_matches,
                     matches_.end(), &AutocompleteMatch::MoreRelevant);
   matches_.resize(num_matches);
 
   default_match_ = begin();
 
+  // TODO(msw): Somehow need to stop inlining stale data "a[mazon]" -> "ad".

[Peter Kasting, 2012/05/04 00:11:33]

We should never be reaching here with stale data.

[msw, 2012/05/04 09:43:40]

It's no longer 'stale'; but see CopyOldMatches for |!done_| synchronous passes.

+  //if (default_match() != end() &&
+  //    default_match()->inline_autocomplete_offset != string16::npos &&
+  //    !StartsWith(default_match()->contents, input.text(), false)) {
+  //  matches_.begin()->inline_autocomplete_offset = string16::npos;
+  //}
+
   // Set the alternate nav URL.
   alternate_nav_url_ = GURL();
   if (((input.type() == AutocompleteInput::UNKNOWN) ||
        (input.type() == AutocompleteInput::REQUESTED_URL)) &&
       (default_match_ != end()) &&
       (default_match_->transition != content::PAGE_TRANSITION_TYPED) &&
       (default_match_->transition != content::PAGE_TRANSITION_KEYWORD) &&
       (input.canonicalized_url() != default_match_->destination_url))
     alternate_nav_url_ = input.canonicalized_url();
 }
@@ skipping 259 matching lines @@
 
 void AutocompleteController::UpdateResult(bool is_synchronous_pass) {
   AutocompleteResult last_result;
   last_result.Swap(&result_);
 
   for (ACProviders::const_iterator i(providers_.begin()); i != providers_.end();
        ++i)
     result_.AppendMatches((*i)->matches());
 
   // Sort the matches and trim to a small number of "best" matches.
+  // TODO(msw): Remove or sanitize old matches?
+  //            (type == SEARCH_SUGGEST && from_previous == true)?
   result_.SortAndCull(input_);
 
   // Need to validate before invoking CopyOldMatches as the old matches are not
   // valid against the current input.
 #ifndef NDEBUG
   result_.Validate();
 #endif
 
   if (!done_) {
     // This conditional needs to match the conditional in Start that invokes
     // StartExpireTimer.
     result_.CopyOldMatches(input_, last_result);
   }
 
   UpdateKeywordDescriptions(&result_);
   UpdateAssociatedKeywords(&result_);
 
-  bool notify_default_match = is_synchronous_pass;

[Peter Kasting, 2012/05/04 00:11:33]

Why wouldn't we want to always notify on the synchronous pass?

[msw, 2012/05/04 09:43:40]

Reverted.

-  if (!is_synchronous_pass) {
-    const bool last_default_was_valid =
-        last_result.default_match() != last_result.end();
-    const bool default_is_valid = result_.default_match() != result_.end();
-    // We've gotten async results. Send notification that the default match
-    // updated if fill_into_edit differs or associated_keyword differ.  (The
-    // latter can change if we've just started Chrome and the keyword database
-    // finishes loading while processing this request.) We don't check the URL
-    // as that may change for the default match even though the fill into edit
-    // hasn't changed (see SearchProvider for one case of this).
-    notify_default_match =
-        (last_default_was_valid != default_is_valid) ||
-        (default_is_valid &&
-          ((result_.default_match()->fill_into_edit !=
-            last_result.default_match()->fill_into_edit) ||
-            (result_.default_match()->associated_keyword.get() !=
-              last_result.default_match()->associated_keyword.get())));
-  }
+  const bool last_default_was_valid =
+      last_result.default_match() != last_result.end();
+  const bool default_is_valid = result_.default_match() != result_.end();
+  // We've gotten async results. Send notification that the default match
+  // updated if fill_into_edit differs or associated_keyword differ.  (The
+  // latter can change if we've just started Chrome and the keyword database
+  // finishes loading while processing this request.) We don't check the URL
+  // as that may change for the default match even though the fill into edit
+  // hasn't changed (see SearchProvider for one case of this).
+  bool notify_default_match =
+      (last_default_was_valid != default_is_valid) ||
+      (default_is_valid &&
+        ((result_.default_match()->fill_into_edit !=
+          last_result.default_match()->fill_into_edit) ||
+          (result_.default_match()->associated_keyword.get() !=
+            last_result.default_match()->associated_keyword.get())));
 
   NotifyChanged(notify_default_match);
 }
 
 void AutocompleteController::UpdateAssociatedKeywords(
     AutocompleteResult* result) {
   if (!keyword_provider_)
     return;
 
   std::set<string16> keywords;
@@ skipping 91 matching lines @@
     const AutocompleteResult& result)
     : text(text),
       input_type(input_type),
       selected_index(selected_index),
       tab_id(tab_id),
       elapsed_time_since_user_first_modified_omnibox(
           elapsed_time_since_user_first_modified_omnibox),
       inline_autocompleted_length(inline_autocompleted_length),
       result(result) {
 }