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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/search_engines/TemplateUrlService.java

Issue 2349473002: Implement native functions to query custom search engines for Android (Closed)
Patch Set: run "git cl format" to format the commit Created 4 years, 3 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
« no previous file with comments | « no previous file | chrome/browser/search_engines/template_url_service_android.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/android/java/src/org/chromium/chrome/browser/search_engines/TemplateUrlService.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/search_engines/TemplateUrlService.java b/chrome/android/java/src/org/chromium/chrome/browser/search_engines/TemplateUrlService.java
index 4f348d2b91ad1f423d164c412c0e90fd05be0343..d467dba6f03c1fddf21d0fa6d7936f59be6475dd 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/search_engines/TemplateUrlService.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/search_engines/TemplateUrlService.java
@@ -47,16 +47,17 @@ public class TemplateUrlService {
public static class TemplateUrl {
private final int mIndex;
private final String mShortName;
+ private boolean mIsPrepopulated;
@CalledByNative("TemplateUrl")
- public static TemplateUrl create(
- int id, String shortName) {
- return new TemplateUrl(id, shortName);
+ public static TemplateUrl create(int id, String shortName, boolean isPrepopulated) {
+ return new TemplateUrl(id, shortName, isPrepopulated);
}
- public TemplateUrl(int index, String shortName) {
+ public TemplateUrl(int index, String shortName, boolean isPrepopulated) {
mIndex = index;
mShortName = shortName;
+ mIsPrepopulated = isPrepopulated;
}
public int getIndex() {
@@ -128,9 +129,8 @@ public class TemplateUrlService {
int templateUrlCount = nativeGetTemplateUrlCount(mNativeTemplateUrlServiceAndroid);
List<TemplateUrl> templateUrls = new ArrayList<TemplateUrl>(templateUrlCount);
for (int i = 0; i < templateUrlCount; i++) {
- TemplateUrl templateUrl = nativeGetPrepopulatedTemplateUrlAt(
- mNativeTemplateUrlServiceAndroid, i);
- if (templateUrl != null) {
+ TemplateUrl templateUrl = nativeGetTemplateUrlAt(mNativeTemplateUrlServiceAndroid, i);
+ if (templateUrl != null && templateUrl.mIsPrepopulated) {
templateUrls.add(templateUrl);
}
}
@@ -176,8 +176,7 @@ public class TemplateUrlService {
assert defaultSearchEngineIndex < nativeGetTemplateUrlCount(
mNativeTemplateUrlServiceAndroid);
- return nativeGetPrepopulatedTemplateUrlAt(
- mNativeTemplateUrlServiceAndroid, defaultSearchEngineIndex);
+ return nativeGetTemplateUrlAt(mNativeTemplateUrlServiceAndroid, defaultSearchEngineIndex);
}
public void setSearchEngine(int selectedIndex) {
@@ -306,8 +305,7 @@ public class TemplateUrlService {
private native void nativeLoad(long nativeTemplateUrlServiceAndroid);
private native boolean nativeIsLoaded(long nativeTemplateUrlServiceAndroid);
private native int nativeGetTemplateUrlCount(long nativeTemplateUrlServiceAndroid);
- private native TemplateUrl nativeGetPrepopulatedTemplateUrlAt(
- long nativeTemplateUrlServiceAndroid, int i);
+ private native TemplateUrl nativeGetTemplateUrlAt(long nativeTemplateUrlServiceAndroid, int i);
private native void nativeSetUserSelectedDefaultSearchProvider(
long nativeTemplateUrlServiceAndroid, int selectedIndex);
private native int nativeGetDefaultSearchProvider(long nativeTemplateUrlServiceAndroid);
« no previous file with comments | « no previous file | chrome/browser/search_engines/template_url_service_android.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698