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

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

Issue 2006023005: Disable fullscreen global permission setting in Android. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@2743
Patch Set: Created 4 years, 7 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/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 d5dc057e206e8403d1a310d0219d1149348f5b98..b12bb5c92b73e05fc001f2623ceed95da392fbd2 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
@@ -21,6 +21,7 @@ import android.widget.ListView;
import android.widget.TextView;
import org.chromium.chrome.R;
+import org.chromium.chrome.browser.ChromeFeatureList;
import org.chromium.chrome.browser.help.HelpAndFeedback;
import org.chromium.chrome.browser.media.cdm.MediaDrmCredentialManager;
import org.chromium.chrome.browser.media.cdm.MediaDrmCredentialManager.MediaDrmCredentialManagerCallback;
@@ -615,6 +616,10 @@ public class SingleCategoryPreferences extends PreferenceFragment
} else if (mCategory.showFullscreenSites()) {
globalToggle.setChecked(
PrefServiceBridge.getInstance().isFullscreenAllowed());
+ // With the simplified flag enabled, the fullscreen global toggle cannot be
+ // disabled.
+ globalToggle.setEnabled(
+ !ChromeFeatureList.isEnabled("ViewsSimplifiedFullscreenUI"));
} else if (mCategory.showGeolocationSites()) {
globalToggle.setChecked(
LocationSettings.getInstance().isChromeLocationSettingEnabled());

Powered by Google App Engine
This is Rietveld 408576698