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

Unified Diff: chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModelSelectorImpl.java

Issue 1408853002: [Tabbed mode] Restore tabs even if TabState is missing (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Added test Created 5 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/tabmodel/TabModelSelectorImpl.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModelSelectorImpl.java b/chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModelSelectorImpl.java
index 2a1a1d330f5352a035159f1058b9b5c39b7031a5..733878998bb1698b1a21efe9cbdd445c5d1d30c9 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModelSelectorImpl.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabModelSelectorImpl.java
@@ -320,24 +320,18 @@ public class TabModelSelectorImpl extends TabModelSelectorBase implements TabMod
* If there is an asynchronous session restore in-progress, try to synchronously restore
* the state of a tab with the given url as a frozen tab. This method has no effect if
* there isn't a tab being restored with this url, or the tab has already been restored.
- *
- * @return true if there exists a tab with the url.
*/
- public boolean tryToRestoreTabStateForUrl(String url) {
- if (!isSessionRestoreInProgress()) return false;
- return mTabSaver.restoreTabStateForUrl(url);
+ public void tryToRestoreTabStateForUrl(String url) {
+ if (isSessionRestoreInProgress()) mTabSaver.restoreTabStateForUrl(url);
}
/**
* If there is an asynchronous session restore in-progress, try to synchronously restore
* the state of a tab with the given id as a frozen tab. This method has no effect if
* there isn't a tab being restored with this id, or the tab has already been restored.
- *
- * @return true if there exists a tab with the id.
*/
- public boolean tryToRestoreTabStateForId(int id) {
- if (!isSessionRestoreInProgress()) return false;
- return mTabSaver.restoreTabStateForId(id);
+ public void tryToRestoreTabStateForId(int id) {
+ if (isSessionRestoreInProgress()) mTabSaver.restoreTabStateForId(id);
}
public void clearState() {

Powered by Google App Engine
This is Rietveld 408576698