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

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

Issue 2470043002: Android: Clean up dead code involving now-removed Fullscreen permission. (Closed)
Patch Set: 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/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 b1d5a6ccc757588685f371c7a458fe02716d8d5d..6cd00bb5d679c59148a9c294825fa814e863d2b7 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
@@ -55,8 +55,6 @@ public class WebsitePermissionsFetcher {
queue.add(new MidiInfoFetcher());
// Cookies are stored per-host.
queue.add(new CookieExceptionInfoFetcher());
- // Fullscreen are stored per-origin.
- queue.add(new FullscreenInfoFetcher());
// Keygen permissions are per-origin.
queue.add(new KeygenInfoFetcher());
// Local storage info is per-origin.
@@ -112,9 +110,6 @@ public class WebsitePermissionsFetcher {
queue.add(new LocalStorageInfoFetcher());
// Website storage is per-host.
queue.add(new WebStorageInfoFetcher());
- } else if (category.showFullscreenSites()) {
- // Full screen is per-origin.
- queue.add(new FullscreenInfoFetcher());
} else if (category.showCameraSites()) {
// Camera capture permission is per-origin and per-embedder.
queue.add(new CameraCaptureInfoFetcher());
@@ -286,21 +281,6 @@ public class WebsitePermissionsFetcher {
}
}
- /**
- * Class for fetching the fullscreen information.
- */
- private class FullscreenInfoFetcher extends Task {
- @Override
- public void run() {
- for (FullscreenInfo info : WebsitePreferenceBridge.getFullscreenInfo()) {
- WebsiteAddress origin = WebsiteAddress.create(info.getOrigin());
- if (origin == null) continue;
- WebsiteAddress embedder = WebsiteAddress.create(info.getEmbedder());
- findOrCreateSite(origin, embedder).setFullscreenInfo(info);
- }
- }
- }
-
private class LocalStorageInfoFetcher extends Task {
@Override
public void runAsync(final TaskQueue queue) {

Powered by Google App Engine
This is Rietveld 408576698