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

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

Issue 2555513003: [Android] Sort custom search engines based on last visited time and display only top 3 most recentl… (Closed)
Patch Set: Fix the conflicts Created 4 years 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.h
diff --git a/chrome/browser/search_engines/template_url_service_android.h b/chrome/browser/search_engines/template_url_service_android.h
index f140e5930718d98d4862f2e2367e5b29452b6154..48f978bdcc7be591232ddc5f43f2e0dd7fe69334 100644
--- a/chrome/browser/search_engines/template_url_service_android.h
+++ b/chrome/browser/search_engines/template_url_service_android.h
@@ -76,6 +76,15 @@ class TemplateUrlServiceAndroid : public TemplateURLServiceObserver {
JNIEnv* env,
const base::android::JavaParamRef<jobject>& obj,
const base::android::JavaParamRef<jstring>& jkeyword);
+ base::android::ScopedJavaLocalRef<jstring> AddSearchEngineForTesting(
+ JNIEnv* env,
+ const base::android::JavaParamRef<jobject>& obj,
+ const base::android::JavaParamRef<jstring>& jkeyword,
+ jint age_in_days);
+ base::android::ScopedJavaLocalRef<jstring> UpdateLastVisitedForTesting(
+ JNIEnv* env,
+ const base::android::JavaParamRef<jobject>& obj,
+ const base::android::JavaParamRef<jstring>& jkeyword);
static bool Register(JNIEnv* env);
@@ -89,7 +98,8 @@ class TemplateUrlServiceAndroid : public TemplateURLServiceObserver {
// Updates |template_urls_| to contain all TemplateURLs. It sorts this list
// with prepopulated engines first, then any default non-prepopulated engine,
- // then other non-prepopulated engines.
+ // then other non-prepopulated engines based on last_visited in descending
+ // order.
void LoadTemplateURLs();
JavaObjectWeakGlobalRef weak_java_obj_;

Powered by Google App Engine
This is Rietveld 408576698