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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/history/HistoryManagerUtils.java

Issue 2830983005: Remove old webui History page on desktop and mobile (Closed)
Patch Set: merge Created 3 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/history/HistoryManagerUtils.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/history/HistoryManagerUtils.java b/chrome/android/java/src/org/chromium/chrome/browser/history/HistoryManagerUtils.java
index 3ce75297f827d5150b88350b0261472815b99db7..057023a5549455fe5fac8fa849fafc9b17c4898f 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/history/HistoryManagerUtils.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/history/HistoryManagerUtils.java
@@ -10,34 +10,16 @@ import android.content.Intent;
import org.chromium.base.ContextUtils;
import org.chromium.chrome.R;
import org.chromium.chrome.browser.ChromeActivity;
-import org.chromium.chrome.browser.ChromeFeatureList;
import org.chromium.chrome.browser.IntentHandler;
import org.chromium.chrome.browser.UrlConstants;
import org.chromium.chrome.browser.tab.Tab;
import org.chromium.content_public.browser.LoadUrlParams;
import org.chromium.ui.base.DeviceFormFactor;
-import org.chromium.ui.base.PageTransition;
/**
* Utility methods for the browsing history manager.
*/
public class HistoryManagerUtils {
- private static final Object NATIVE_HISTORY_ENABLED_LOCK = new Object();
- private static Boolean sNativeHistoryEnabled;
-
- /**
- * @return Whether the Android-specific browsing history manager is enabled.
- */
- public static boolean isAndroidHistoryManagerEnabled() {
- synchronized (NATIVE_HISTORY_ENABLED_LOCK) {
- if (sNativeHistoryEnabled == null) {
- sNativeHistoryEnabled = ChromeFeatureList.isEnabled("AndroidHistoryManager");
- }
- }
-
- return sNativeHistoryEnabled;
- }
-
/**
* Opens the browsing history manager.
*
@@ -46,11 +28,6 @@ public class HistoryManagerUtils {
* {@link HistoryManager}.
*/
public static void showHistoryManager(ChromeActivity activity, Tab tab) {
- if (!isAndroidHistoryManagerEnabled()) {
- tab.loadUrl(new LoadUrlParams(UrlConstants.HISTORY_URL, PageTransition.AUTO_TOPLEVEL));
- return;
- }
-
Context appContext = ContextUtils.getApplicationContext();
if (activity.getBottomSheet() != null) {
activity.getBottomSheetContentController().showContentAndOpenSheet(R.id.action_history);

Powered by Google App Engine
This is Rietveld 408576698