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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java

Issue 2604453004: Delete dead code for CurrentlyOpenTabs (Closed)
Patch Set: Created 4 years 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/ntp/RecentTabsManager.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java b/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java
index aec39b8f0126197ad9cd8df2e047acf0191d4779..cc198aa1f1790ae9708524048efefd1553ca4a68 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/ntp/RecentTabsManager.java
@@ -164,9 +164,6 @@ public class RecentTabsManager implements AndroidSyncSettingsObserver, SignInSta
mSignInManager.addSignInStateObserver(this);
}
- protected void updateCurrentlyOpenTabs() {
- }
-
private void updateRecentlyClosedTabs() {
mRecentlyClosedTabs = mRecentlyClosedBridge.getRecentlyClosedTabs(
RECENTLY_CLOSED_MAX_TAB_COUNT);
@@ -180,13 +177,6 @@ public class RecentTabsManager implements AndroidSyncSettingsObserver, SignInSta
}
/**
- * @return Most up-to-date list of currently open tabs.
- */
- public List<CurrentlyOpenTab> getCurrentlyOpenTabs() {
- return null;
- }
-
- /**
* @return Most up-to-date list of foreign sessions.
*/
public List<ForeignSession> getForeignSessions() {
@@ -271,49 +261,6 @@ public class RecentTabsManager implements AndroidSyncSettingsObserver, SignInSta
}
/**
- * Sets the persistent expanded/collapsed state of the currently open tabs list.
- *
- * @param isCollapsed Whether the currently open tabs list is collapsed.
- */
- public void setCurrentlyOpenTabsCollapsed(boolean isCollapsed) {
- if (mIsDestroyed) return;
- mNewTabPagePrefs.setCurrentlyOpenTabsCollapsed(isCollapsed);
- }
-
- /**
- * Determine the expanded/collapsed state of the currently open tabs list.
- *
- * @return Whether the currently open tabs list is collapsed.
- */
- public boolean isCurrentlyOpenTabsCollapsed() {
- return mNewTabPagePrefs.getCurrentlyOpenTabsCollapsed();
- }
-
- /**
- * Sets the state for showing all tabs in the currently open tabs list. This is intended to
- * be overridden in extending classes and set to true when the user clicks the "More" button
- * at the end of the list.
- * @param showingAll Whether the currently open tabs list should start to show all.
- */
- public void setCurrentlyOpenTabsShowAll(boolean showingAll) {
- }
-
- /**
- * @return Whether the currently open tabs group shows all tabs. If it is not, only a limited
- * number of tabs is shown with a "More" button at the end of the list to show all.
- */
- public boolean isCurrentlyOpenTabsShowingAll() {
- return false;
- }
-
- /**
- * Closes the specified currently open tab.
- * @param tab Information about the tab that should be closed.
- */
- public void closeTab(CurrentlyOpenTab tab) {
- }
-
- /**
* Sets the persistent expanded/collapsed state of a foreign session list.
*
* @param session foreign session to collapsed.

Powered by Google App Engine
This is Rietveld 408576698