Bug 633274 - Improve nsINavBookmarkObserver (test).

r=sdwilsh
This commit is contained in:
Marco Bonardo 2011-05-13 21:24:29 +02:00
Родитель f4343a9439
Коммит 26ab22f5a4
1 изменённых файлов: 442 добавлений и 0 удалений

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

@ -0,0 +1,442 @@
/* Any copyright is dedicated to the Public Domain.
http://creativecommons.org/publicdomain/zero/1.0/ */
// Tests that each nsINavBookmarksObserver method gets the correct input.
let gBookmarksObserver = {
expected: [],
validate: function (aMethodName, aArguments) {
do_check_eq(this.expected[0].name, aMethodName);
let args = this.expected.shift().args;
do_check_eq(aArguments.length, args.length);
for (let i = 0; i < aArguments.length; i++) {
do_log_info(aMethodName + "(args[" + i + "]: " + args[i].name + ")");
do_check_true(args[i].check(aArguments[i]));
}
if (this.expected.length == 0) {
run_next_test();
}
},
// nsINavBookmarkObserver
onBeginUpdateBatch: function onBeginUpdateBatch()
this.validate(arguments.callee.name, arguments),
onEndUpdateBatch: function onEndUpdateBatch()
this.validate(arguments.callee.name, arguments),
onItemAdded: function onItemAdded()
this.validate(arguments.callee.name, arguments),
onBeforeItemRemoved: function onBeforeItemRemoved()
this.validate(arguments.callee.name, arguments),
onItemRemoved: function onItemRemoved()
this.validate(arguments.callee.name, arguments),
onItemChanged: function onItemChanged()
this.validate(arguments.callee.name, arguments),
onItemVisited: function onItemVisited()
this.validate(arguments.callee.name, arguments),
onItemMoved: function onItemMoved()
this.validate(arguments.callee.name, arguments),
// nsISupports
QueryInterface: XPCOMUtils.generateQI([Ci.nsINavBookmarkObserver]),
}
add_test(function batch() {
gBookmarksObserver.expected = [
{ name: "onBeginUpdateBatch",
args: [] },
{ name: "onEndUpdateBatch",
args: [] },
];
PlacesUtils.bookmarks.runInBatchMode({
runBatched: function () {
// Nothing.
}
}, null);
});
add_test(function onItemAdded_bookmark() {
const TITLE = "Bookmark 1";
let uri = NetUtil.newURI("http://1.mozilla.org/");
gBookmarksObserver.expected = [
{ name: "onItemAdded",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "parentId", check: function (v) v === PlacesUtils.unfiledBookmarksFolderId },
{ name: "index", check: function (v) v === 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "uri", check: function (v) v instanceof Ci.nsIURI && v.equals(uri) },
{ name: "title", check: function (v) v === TITLE },
{ name: "dateAdded", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
];
PlacesUtils.bookmarks.insertBookmark(PlacesUtils.unfiledBookmarksFolderId,
uri, PlacesUtils.bookmarks.DEFAULT_INDEX,
TITLE);
});
add_test(function onItemAdded_separator() {
gBookmarksObserver.expected = [
{ name: "onItemAdded",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "parentId", check: function (v) v === PlacesUtils.unfiledBookmarksFolderId },
{ name: "index", check: function (v) v === 1 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_SEPARATOR },
{ name: "uri", check: function (v) v === null },
{ name: "title", check: function (v) v === null },
{ name: "dateAdded", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
];
PlacesUtils.bookmarks.insertSeparator(PlacesUtils.unfiledBookmarksFolderId,
PlacesUtils.bookmarks.DEFAULT_INDEX);
});
add_test(function onItemAdded_folder() {
const TITLE = "Folder 1";
gBookmarksObserver.expected = [
{ name: "onItemAdded",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "parentId", check: function (v) v === PlacesUtils.unfiledBookmarksFolderId },
{ name: "index", check: function (v) v === 2 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_FOLDER },
{ name: "uri", check: function (v) v === null },
{ name: "title", check: function (v) v === TITLE },
{ name: "dateAdded", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
];
PlacesUtils.bookmarks.createFolder(PlacesUtils.unfiledBookmarksFolderId,
TITLE,
PlacesUtils.bookmarks.DEFAULT_INDEX);
});
add_test(function onItemChanged_title_bookmark() {
let id = PlacesUtils.bookmarks.getIdForItemAt(PlacesUtils.unfiledBookmarksFolderId, 0);
let uri = PlacesUtils.bookmarks.getBookmarkURI(id);
const TITLE = "New title";
gBookmarksObserver.expected = [
{ name: "onItemChanged", // This is an unfortunate effect of bug 653910.
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "property", check: function (v) v === "title" },
{ name: "isAnno", check: function (v) v === false },
{ name: "newValue", check: function (v) v === TITLE },
{ name: "lastModified", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "parentId", check: function (v) v === PlacesUtils.unfiledBookmarksFolderId },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
];
PlacesUtils.bookmarks.setItemTitle(id, TITLE);
});
add_test(function onItemChanged_tags_bookmark() {
let id = PlacesUtils.bookmarks.getIdForItemAt(PlacesUtils.unfiledBookmarksFolderId, 0);
let uri = PlacesUtils.bookmarks.getBookmarkURI(id);
const TITLE = "New title";
const TAG = "tag"
gBookmarksObserver.expected = [
{ name: "onBeginUpdateBatch", // Tag addition uses a batch.
args: [] },
{ name: "onItemAdded", // This is the tag folder.
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "parentId", check: function (v) v === PlacesUtils.tagsFolderId },
{ name: "index", check: function (v) v === 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_FOLDER },
{ name: "uri", check: function (v) v === null },
{ name: "title", check: function (v) v === TAG },
{ name: "dateAdded", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemAdded", // This is the tag.
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "parentId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "index", check: function (v) v === 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "uri", check: function (v) v instanceof Ci.nsIURI && v.equals(uri) },
{ name: "title", check: function (v) v === null },
{ name: "dateAdded", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemChanged",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "property", check: function (v) v === "tags" },
{ name: "isAnno", check: function (v) v === false },
{ name: "newValue", check: function (v) v === "" },
{ name: "lastModified", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "parentId", check: function (v) v === PlacesUtils.unfiledBookmarksFolderId },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onEndUpdateBatch",
args: [] },
{ name: "onBeginUpdateBatch", // Tag removal uses a batch.
args: [] },
{ name: "onBeforeItemRemoved", // This is the tag.
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "parentId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemChanged", // This is an unfortunate effect of bug 653910.
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "property", check: function (v) v === "" },
{ name: "isAnno", check: function (v) v === true },
{ name: "newValue", check: function (v) v === "" },
{ name: "lastModified", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "parentId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemRemoved", // This is the tag.
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "parentId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "index", check: function (v) v === 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "uri", check: function (v) v instanceof Ci.nsIURI && v.equals(uri) },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onBeforeItemRemoved", // This is the tag folder.
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_FOLDER },
{ name: "parentId", check: function (v) v === PlacesUtils.tagsFolderId },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemChanged", // This is an unfortunate effect of bug 653910.
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "property", check: function (v) v === "" },
{ name: "isAnno", check: function (v) v === true },
{ name: "newValue", check: function (v) v === "" },
{ name: "lastModified", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_FOLDER },
{ name: "parentId", check: function (v) v === PlacesUtils.tagsFolderId },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemRemoved", // This is the tag folder.
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "parentId", check: function (v) v === PlacesUtils.tagsFolderId },
{ name: "index", check: function (v) v === 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_FOLDER },
{ name: "uri", check: function (v) v === null },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemChanged",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "property", check: function (v) v === "tags" },
{ name: "isAnno", check: function (v) v === false },
{ name: "newValue", check: function (v) v === "" },
{ name: "lastModified", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "parentId", check: function (v) v === PlacesUtils.unfiledBookmarksFolderId },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onEndUpdateBatch",
args: [] },
];
PlacesUtils.tagging.tagURI(uri, [TAG]);
PlacesUtils.tagging.untagURI(uri, [TAG]);
});
add_test(function onItemMoved_bookmark() {
let id = PlacesUtils.bookmarks.getIdForItemAt(PlacesUtils.unfiledBookmarksFolderId, 0);
let uri = PlacesUtils.bookmarks.getBookmarkURI(id);
gBookmarksObserver.expected = [
{ name: "onItemMoved",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "oldParentId", check: function (v) v === PlacesUtils.unfiledBookmarksFolderId },
{ name: "oldIndex", check: function (v) v === 0 },
{ name: "newParentId", check: function (v) v === PlacesUtils.toolbarFolderId },
{ name: "newIndex", check: function (v) v === 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "oldParentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "newParentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemMoved",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "oldParentId", check: function (v) v === PlacesUtils.toolbarFolderId },
{ name: "oldIndex", check: function (v) v === 0 },
{ name: "newParentId", check: function (v) v === PlacesUtils.unfiledBookmarksFolderId },
{ name: "newIndex", check: function (v) v === 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "oldParentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "newParentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
];
PlacesUtils.bookmarks.moveItem(id, PlacesUtils.toolbarFolderId, 0);
PlacesUtils.bookmarks.moveItem(id, PlacesUtils.unfiledBookmarksFolderId, 0);
});
add_test(function onItemMoved_bookmark() {
let id = PlacesUtils.bookmarks.getIdForItemAt(PlacesUtils.unfiledBookmarksFolderId, 0);
let uri = PlacesUtils.bookmarks.getBookmarkURI(id);
gBookmarksObserver.expected = [
{ name: "onItemVisited",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "visitId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "time", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "transitionType", check: function (v) v === PlacesUtils.history.TRANSITION_TYPED },
{ name: "uri", check: function (v) v instanceof Ci.nsIURI && v.equals(uri) },
{ name: "parentId", check: function (v) v === PlacesUtils.unfiledBookmarksFolderId },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
];
PlacesUtils.history.addVisit(uri, Date.now() * 1000, null,
PlacesUtils.history.TRANSITION_TYPED, false, 0);
});
add_test(function onItemRemoved_bookmark() {
let id = PlacesUtils.bookmarks.getIdForItemAt(PlacesUtils.unfiledBookmarksFolderId, 0);
let uri = PlacesUtils.bookmarks.getBookmarkURI(id);
gBookmarksObserver.expected = [
{ name: "onBeforeItemRemoved",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "parentId", check: function (v) v === PlacesUtils.unfiledBookmarksFolderId },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemChanged", // This is an unfortunate effect of bug 653910.
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "property", check: function (v) v === "" },
{ name: "isAnno", check: function (v) v === true },
{ name: "newValue", check: function (v) v === "" },
{ name: "lastModified", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "parentId", check: function (v) v === PlacesUtils.unfiledBookmarksFolderId },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemRemoved",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "parentId", check: function (v) v === PlacesUtils.unfiledBookmarksFolderId },
{ name: "index", check: function (v) v === 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_BOOKMARK },
{ name: "uri", check: function (v) v instanceof Ci.nsIURI && v.equals(uri) },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
];
PlacesUtils.bookmarks.removeItem(id);
});
add_test(function onItemRemoved_separator() {
let id = PlacesUtils.bookmarks.getIdForItemAt(PlacesUtils.unfiledBookmarksFolderId, 0);
gBookmarksObserver.expected = [
{ name: "onBeforeItemRemoved",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_SEPARATOR },
{ name: "parentId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemChanged", // This is an unfortunate effect of bug 653910.
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "property", check: function (v) v === "" },
{ name: "isAnno", check: function (v) v === true },
{ name: "newValue", check: function (v) v === "" },
{ name: "lastModified", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_SEPARATOR },
{ name: "parentId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemRemoved",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "parentId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "index", check: function (v) v === 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_SEPARATOR },
{ name: "uri", check: function (v) v === null },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
];
PlacesUtils.bookmarks.removeItem(id);
});
add_test(function onItemRemoved_folder() {
let id = PlacesUtils.bookmarks.getIdForItemAt(PlacesUtils.unfiledBookmarksFolderId, 0);
const TITLE = "Folder 2";
gBookmarksObserver.expected = [
{ name: "onBeforeItemRemoved",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_FOLDER },
{ name: "parentId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemChanged", // This is an unfortunate effect of bug 653910.
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "property", check: function (v) v === "" },
{ name: "isAnno", check: function (v) v === true },
{ name: "newValue", check: function (v) v === "" },
{ name: "lastModified", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_FOLDER },
{ name: "parentId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
{ name: "onItemRemoved",
args: [
{ name: "itemId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "parentId", check: function (v) typeof(v) == "number" && v > 0 },
{ name: "index", check: function (v) v === 0 },
{ name: "itemType", check: function (v) v === PlacesUtils.bookmarks.TYPE_FOLDER },
{ name: "uri", check: function (v) v === null },
{ name: "guid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
{ name: "parentGuid", check: function (v) typeof(v) == "string" && /^[a-zA-Z0-9\-_]{12}$/.test(v) },
] },
];
PlacesUtils.bookmarks.removeItem(id);
});
function run_test() {
PlacesUtils.bookmarks.addObserver(gBookmarksObserver, false);
run_next_test();
}
do_register_cleanup(function () {
PlacesUtils.bookmarks.removeObserver(gBookmarksObserver);
});