Permalink
Browse files

Bug 812584: Remove a failed system update from the update queue so it…

… isn't offered again. r=etienne
  • Loading branch information...
1 parent fd97e30 commit 48ecf30bb298c80b825b69575b26db792dcf1527 @marshall committed Dec 5, 2012
Showing with 6 additions and 0 deletions.
  1. +1 −0 apps/system/js/updatable.js
  2. +5 −0 apps/system/test/unit/updatable_test.js
@@ -148,6 +148,7 @@ SystemUpdatable.prototype.handleEvent = function(evt) {
SystemUpdatable.prototype.errorCallBack = function() {
UpdateManager.requestErrorBanner();
UpdateManager.removeFromDownloadsQueue(this);
+ UpdateManager.removeFromUpdatesQueue(this);
};
SystemUpdatable.prototype.showApplyPrompt = function() {
@@ -399,6 +399,11 @@ suite('system/Updatable', function() {
assert.isNotNull(MockUpdateManager.mLastDownloadsRemoval);
assert.equal(subject, MockUpdateManager.mLastDownloadsRemoval);
});
+
+ test('should remove self from update queue', function() {
+ assert.isNotNull(MockUpdateManager.mLastUpdatesRemoval);
+ assert.equal(subject, MockUpdateManager.mLastUpdatesRemoval);
+ });
});
suite('update-progress', function() {

0 comments on commit 48ecf30

Please sign in to comment.