Closed Bug 601973 Opened 12 years ago Closed 12 years ago
Engine ._test Decrypt() yields wrong results
Bug 600995 introduced a small bug in SyncEngine._testDecrypt() -- the wrong 'this' object is used. This causes two crossweage tests to fail: http://brasstacks.mozilla.com/resultserv/crossweave/testrun/31155/
Fix a 'this' scoping error in SyncEngine._testDecrypt(). Rename this method to canDecrypt() since it's clearly public API. Provide tests for SyncEngine.canDecrypt() as well as Service.wipeClient().
Assignee: nobody → philipp
Attachment #480975 - Flags: review?(mconnor)
Attachment #480975 - Flags: review?(mconnor) → review+
Pushed to fx-sync http://hg.mozilla.org/services/fx-sync/rev/57142f068ebd Keeping this open until I land this on m-c.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
(In reply to comment #2) > Keeping this open until I land this on m-c. And of course I did resolve it... force of habit. Keeping it resolved then.
Merged to m-c: http://hg.mozilla.org/mozilla-central/rev/13b0effff79d
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.