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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/device/DeviceClassManager.java

Issue 1460573002: Remove Undo Options from DeviceClassManager (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Remove phantom imports Created 5 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/device/DeviceClassManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/device/DeviceClassManager.java b/chrome/android/java/src/org/chromium/chrome/browser/device/DeviceClassManager.java
index d1f37b61c7798ee9558f6e050fda73093c4da904..6ad59ae392d4e12504e48957eb9e5cc00e79489c 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/device/DeviceClassManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/device/DeviceClassManager.java
@@ -37,7 +37,6 @@ public class DeviceClassManager {
private boolean mEnablePrerendering;
private boolean mEnableToolbarSwipe;
private boolean mEnableToolbarSwipeInDocumentMode;
- private boolean mEnableUndo;
private boolean mDisableDomainReliability;
private final boolean mEnableFullscreen;
@@ -84,7 +83,6 @@ public class DeviceClassManager {
.hasSwitch(ChromeSwitches.ENABLE_ACCESSIBILITY_TAB_SWITCHER);
mEnableFullscreen =
!commandLine.hasSwitch(ChromeSwitches.DISABLE_FULLSCREEN);
- mEnableUndo = commandLine.hasSwitch(ChromeSwitches.ENABLE_HIGH_END_UI_UNDO);
mEnableToolbarSwipeInDocumentMode =
commandLine.hasSwitch(ChromeSwitches.ENABLE_TOOLBAR_SWIPE_IN_DOCUMENT_MODE);
@@ -92,10 +90,6 @@ public class DeviceClassManager {
if (mEnableAccessibilityLayout) {
mEnableAnimations = false;
}
-
- if (SysUtils.isLowEndDevice() || mEnableAccessibilityLayout) {
- mEnableUndo = true;
- }
}
/**
@@ -151,13 +145,6 @@ public class DeviceClassManager {
}
/**
- * @return Whether or not undo is enabled.
- */
- public static boolean enableUndo(Context context) {
- return getInstance().mEnableUndo || isAccessibilityModeEnabled(context);
- }
-
- /**
* @return Whether or not to disable domain reliability.
*/
public static boolean disableDomainReliability() {

Powered by Google App Engine
This is Rietveld 408576698