Omnibox metrics logging patch splitout, part 3: Pass match.type to the edit and let it make the determination about showing the search hint, instead of making the determination in the popup.

In the ultimate patch, the types on which we'll show the hint will expand, so this saved duplicating some logic at points in the popup; in retrospect it makes more sense anyway, since there's no reason for the popup to know anything about the search hint.  That's more the edit's job anyway.
Review URL: http://codereview.chromium.org/10839

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@5276 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 4 insertions, 8 deletions

diff --git a/chrome/browser/autocomplete/autocomplete_edit.h b/chrome/browser/autocomplete/autocomplete_edit.h
index 15e3688..99309ef 100644
--- a/chrome/browser/autocomplete/autocomplete_edit.h
+++ b/chrome/browser/autocomplete/autocomplete_edit.h
@@ -260,22 +260,18 @@ class AutocompleteEditModel {
   //   |text| is either the new temporary text (if |is_temporary_text| is true)
   //     from the user manually selecting a different match, or the inline
   //     autocomplete text (if |is_temporary_text| is false).
-  //   |previous_selected_match| is only used when changing the temporary text;
-  //     it is the match that was (manually or automatically) selected before
-  //     the current manual selection, and is saved to be restored later if the
-  //     user hits <esc>.
-  //   |can_show_search_hint| is true if the current choice is nonexistent or a
-  //     search result; in these cases it may be OK to show the "Type to search"
-  //     hint (see comments on show_search_hint_).
   //   |keyword| is the keyword to show a hint for if |is_keyword_hint| is true,
   //     or the currently selected keyword if |is_keyword_hint| is false (see
   //     comments on keyword_ and is_keyword_hint_).
+  //   |type| is the type of match selected; this is used to determine whether
+  //     we can show the "Type to search" hint (see comments on
+  //     show_search_hint_).
   void OnPopupDataChanged(
       const std::wstring& text,
       bool is_temporary_text,
       const std::wstring& keyword,
       bool is_keyword_hint,
-      bool can_show_search_hint);
+      AutocompleteMatch::Type type);
 
   // Called by the AutocompleteEditView after something changes, with details
   // about what state changes occured.  Updates internal state, updates the