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

Unified Diff: chrome/browser/android/omnibox/autocomplete_controller_android.cc

Issue 343523003: Remove AutocompleteInput Type and PageClassification. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix Created 6 years, 6 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/android/omnibox/autocomplete_controller_android.cc
diff --git a/chrome/browser/android/omnibox/autocomplete_controller_android.cc b/chrome/browser/android/omnibox/autocomplete_controller_android.cc
index 1bd65ad7c062339fb85008aba5783597de706cf5..c45df3f1a86e5bca682d3f95a982e6a961ae01b9 100644
--- a/chrome/browser/android/omnibox/autocomplete_controller_android.cc
+++ b/chrome/browser/android/omnibox/autocomplete_controller_android.cc
@@ -47,6 +47,7 @@ using base::android::AttachCurrentThread;
using base::android::ConvertJavaStringToUTF16;
using base::android::ConvertUTF8ToJavaString;
using base::android::ConvertUTF16ToJavaString;
+using metrics::OmniboxEventProto;
namespace {
@@ -85,7 +86,7 @@ ZeroSuggestPrefetcher::ZeroSuggestPrefetcher(Profile* profile) : controller_(
base::string16::npos,
base::string16(),
GURL(fake_request_source),
- AutocompleteInput::INVALID_SPEC,
+ OmniboxEventProto::INVALID_SPEC,
false,
false,
true,
@@ -138,8 +139,8 @@ void AutocompleteControllerAndroid::Start(JNIEnv* env,
if (j_desired_tld != NULL)
desired_tld = ConvertJavaStringToUTF16(env, j_desired_tld);
base::string16 text = ConvertJavaStringToUTF16(env, j_text);
- AutocompleteInput::PageClassification page_classification =
- AutocompleteInput::OTHER;
+ OmniboxEventProto::PageClassification page_classification =
+ OmniboxEventProto::OTHER;
input_ = AutocompleteInput(text,
base::string16::npos,
desired_tld,
@@ -208,7 +209,7 @@ void AutocompleteControllerAndroid::OnSuggestionSelected(
jobject j_web_contents) {
base::string16 url = ConvertJavaStringToUTF16(env, j_current_url);
const GURL current_url = GURL(url);
- AutocompleteInput::PageClassification current_page_classification =
+ OmniboxEventProto::PageClassification current_page_classification =
ClassifyPage(current_url, is_query_in_omnibox, focused_from_fakebox);
const base::TimeTicks& now(base::TimeTicks::Now());
content::WebContents* web_contents =
@@ -369,41 +370,41 @@ void AutocompleteControllerAndroid::NotifySuggestionsReceived(
j_autocomplete_result);
}
-AutocompleteInput::PageClassification
+OmniboxEventProto::PageClassification
AutocompleteControllerAndroid::ClassifyPage(const GURL& gurl,
bool is_query_in_omnibox,
bool focused_from_fakebox) const {
if (!gurl.is_valid())
- return AutocompleteInput::INVALID_SPEC;
+ return OmniboxEventProto::INVALID_SPEC;
const std::string& url = gurl.spec();
if (gurl.SchemeIs(content::kChromeUIScheme) &&
gurl.host() == chrome::kChromeUINewTabHost) {
- return AutocompleteInput::NTP;
+ return OmniboxEventProto::NTP;
}
if (url == chrome::kChromeUINativeNewTabURL) {
return focused_from_fakebox ?
- AutocompleteInput::INSTANT_NTP_WITH_FAKEBOX_AS_STARTING_FOCUS :
- AutocompleteInput::INSTANT_NTP_WITH_OMNIBOX_AS_STARTING_FOCUS;
+ OmniboxEventProto::INSTANT_NTP_WITH_FAKEBOX_AS_STARTING_FOCUS :
+ OmniboxEventProto::INSTANT_NTP_WITH_OMNIBOX_AS_STARTING_FOCUS;
}
if (url == url::kAboutBlankURL)
- return AutocompleteInput::BLANK;
+ return OmniboxEventProto::BLANK;
if (url == profile_->GetPrefs()->GetString(prefs::kHomePage))
- return AutocompleteInput::HOME_PAGE;
+ return OmniboxEventProto::HOME_PAGE;
if (is_query_in_omnibox)
- return AutocompleteInput::SEARCH_RESULT_PAGE_DOING_SEARCH_TERM_REPLACEMENT;
+ return OmniboxEventProto::SEARCH_RESULT_PAGE_DOING_SEARCH_TERM_REPLACEMENT;
bool is_search_url = TemplateURLServiceFactory::GetForProfile(profile_)->
IsSearchResultsPageFromDefaultSearchProvider(gurl);
if (is_search_url)
- return AutocompleteInput::SEARCH_RESULT_PAGE_NO_SEARCH_TERM_REPLACEMENT;
+ return OmniboxEventProto::SEARCH_RESULT_PAGE_NO_SEARCH_TERM_REPLACEMENT;
- return AutocompleteInput::OTHER;
+ return OmniboxEventProto::OTHER;
}
ScopedJavaLocalRef<jobject>
@@ -502,7 +503,7 @@ static jstring QualifyPartialURLQuery(
query_string,
false,
false,
- AutocompleteInput::INVALID_SPEC,
+ OmniboxEventProto::INVALID_SPEC,
&match,
NULL);
if (!match.destination_url.is_valid())

Powered by Google App Engine
This is Rietveld 408576698