Bommels05/Mozilla
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Implements "Bookmark All Tabs..." for Places. (Also fixes a small error

Browse Source 
when undoing aggregate transactions.)

bug=330929
r=annie.sullivan@gmail.com
sr=ben@mozilla.org


git-svn-id: svn://10.0.0.236/trunk@193476 18797224-902f-48f8-a5cc-f745e15eee43
This commit is contained in:
joe%retrovirus.com
2006-04-04 01:26:58 +00:00
parent d7ec54429b
commit abc6a9eccd
6 changed files with 203 additions and 53 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
mozilla/browser/base/content/browser-sets.inc
View File

@@ -148,6 +148,7 @@
#ifdef MOZ_PLACES
<command id="Browser:AddBookmarkAs"
oncommand="PlacesCommandHook.bookmarkCurrentPage();"/>
<command id="Browser:BookmarkAllTabs" oncommand="BrowserController.doCommand('Browser:BookmarkAllTabs');"/>
#else
<command id="Browser:AddBookmarkAs" oncommand="addBookmarkAs(document.getElementById('content'), false);"/>
<command id="Browser:BookmarkAllTabs" oncommand="addBookmarkAs(document.getElementById('content'), true);"/>

38
mozilla/browser/base/content/browser.js
View File

@@ -6294,12 +6294,44 @@ function BookmarkAllTabsCommand() {
}
BookmarkAllTabsCommand.prototype = {
get enabled() {
//LOG("BookmarkAllTabs.enabled: " + getBrowser().tabContainer.childNodes.length > 1);
return getBrowser().tabContainer.childNodes.length > 1;
},
execute: function BATC_execute() {
LOG("BookmarkAllTabs.execute: IMPLEMENT ME");
var tabURIs = this._getUniqueTabInfo(getBrowser());
PlacesController.showAddMultiBookmarkUI(tabURIs);
},
/**
* This function returns a list of nsIURI objects characterizing the
* tabs currently open in the given browser. The URIs will appear in the
* list in the order in which their corresponding tabs appeared. However,
* only the first instance of each URI will be returned.
*
* @param aTabBrowser the tabBrowser to get the contents of
*
* @returns a list of nsIURI objects representing unique locations open
*/
_getUniqueTabInfo: function BATC__getUniqueTabInfo(aTabBrowser) {
var tabList = [];
var seenURIs = [];
const activeBrowser = aTabBrowser.selectedBrowser;
const browsers = aTabBrowser.browsers;
for (var i = 0; i < browsers.length; ++i) {
var webNav = browsers[i].webNavigation;
var uri = webNav.currentURI;
// skip redundant entries
if (uri.spec in seenURIs)
continue;
// add to the set of seen URIs
seenURIs[uri.spec] = true;
tabList.push(uri);
}
return tabList;
}
};
BookmarkAllTabsCommand.NAME = "Browser:BookmarkAllTabs";