summaryrefslogtreecommitdiffstats
path: root/services/sync/modules/engines/bookmarks.js
diff options
context:
space:
mode:
Diffstat (limited to 'services/sync/modules/engines/bookmarks.js')
-rw-r--r--services/sync/modules/engines/bookmarks.js13
1 files changed, 5 insertions, 8 deletions
diff --git a/services/sync/modules/engines/bookmarks.js b/services/sync/modules/engines/bookmarks.js
index 42d91f57e..41283c06d 100644
--- a/services/sync/modules/engines/bookmarks.js
+++ b/services/sync/modules/engines/bookmarks.js
@@ -373,8 +373,7 @@ BookmarksEngine.prototype = {
// Failure to create a backup is somewhat bad, but probably not bad
// enough to prevent syncing of bookmarks - so just log the error and
// continue.
- this._log.warn("Got exception \"" + Utils.exceptionStr(ex) +
- "\" backing up bookmarks, but continuing with sync.");
+ this._log.warn("Got exception backing up bookmarks, but continuing with sync.", ex);
cb();
}
);
@@ -389,9 +388,8 @@ BookmarksEngine.prototype = {
try {
guidMap = this._buildGUIDMap();
} catch (ex) {
- this._log.warn("Got exception \"" + Utils.exceptionStr(ex) +
- "\" building GUID map." +
- " Skipping all other incoming items.");
+ this._log.warn("Got exception building GUID map." +
+ " Skipping all other incoming items.", ex);
throw {code: Engine.prototype.eEngineAbortApplyIncoming,
cause: ex};
}
@@ -637,7 +635,7 @@ BookmarksStore.prototype = {
return true;
}
} catch(ex) {
- this._log.debug("Failed to reparent item. " + Utils.exceptionStr(ex));
+ this._log.debug("Failed to reparent item. ", ex);
}
return false;
},
@@ -1265,8 +1263,7 @@ BookmarksStore.prototype = {
let u = PlacesUtils.bookmarks.getBookmarkURI(itemID);
this._tagURI(u, tags);
} catch (e) {
- this._log.warn("Got exception fetching URI for " + itemID + ": not tagging. " +
- Utils.exceptionStr(e));
+ this._log.warn("Got exception fetching URI for " + itemID + ": not tagging. ", e);
// I guess it doesn't have a URI. Don't try to tag it.
return;