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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePermissionsFetcher.java

Issue 2953853002: [Android Site Settings] Disable fetching important sites when not needed (Closed)
Patch Set: comments Created 3 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/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePermissionsFetcher.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePermissionsFetcher.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePermissionsFetcher.java
index e1148612d325aaa1bfce2946856754f049dfd0db..5c2c48356996c12c3908db3c573669f31df4e85b 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePermissionsFetcher.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/WebsitePermissionsFetcher.java
@@ -34,11 +34,23 @@ public class WebsitePermissionsFetcher {
// The callback to run when the permissions have been fetched.
private final WebsitePermissionsCallback mCallback;
+ private final boolean mFetchSiteImportantInfo;
+
/**
* @param callback The callback to run when the fetch is complete.
*/
public WebsitePermissionsFetcher(WebsitePermissionsCallback callback) {
+ this(callback, false);
+ }
+
+ /**
+ * @param callback The callback to run when the fetch is complete.
+ * @param fetchSiteImportantInfo if the fetcher should query whether each site is 'important'.
+ */
+ public WebsitePermissionsFetcher(
+ WebsitePermissionsCallback callback, boolean fetchSiteImportantInfo) {
mCallback = callback;
+ mFetchSiteImportantInfo = fetchSiteImportantInfo;
}
/**
@@ -298,7 +310,7 @@ public class WebsitePermissionsFetcher {
}
queue.next();
}
- });
+ }, mFetchSiteImportantInfo);
}
}

Powered by Google App Engine
This is Rietveld 408576698