From 536938dc74edca3e4c3f400d6da28316d9e5324a Mon Sep 17 00:00:00 2001 From: "jonathandicarlo@jonathan-dicarlos-macbook-pro.local" Date: Mon, 16 Mar 2009 16:49:56 -0700 Subject: [PATCH] If you click the button to show synced tabs before Weave has been configured, it will now take you straight to the sign-in screen. Fixes bug 482818. --- services/sync/modules/engines/tabs.js | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/services/sync/modules/engines/tabs.js b/services/sync/modules/engines/tabs.js index 540581d8628..b65efbace7f 100644 --- a/services/sync/modules/engines/tabs.js +++ b/services/sync/modules/engines/tabs.js @@ -241,6 +241,7 @@ TabStore.prototype = { if (urlHistory.length >= 10) break; } + // TODO add last-visited date for this tab... but how? record.addTab(currentPage.title, urlHistory); } } @@ -256,7 +257,9 @@ TabStore.prototype = { let urlHistory = [url]; this._log.debug("Wrapping a tab with title " + title); // TODO how to get older entries in urlHistory? + // can we use BrowserUI._getHistory somehow? record.addTab(title, urlHistory); + // TODO add last-visited date for this tab... but how? } }, @@ -336,6 +339,10 @@ TabStore.prototype = { }; +/* TODO let's have TabTracker keep track of open/close switch events + * and maintain most-recently used date of each tab... + */ + function TabTracker() { this._TabTracker_init(); }