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

Unified Diff: chrome/browser/search_engines/template_url_service_android.cc

Issue 2367373003: [Android] Allow setting recently visited search engines as default search engine (Closed)
Patch Set: Update based on Dan and Peter's comments. Created 4 years, 1 month 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/search_engines/template_url_service_android.cc
diff --git a/chrome/browser/search_engines/template_url_service_android.cc b/chrome/browser/search_engines/template_url_service_android.cc
index 2694d44065ce7e070b5ba73ed2c84c0354f8eda7..28135fc8a483b13f5c9ad4db969e6dd96e0afa42 100644
--- a/chrome/browser/search_engines/template_url_service_android.cc
+++ b/chrome/browser/search_engines/template_url_service_android.cc
@@ -42,6 +42,9 @@ TemplateUrlServiceAndroid::TemplateUrlServiceAndroid(JNIEnv* env,
base::Bind(&TemplateUrlServiceAndroid::OnTemplateURLServiceLoaded,
base::Unretained(this)));
template_url_service_->AddObserver(this);
+ if (template_url_service_->loaded() && template_urls_.size() == 0) {
Peter Kasting 2016/11/29 03:32:18 Nit: Prefer .empty() to .size() == 0. {} not nece
ltian 2016/11/30 06:36:17 Done.
+ LoadTemplateURLs();
+ }
}
TemplateUrlServiceAndroid::~TemplateUrlServiceAndroid() {
@@ -56,40 +59,37 @@ void TemplateUrlServiceAndroid::Load(JNIEnv* env,
void TemplateUrlServiceAndroid::SetUserSelectedDefaultSearchProvider(
JNIEnv* env,
const JavaParamRef<jobject>& obj,
- jint selected_index) {
- std::vector<TemplateURL*> template_urls =
- template_url_service_->GetTemplateURLs();
- size_t selected_index_size_t = static_cast<size_t>(selected_index);
- DCHECK_LT(selected_index_size_t, template_urls.size()) <<
- "Wrong index for search engine";
-
- TemplateURL* template_url = template_urls[selected_index_size_t];
+ const JavaParamRef<jstring>& jkeyword) {
+ base::string16 keyword(
+ base::android::ConvertJavaStringToUTF16(env, jkeyword));
+ TemplateURL* template_url =
+ template_url_service_->GetTemplateURLForKeyword(keyword);
template_url_service_->SetUserSelectedDefaultSearchProvider(template_url);
}
-jint TemplateUrlServiceAndroid::GetDefaultSearchProvider(
+jint TemplateUrlServiceAndroid::GetDefaultSearchProviderIndex(
JNIEnv* env,
- const JavaParamRef<jobject>& obj) {
- std::vector<TemplateURL*> template_urls =
- template_url_service_->GetTemplateURLs();
+ const base::android::JavaParamRef<jobject>& obj) const {
TemplateURL* default_search_provider =
- template_url_service_->GetDefaultSearchProvider();
- for (size_t i = 0; i < template_urls.size(); ++i) {
- if (default_search_provider == template_urls[i])
- return i;
- }
- return -1;
+ template_url_service_->GetDefaultSearchProvider();
+ auto it = std::find(template_urls_.begin(), template_urls_.end(),
+ default_search_provider);
+ size_t default_search_provider_index_ = (it == template_urls_.end())
+ ? -1
+ : (it - template_urls_.begin());
Peter Kasting 2016/11/29 03:32:18 Nit: Indenting looks a bit odd. Is this what git
ltian 2016/11/30 06:36:17 Sorry the Eclipse setting and git cl format do in
+ return default_search_provider_index_;
}
-jboolean TemplateUrlServiceAndroid::IsLoaded(JNIEnv* env,
- const JavaParamRef<jobject>& obj) {
+jboolean TemplateUrlServiceAndroid::IsLoaded(
+ JNIEnv* env,
+ const JavaParamRef<jobject>& obj) const {
return template_url_service_->loaded();
}
jint TemplateUrlServiceAndroid::GetTemplateUrlCount(
JNIEnv* env,
- const JavaParamRef<jobject>& obj) {
- return template_url_service_->GetTemplateURLs().size();
+ const JavaParamRef<jobject>& obj) const {
+ return template_urls_.size();
}
jboolean TemplateUrlServiceAndroid::IsSearchProviderManaged(
@@ -122,17 +122,14 @@ jboolean TemplateUrlServiceAndroid::IsDefaultSearchEngineGoogle(
base::android::ScopedJavaLocalRef<jobject>
TemplateUrlServiceAndroid::GetTemplateUrlAt(JNIEnv* env,
const JavaParamRef<jobject>& obj,
- jint index) {
- TemplateURL* template_url = template_url_service_->GetTemplateURLs()[index];
+ jint index) const {
+ TemplateURL* template_url = template_urls_[index];
return Java_TemplateUrl_create(
env, index,
base::android::ConvertUTF16ToJavaString(env, template_url->short_name()),
- IsPrepopulatedTemplate(template_url) ||
- template_url->created_by_policy());
-}
-
-bool TemplateUrlServiceAndroid::IsPrepopulatedTemplate(TemplateURL* url) {
- return url->prepopulate_id() > 0;
+ base::android::ConvertUTF8ToJavaString(env, template_url->url()),
+ template_url_service_->ShowInPrepopulatedList(template_url),
+ base::android::ConvertUTF16ToJavaString(env, template_url->keyword()));
}
void TemplateUrlServiceAndroid::OnTemplateURLServiceLoaded() {
@@ -140,15 +137,37 @@ void TemplateUrlServiceAndroid::OnTemplateURLServiceLoaded() {
JNIEnv* env = base::android::AttachCurrentThread();
if (weak_java_obj_.get(env).is_null())
return;
+ LoadTemplateURLs();
Java_TemplateUrlService_templateUrlServiceLoaded(env,
weak_java_obj_.get(env));
}
+void TemplateUrlServiceAndroid::LoadTemplateURLs() {
+ template_urls_.clear();
Peter Kasting 2016/11/29 03:32:18 This call is unnecessary, since you reset template
ltian 2016/11/30 06:36:17 Done.
+
+ template_urls_ = template_url_service_->GetTemplateURLs();
+ TemplateURL* default_search_provider =
+ template_url_service_->GetDefaultSearchProvider();
+
+ auto typeCompare = [&] (const TemplateURL* lhs, const TemplateURL* rhs)
Peter Kasting 2016/11/29 03:32:18 Nit: Use normal variable capitalization scheme. I
ltian 2016/11/30 06:36:17 Done.
+ -> bool {
Peter Kasting 2016/11/29 03:32:18 Nit: Is the return type necessary for compilation?
ltian 2016/11/30 06:36:17 Done.
+ if (template_url_service_->ShowInPrepopulatedList(lhs)) {
+ return !template_url_service_->ShowInPrepopulatedList(rhs) ||
Peter Kasting 2016/11/29 03:32:18 Nit: This call is verbose and you make it regardle
ltian 2016/11/30 06:36:17 Done.
+ (lhs->prepopulate_id() < rhs->prepopulate_id());
+ }
+ return (lhs == default_search_provider &&
+ !template_url_service_->ShowInPrepopulatedList(rhs));
+ };
+
Peter Kasting 2016/11/29 03:32:18 Nit: I'd avoid this blank line, since the lambda j
ltian 2016/11/30 06:36:17 Done.
+ std::sort(template_urls_.begin(), template_urls_.end(), typeCompare);
+}
+
void TemplateUrlServiceAndroid::OnTemplateURLServiceChanged() {
JNIEnv* env = base::android::AttachCurrentThread();
if (weak_java_obj_.get(env).is_null())
return;
+ LoadTemplateURLs();
Java_TemplateUrlService_onTemplateURLServiceChanged(env,
weak_java_obj_.get(env));
@@ -259,8 +278,11 @@ base::android::ScopedJavaLocalRef<jstring>
TemplateUrlServiceAndroid::GetSearchEngineUrlFromTemplateUrl(
JNIEnv* env,
const JavaParamRef<jobject>& obj,
- jint index) {
- TemplateURL* template_url = template_url_service_->GetTemplateURLs()[index];
+ const JavaParamRef<jstring>& jkeyword) {
+ base::string16 keyword =
+ base::android::ConvertJavaStringToUTF16(env, jkeyword);
+ TemplateURL* template_url =
+ template_url_service_->GetTemplateURLForKeyword(keyword);
std::string url(template_url->url_ref().ReplaceSearchTerms(
TemplateURLRef::SearchTermsArgs(base::ASCIIToUTF16("query")),
template_url_service_->search_terms_data()));

Powered by Google App Engine
This is Rietveld 408576698