summaryrefslogtreecommitdiffstats
path: root/chrome/browser/android/omnibox/autocomplete_controller_android.cc
diff options
context:
space:
mode:
authormukai@chromium.org <mukai@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-06-20 16:56:03 +0000
committermukai@chromium.org <mukai@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2014-06-20 16:56:03 +0000
commit332d17d2d2889750fd87da90c2d9a796163e981c (patch)
tree584667b51d7aa63b058606adc8027946ead2046f /chrome/browser/android/omnibox/autocomplete_controller_android.cc
parentac8a16bc1ec072211e6e24a781d0bfd19c6ad9b8 (diff)
downloadchromium_src-332d17d2d2889750fd87da90c2d9a796163e981c.zip
chromium_src-332d17d2d2889750fd87da90c2d9a796163e981c.tar.gz
chromium_src-332d17d2d2889750fd87da90c2d9a796163e981c.tar.bz2
Remove AutocompleteInput Type and PageClassification.
They are just as same as the enums defined in .proto files. Removing the declaration of these types reduces unnecessary code dependency to autocomplete. BUG=384232 R=blundell@chromium.org, mpearson@chromium.org, pkasting@chromium.org TBR=brettw@chromium.org, cpu@chromium.org, miguelg@chromium.org, mpcomplete@chromium.org TEST=compile succeeds Review URL: https://codereview.chromium.org/343523003 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@278722 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/android/omnibox/autocomplete_controller_android.cc')
-rw-r--r--chrome/browser/android/omnibox/autocomplete_controller_android.cc32
1 files changed, 17 insertions, 15 deletions
diff --git a/chrome/browser/android/omnibox/autocomplete_controller_android.cc b/chrome/browser/android/omnibox/autocomplete_controller_android.cc
index ffbb3eb..27ee25a 100644
--- a/chrome/browser/android/omnibox/autocomplete_controller_android.cc
+++ b/chrome/browser/android/omnibox/autocomplete_controller_android.cc
@@ -36,6 +36,7 @@
#include "chrome/common/pref_names.h"
#include "chrome/common/url_constants.h"
#include "components/keyed_service/content/browser_context_dependency_manager.h"
+#include "components/metrics/proto/omnibox_event.pb.h"
#include "content/public/browser/notification_details.h"
#include "content/public/browser/notification_service.h"
#include "content/public/browser/notification_source.h"
@@ -51,6 +52,7 @@ using base::android::AttachCurrentThread;
using base::android::ConvertJavaStringToUTF16;
using base::android::ConvertUTF8ToJavaString;
using base::android::ConvertUTF16ToJavaString;
+using metrics::OmniboxEventProto;
namespace {
@@ -89,7 +91,7 @@ ZeroSuggestPrefetcher::ZeroSuggestPrefetcher(Profile* profile) : controller_(
base::string16::npos,
base::string16(),
GURL(fake_request_source),
- AutocompleteInput::INVALID_SPEC,
+ OmniboxEventProto::INVALID_SPEC,
false,
false,
true,
@@ -142,8 +144,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,
@@ -212,7 +214,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 =
@@ -397,41 +399,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>
@@ -530,7 +532,7 @@ static jstring QualifyPartialURLQuery(
query_string,
false,
false,
- AutocompleteInput::INVALID_SPEC,
+ OmniboxEventProto::INVALID_SPEC,
&match,
NULL);
if (!match.destination_url.is_valid())