mirrors/signal-desktop
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Move refresh prekeys out of SignalProtocolStore

Browse source 
Use an event/listener instead

// FREEBIE
This commit is contained in:
lilia 2017-05-22 15:24:24 -07:00 committed by Scott Nonnenberg
parent 681418cf6b
commit 16f3717824
3 changed files with 10 additions and 22 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
js/background.js
View file

@ -21,6 +21,9 @@
// start a background worker for ecc
textsecure.startWorker('/js/libsignal-protocol-worker.js');
Whisper.KeyChangeListener.init(textsecure.storage.protocol);
textsecure.storage.protocol.on('removePreKey', function() {
getAccountManager().refreshPreKeys();
});
extension.onLaunched(function() {
console.log('extension launched');

17
js/signal_protocol_store.js
View file

@ -136,16 +136,13 @@
removePreKey: function(keyId) {
var prekey = new PreKey({id: keyId});
return Promise.all([
new Promise(function(resolve) {
getAccountManager().refreshPreKeys().then(resolve);
}),
new Promise(function(resolve) {
prekey.destroy().then(function() {
resolve();
});
})
]);
this.trigger('removePreKey');
return new Promise(function(resolve) {
prekey.destroy().then(function() {
resolve();
});
});
},
/* Returns a signed keypair object or undefined */

12
test/storage_test.js
View file

@ -91,21 +91,9 @@ describe("SignalProtocolStore", function() {
});
});
describe('removePreKey', function() {
var oldGetAccountManager;
before(function(done) {
oldGetAccountManager = window.getAccountManager;
window.getAccountManager = function() {
return {
refreshPreKeys: function() {
return Promise.resolve();
}
};
};
store.storePreKey(2, testKey).then(done);
});
after(function() {
window.getAccountManager = oldGetAccountManager;
});
it('deletes prekeys', function(done) {
store.removePreKey(2, testKey).then(function() {
return store.loadPreKey(2).then(function(key) {