Skip to content

Commit

Permalink
Hotfix for Edge 16
Browse files Browse the repository at this point in the history
  • Loading branch information
adolfintel committed Oct 12, 2017
1 parent f63afea commit b1149b3
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 5 deletions.
4 changes: 2 additions & 2 deletions doc.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# HTML5 Speedtest

> by Federico Dossena
> Version 4.3.2, September 5 2017
> Version 4.3.2 Hotfix 1, October 12 2017
> [https://github.com/adolfintel/speedtest/](https://github.com/adolfintel/speedtest/)

Expand Down Expand Up @@ -291,7 +291,7 @@ At the moment there is no front-end to see the telemetry data; you can connect t

## Known bugs and limitations
* The ping/jitter test is measured by seeing how long it takes for an empty XHR to complete. It is not an acutal ICMP ping
* __IE11, Edge 15 (15 only):__ the upload test is not precise, especially on very fast connections
* __IE11, Edge 15 and 16 (only):__ the upload test is not precise, especially on very fast connections
* __IE11:__ the upload test may not work over HTTPS
* __Firefox:__ on some Linux systems with hardware acceleration turned off, the page rendering makes the browser lag, reducing the accuracy of the ping/jitter test

Expand Down
4 changes: 2 additions & 2 deletions speedtest_worker.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
HTML5 Speedtest v4.3.2
HTML5 Speedtest v4.3.2 Hotfix 1
by Federico Dossena
https://github.com/adolfintel/speedtest/
GNU LGPLv3 License
Expand Down Expand Up @@ -86,7 +86,7 @@ this.addEventListener('message', function (e) {
if (/Edge.(\d+\.\d+)/i.test(ua)) {
// edge more precise with 3 download streams
settings.xhr_dlMultistream = 3
if (/Edge\/15.(\d+)/i.test(ua)) {
if (/Edge\/15.(\d+)/i.test(ua) || /Edge\/16.(\d+)/i.test(ua)) {
//Edge 15 introduced a bug that causes onprogress events to not get fired, so for Edge 15, we have to use the "small chunks" workaround that reduces accuracy
settings.forceIE11Workaround = true
}
Expand Down
Loading

0 comments on commit b1149b3

Please sign in to comment.