Every other bookmark isn't deleted (80833)

We don't need to increment count since we splice elements out of the selection array.
r=jag, sr=blake, a=chofmann.
This commit is contained in:
ben%netscape.com 2001-06-06 06:06:11 +00:00
Родитель 235e8852b8
Коммит 92fb92d1f0
1 изменённых файлов: 3 добавлений и 5 удалений

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

@ -590,8 +590,6 @@ BookmarksUIElement.prototype = {
ksRDFC.Init(kBMDS, krParent);
nextElement = this.getNextElement(aSelection[count]);
ksRDFC.RemoveElement(krBookmark, true);
// Manipulate the selection array ourselves.
aSelection.splice(count,1);
try {
// XXX - UGH. Template builder is NOT removing the element from the
@ -602,8 +600,8 @@ BookmarksUIElement.prototype = {
}
catch (e) {
}
++count;
// Manipulate the selection array ourselves.
aSelection.splice(count,1);
}
this.selectElement(nextElement);
},
@ -840,7 +838,7 @@ var BookmarksUtils = {
var fw = document.commandDispatcher.focusedWindow;
aCharset = fw.document.characterSet;
}
if (aShowDialog)
openDialog("chrome://communicator/content/bookmarks/addBookmark.xul", "",
"centerscreen,chrome,dialog=yes,resizable,dependent", aTitle, aURL, null, aCharset);