Don't reset the presence timer on every dispatch

Only on user activity (there's now a dispatch every time we set a
member event, ie. A LOT, hence this now being a problem)
This commit is contained in:
David Baker 2018-03-06 17:48:21 +00:00
parent d167dbcf86
commit a97e7375d5

View File

@ -1,5 +1,6 @@
/* /*
Copyright 2015, 2016 OpenMarket Ltd Copyright 2015, 2016 OpenMarket Ltd
Copyright 2018 New Vector Ltd
Licensed under the Apache License, Version 2.0 (the "License"); Licensed under the Apache License, Version 2.0 (the "License");
you may not use this file except in compliance with the License. you may not use this file except in compliance with the License.
@ -31,7 +32,7 @@ class Presence {
this.running = true; this.running = true;
if (undefined === this.state) { if (undefined === this.state) {
this._resetTimer(); this._resetTimer();
this.dispatcherRef = dis.register(this._onUserActivity.bind(this)); this.dispatcherRef = dis.register(this._onAction.bind(this));
} }
} }
@ -125,10 +126,11 @@ class Presence {
this.setState("unavailable"); this.setState("unavailable");
} }
_onUserActivity(payload) { _onAction(payload) {
if (payload.action === "sync_state" || payload.action === "self_presence_updated") return; if (payload.action === "user_activity") {
this._resetTimer(); this._resetTimer();
} }
}
/** /**
* Callback called when the user made an action on the page * Callback called when the user made an action on the page