1404093 - speedtest cannot running if tab opened from link

Status: RESOLVED WORKSFORME

(Core :: DOM: Core & HTML, defect, P2)

Description

[Kuchenmann]

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64; rv:56.0) Gecko/20100101 Firefox/56.0
Build ID: 20170926190823

Steps to reproduce:

https://www.kabelplus.at/SpecialPages/SpeedTest/Speed-Test.aspx
and then starting the speedtest on clicking link "Zum Starten klicken Sie hier".
After clicking on the link, the URL http://speed.kabelplus.at/test/index.html is opened in a new tab. Here you can start this speedtest by clicking on "LOS" which means start.


Actual results:

After trying for some time, speedtest is stopped with error "could not connect to server".


Expected results:

Speedtest should do a ping, then download and upload. 
But it works, when the URL http://speed.kabelplus.at/test/index.html is directly copied in the FIRST TAB of the firefox browser.

Comment 1

[YF (Yang)]

SecurityError: The operation is insecure.
testBundle.js:1
_createSocket
http://c.speedtestcustom.com/static/js/testBundle.js:1:1226593
connect
http://c.speedtestcustom.com/static/js/testBundle.js:1:1224776
_doCreateConnection
http://c.speedtestcustom.com/static/js/testBundle.js:1:1192974
initialize/this._createConnection/<
http://c.speedtestcustom.com/static/js/testBundle.js:1:1191290
n
http://c.speedtestcustom.com/static/js/testBundle.js:1:984228
Tt/<
http://c.speedtestcustom.com/static/js/testBundle.js:1:977381

Comment 2

[Valentin Gosu [:valentin]]

"SecurityError: The operation is insecure" is shown when WebSocket.cpp returns NS_ERROR_DOM_SECURITY_ERR.

I tried to do a mozregression, and it seems in today's build this issue doesn't reproduce.

Likely one of the last few commits fixed it
https://hg.mozilla.org/mozilla-central/log/tip/dom/base/WebSocket.cpp

[Tracking Requested - why for this release]:
This bug reproduces in Firefox 57.
We probably need to uplift the bug that fixes it.

Comment 3

[Andrea Marchesini [:baku]]

This is fix by bug 1404910. Ritu, can you reconsider bug 1404910 uplift? Thank you.

Comment 4

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Tracked for 57 and I can take the fix for bug 1404910. Just curious, is this a new regression in 57?

Comment 5

[Andrea Marchesini [:baku]]

It's a bug landed in 55.

Comment 6

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Given that this is not a new regression and that we are a week away from entering a hard code freeze, this is now a fix-optional for 57.

Comment 7

[Hsin-Yi Tsai (she/her) | away [:hsinyi]]

The UI of the site changed since the bug was reported. Now if I click "Go" to start testing, everything works as expected to me.
Please file a new bug, if you see a similar issue on a recent version of Firefox.