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

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

Issue 2296833002: Add tab persistence support for CustomTabs. (Closed)
Patch Set: Address yusufo@s comments Created 4 years, 4 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/TabPersistencePolicy.java
diff --git a/chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabPersistencePolicy.java b/chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabPersistencePolicy.java
index aedf958d3533b0eb9b0578be9f15c14d45436813..a91bcde20efd6597ce49dda22c36413267e619c7 100644
--- a/chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabPersistencePolicy.java
+++ b/chrome/android/java/src/org/chromium/chrome/browser/tabmodel/TabPersistencePolicy.java
@@ -43,18 +43,21 @@ public interface TabPersistencePolicy {
String getStateToBeMergedFileName();
/**
- * Performs any necessary migration required before accessing the tab information.
+ * Performs any necessary initialization required before accessing the tab information. This
+ * can include cleanups or migrations that must occur before the tab state information can be
+ * read reliably.
*
* @param executor The executor that any asynchronous tasks should be run on.
- * @return Whether any migration is necessary.
+ * @return Whether any blocking initialization is necessary.
*/
- boolean performMigration(Executor executor);
+ boolean performInitialization(Executor executor);
/**
- * Waits for the task that migrates all state files to their new location to
- * finish.
+ * Waits for the any pending initialization to finish.
+ *
+ * @see #performInitialization(Executor)
*/
- void waitForMigrationToFinish();
+ void waitForInitializationToFinish();
/**
* @return Whether a merge is currently in progress.

Powered by Google App Engine
This is Rietveld 408576698