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

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

Issue 2425763002: [NTP] Add functionality to ViewUtils to setEnabled recursively. (Closed)
Patch Set: Created 4 years, 2 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/ManagedPreferenceDelegate.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/preferences/ManagedPreferenceDelegate.java b/chrome/android/java/src/org/chromium/chrome/browser/preferences/ManagedPreferenceDelegate.java
index b195030154c28c17063d3cb5d67ccdfda4e577b9..df0edf65ca51792a2a31f76ac346d6c0021b3a32 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/preferences/ManagedPreferenceDelegate.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/preferences/ManagedPreferenceDelegate.java
@@ -6,7 +6,8 @@ package org.chromium.chrome.browser.preferences;
import android.preference.Preference;
import android.view.View;
-import android.view.ViewGroup;
+
+import org.chromium.chrome.browser.util.ViewUtils;
/**
* A delegate that determines whether a Preference is managed by enterprise policy. This is used
@@ -80,7 +81,7 @@ public abstract class ManagedPreferenceDelegate {
*/
public void onBindViewToPreference(Preference preference, View view) {
if (isPreferenceClickDisabledByPolicy(preference)) {
- disableView(view);
+ ViewUtils.setEnabledRecursive(view, false);
}
}
@@ -101,17 +102,4 @@ public abstract class ManagedPreferenceDelegate {
return false;
}
-
- /**
- * Disables the given View and any subviews, recursively.
- */
- private static void disableView(View view) {
- view.setEnabled(false);
- if (view instanceof ViewGroup) {
- ViewGroup group = (ViewGroup) view;
- for (int i = 0; i < group.getChildCount(); i++) {
- disableView(group.getChildAt(i));
- }
- }
- }
}

Powered by Google App Engine
This is Rietveld 408576698