Fix for bug 200002. bookmarks.js : missing semicolon in line #98 ?

r=varga, sr=brendan, patch by Andreas Kunz <durbacher@gmx.de>
This commit is contained in:
varga%netscape.com 2007-05-05 04:36:13 +00:00
Родитель 67d8581b54
Коммит 296fedbe6b
1 изменённых файлов: 20 добавлений и 20 удалений

Просмотреть файл

@ -93,7 +93,7 @@ function initServices()
kPREFContractID = "@mozilla.org/preferences-service;1"; kPREFContractID = "@mozilla.org/preferences-service;1";
kPREFIID = Components.interfaces.nsIPrefService; kPREFIID = Components.interfaces.nsIPrefService;
PREF = Components.classes[kPREFContractID].getService(kPREFIID) PREF = Components.classes[kPREFContractID].getService(kPREFIID)
.getBranch(null) .getBranch(null);
kSOUNDContractID = "@mozilla.org/sound;1"; kSOUNDContractID = "@mozilla.org/sound;1";
kSOUNDIID = Components.interfaces.nsISound; kSOUNDIID = Components.interfaces.nsISound;
@ -148,7 +148,7 @@ var BookmarksCommand = {
{ {
var xulElement = document.createElementNS(XUL_NS, "menuitem"); var xulElement = document.createElementNS(XUL_NS, "menuitem");
xulElement.setAttribute("cmd", aCommandName); xulElement.setAttribute("cmd", aCommandName);
var cmd = "cmd_" + aCommandName.substring(NC_NS_CMD.length) var cmd = "cmd_" + aCommandName.substring(NC_NS_CMD.length);
xulElement.setAttribute("command", cmd); xulElement.setAttribute("command", cmd);
switch (aCommandName) { switch (aCommandName) {
@ -430,15 +430,15 @@ var BookmarksCommand = {
xferable.addDataFlavor("moz/bookmarkclipboarditem"); xferable.addDataFlavor("moz/bookmarkclipboarditem");
bmstring.data = sBookmarkItem; bmstring.data = sBookmarkItem;
xferable.setTransferData("moz/bookmarkclipboarditem", bmstring, sBookmarkItem.length*2) xferable.setTransferData("moz/bookmarkclipboarditem", bmstring, sBookmarkItem.length*2);
xferable.addDataFlavor("text/html"); xferable.addDataFlavor("text/html");
htmlstring.data = sTextHTML; htmlstring.data = sTextHTML;
xferable.setTransferData("text/html", htmlstring, sTextHTML.length*2) xferable.setTransferData("text/html", htmlstring, sTextHTML.length*2);
xferable.addDataFlavor("text/unicode"); xferable.addDataFlavor("text/unicode");
unicodestring.data = sTextUnicode; unicodestring.data = sTextUnicode;
xferable.setTransferData("text/unicode", unicodestring, sTextUnicode.length*2) xferable.setTransferData("text/unicode", unicodestring, sTextUnicode.length*2);
const kClipboardContractID = "@mozilla.org/widget/clipboard;1"; const kClipboardContractID = "@mozilla.org/widget/clipboard;1";
const kClipboardIID = Components.interfaces.nsIClipboard; const kClipboardIID = Components.interfaces.nsIClipboard;
@ -491,7 +491,7 @@ var BookmarksCommand = {
return; return;
} }
var selection = {item: items, parent:Array(items.length), length: items.length} var selection = {item: items, parent:Array(items.length), length: items.length};
BookmarksUtils.checkSelection(selection); BookmarksUtils.checkSelection(selection);
BookmarksUtils.insertSelection("paste", selection, aTarget, true); BookmarksUtils.insertSelection("paste", selection, aTarget, true);
}, },
@ -537,7 +537,7 @@ var BookmarksCommand = {
// requires utilityOverlay.js if opening in new window for getTopWin() // requires utilityOverlay.js if opening in new window for getTopWin()
openOneBookmark: function (aURI, aTargetBrowser, aDS) openOneBookmark: function (aURI, aTargetBrowser, aDS)
{ {
var url = BookmarksUtils.getProperty(aURI, NC_NS+"URL", aDS) var url = BookmarksUtils.getProperty(aURI, NC_NS+"URL", aDS);
// Ignore "NC:" and empty urls. // Ignore "NC:" and empty urls.
if (url == "") if (url == "")
return; return;
@ -632,7 +632,7 @@ var BookmarksCommand = {
{ {
var rSeparator = BMSVC.createSeparator(); var rSeparator = BMSVC.createSeparator();
var selection = BookmarksUtils.getSelectionFromResource(rSeparator); var selection = BookmarksUtils.getSelectionFromResource(rSeparator);
BookmarksUtils.insertSelection("newseparator", selection, aTarget) BookmarksUtils.insertSelection("newseparator", selection, aTarget);
}, },
importBookmarks: function () importBookmarks: function ()
@ -672,7 +672,7 @@ var BookmarksCommand = {
transaction.index .push(index); transaction.index .push(index);
transaction.isValid.push(true); transaction.isValid.push(true);
} }
var isCancelled = !BookmarksUtils.any(transaction.isValid) var isCancelled = !BookmarksUtils.any(transaction.isValid);
if (!isCancelled) { if (!isCancelled) {
BMSVC.transactionManager.doTransaction(transaction); BMSVC.transactionManager.doTransaction(transaction);
BookmarksUtils.flushDataSource(); BookmarksUtils.flushDataSource();
@ -823,11 +823,11 @@ var BookmarksController = {
(type0 == "Folder" || type0 == "PersonalToolbarFolder"); (type0 == "Folder" || type0 == "PersonalToolbarFolder");
case "cmd_bm_setpersonaltoolbarfolder": case "cmd_bm_setpersonaltoolbarfolder":
if (length != 1) if (length != 1)
return false return false;
return item0 != "NC:PersonalToolbarFolder" && type0 == "Folder"; return item0 != "NC:PersonalToolbarFolder" && type0 == "Folder";
case "cmd_bm_setnewsearchfolder": case "cmd_bm_setnewsearchfolder":
if (length != 1) if (length != 1)
return false return false;
return item0 != "NC:NewSearchFolder" && return item0 != "NC:NewSearchFolder" &&
(type0 == "Folder" || type0 == "PersonalToolbarFolder"); (type0 == "Folder" || type0 == "PersonalToolbarFolder");
case "cmd_bm_movebookmark": case "cmd_bm_movebookmark":
@ -1004,7 +1004,7 @@ var BookmarksUtils = {
else else
bundle = this._bundle.formatStringFromName(aStringKey, aReplaceString, aReplaceString.length); bundle = this._bundle.formatStringFromName(aStringKey, aReplaceString, aReplaceString.length);
} catch (e) { } catch (e) {
dump("Bookmark bundle "+aStringKey+" not found!\n") dump("Bookmark bundle "+aStringKey+" not found!\n");
bundle = ""; bundle = "";
} }
@ -1036,9 +1036,9 @@ var BookmarksUtils = {
// Determine the rdf:type property for the given resource. // Determine the rdf:type property for the given resource.
resolveType: function (aResource) resolveType: function (aResource)
{ {
var type = this.getProperty(aResource, RDF_NS+"type") var type = this.getProperty(aResource, RDF_NS+"type");
if (type != "") if (type != "")
type = type.split("#")[1] type = type.split("#")[1];
if (type == "Folder") { if (type == "Folder") {
if (this.isPersonalToolbarFolder(aResource)) if (this.isPersonalToolbarFolder(aResource))
type = "PersonalToolbarFolder"; type = "PersonalToolbarFolder";
@ -1051,13 +1051,13 @@ var BookmarksUtils = {
///////////////////////////////////////////////////////////////////////////// /////////////////////////////////////////////////////////////////////////////
// Returns true if aResource is a folder group // Returns true if aResource is a folder group
isFolderGroup: function (aResource) { isFolderGroup: function (aResource) {
return this.getProperty(aResource, NC_NS+"FolderGroup") == "true" return this.getProperty(aResource, NC_NS+"FolderGroup") == "true";
}, },
///////////////////////////////////////////////////////////////////////////// /////////////////////////////////////////////////////////////////////////////
// Returns true if aResource is the Personal Toolbar Folder // Returns true if aResource is the Personal Toolbar Folder
isPersonalToolbarFolder: function (aResource) { isPersonalToolbarFolder: function (aResource) {
return this.getProperty(aResource, NC_NS+"FolderType") == "NC:PersonalToolbarFolder" return this.getProperty(aResource, NC_NS+"FolderType") == "NC:PersonalToolbarFolder";
}, },
///////////////////////////////////////////////////////////////////////////// /////////////////////////////////////////////////////////////////////////////
@ -1112,7 +1112,7 @@ var BookmarksUtils = {
var rName = this.getProperty(aFolder, NC_NS+"Name"); var rName = this.getProperty(aFolder, NC_NS+"Name");
var newFolder; var newFolder;
if (this.isFolderGroup(aFolder)) if (this.isFolderGroup(aFolder))
newFolder = BMSVC.createGroup(rName) newFolder = BMSVC.createGroup(rName);
else else
newFolder = BMSVC.createFolder(rName); newFolder = BMSVC.createFolder(rName);
@ -1266,7 +1266,7 @@ var BookmarksUtils = {
// Returns true is aItem is a child of aContainer // Returns true is aItem is a child of aContainer
isChildOfContainer: function (aItem, aContainer) isChildOfContainer: function (aItem, aContainer)
{ {
RDFC.Init(BMDS, aContainer) RDFC.Init(BMDS, aContainer);
var rChildren = RDFC.GetElements(); var rChildren = RDFC.GetElements();
while (rChildren.hasMoreElements()) { while (rChildren.hasMoreElements()) {
if (aItem == rChildren.getNext()) if (aItem == rChildren.getNext())
@ -1293,7 +1293,7 @@ var BookmarksUtils = {
} }
if (aAction != "move" && !BookmarksUtils.all(transaction.isValid)) if (aAction != "move" && !BookmarksUtils.all(transaction.isValid))
SOUND.beep(); SOUND.beep();
var isCancelled = !BookmarksUtils.any(transaction.isValid) var isCancelled = !BookmarksUtils.any(transaction.isValid);
if (!isCancelled) { if (!isCancelled) {
BMSVC.transactionManager.doTransaction(transaction); BMSVC.transactionManager.doTransaction(transaction);
if (aAction != "move") if (aAction != "move")
@ -1344,7 +1344,7 @@ var BookmarksUtils = {
} }
if (!BookmarksUtils.all(transaction.isValid)) if (!BookmarksUtils.all(transaction.isValid))
SOUND.beep(); SOUND.beep();
var isCancelled = !BookmarksUtils.any(transaction.isValid) var isCancelled = !BookmarksUtils.any(transaction.isValid);
if (!isCancelled) { if (!isCancelled) {
BMSVC.transactionManager.doTransaction(transaction); BMSVC.transactionManager.doTransaction(transaction);
BookmarksUtils.flushDataSource(); BookmarksUtils.flushDataSource();