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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleCategoryPreferences.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/SingleCategoryPreferences.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleCategoryPreferences.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleCategoryPreferences.java
index 91ab09f4a5994bdc01da7a7b673519af039a4c64..df2aa1f54732db67562ad3a37d3b263ac8ce05e7 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleCategoryPreferences.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/website/SingleCategoryPreferences.java
@@ -215,8 +215,6 @@ public class SingleCategoryPreferences extends PreferenceFragment
return website.site().getCameraPermission() == ContentSetting.BLOCK;
} else if (mCategory.showCookiesSites()) {
return website.site().getCookiePermission() == ContentSetting.BLOCK;
- } else if (mCategory.showFullscreenSites()) {
- return website.site().getFullscreenPermission() == ContentSetting.ASK;
} else if (mCategory.showGeolocationSites()) {
return website.site().getGeolocationPermission() == ContentSetting.BLOCK;
} else if (mCategory.showJavaScriptSites()) {
@@ -477,8 +475,6 @@ public class SingleCategoryPreferences extends PreferenceFragment
} else if (mCategory.showCookiesSites()) {
PrefServiceBridge.getInstance().setAllowCookiesEnabled((boolean) newValue);
updateThirdPartyCookiesCheckBox();
- } else if (mCategory.showFullscreenSites()) {
- PrefServiceBridge.getInstance().setFullscreenAllowed((boolean) newValue);
} else if (mCategory.showGeolocationSites()) {
PrefServiceBridge.getInstance().setAllowLocationEnabled((boolean) newValue);
} else if (mCategory.showJavaScriptSites()) {

Powered by Google App Engine
This is Rietveld 408576698