Bug 1003839 - Live bookmark is still created despite cancelling the s…

…ubscription. r=mak
Brli · May 21, 2014 · 76cc54b · 76cc54b
1 parent 099f7e1
commit 76cc54b
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/toolkit/components/places/PlacesUtils.jsm b/toolkit/components/places/PlacesUtils.jsm
@@ -2205,10 +2205,10 @@ PlacesCreateLivemarkTransaction.prototype = {
 
   undoTransaction: function CLTXN_undoTransaction()
   {
-    // The getLivemark callback is expected to receive a failure status but it
-    // is used just to serialize, so doesn't matter.
+    // The getLivemark callback may fail, but it is used just to serialize,
+    // so it doesn't matter.
     PlacesUtils.livemarks.getLivemark({ id: this.item.id })
-      .then(null, () => {
+      .then(null, null).then( () => {
         PlacesUtils.bookmarks.removeItem(this.item.id);
       });
   }