erabo/spiegel-keyman
1
0
Fork 0
mirror of https://github.com/keymanapp/keyman.git synced 2026-10-12 17:07:31 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 12

change(common/core/web): thresholding Date.now() -> performance.now()

Browse source
This commit is contained in:
jahorton 2021-06-23 08:28:37 +07:00
parent 6257f5884b
commit e57ba467fe
1 changed files with 19 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
common/core/web/input-processor/src/text/inputProcessor.ts
View file

@ -134,7 +134,24 @@ namespace com.keyman.text {
//
// Consider use of https://developer.mozilla.org/en-US/docs/Web/API/Performance/now instead?
// Would allow finer-tuned control.
let TIMEOUT_THRESHOLD = Date.now() + 16; // + 16ms.
let TIMEOUT_THRESHOLD: number = Number.MAX_VALUE;
let _globalThis = com.keyman.utils.getGlobalObject();
let timer: () => number;
// Available by default on `window` in browsers, but _not_ on `global` in Node,
// surprisingly. Since we can't use code dependent on `require` statements
// at present, we have to condition upon it actually existing.
if(_globalThis['performance'] && _globalThis['performance']['now']) {
timer = function() {
return _globalThis['performance']['now']();
};
TIMEOUT_THRESHOLD = timer() + 16; // + 16ms.
} // else {
// We _could_ just use Date.now() as a backup... but that (probably) only matters
// when unit testing. So... we actually don't _need_ time thresholding when in
// a Node environment.
// }
// Tracks a minimum probability for keystroke probability. Anything less will not be
// included in alternate calculations.
@ -156,7 +173,7 @@ namespace com.keyman.text {
for(let pair of keyDistribution) {
if(pair.p < KEYSTROKE_EPSILON) {
break;
} else if(Date.now() >= TIMEOUT_THRESHOLD) {
} else if(timer && timer() >= TIMEOUT_THRESHOLD) {
break;
}