Open Bug 1755178 Opened 2 years ago Updated 2 years ago

readyState 3 fires many times per chunk

Categories

(Core :: DOM: Networking, defect, P3)

Product:
Core
Component:
DOM: Networking
Version:
Firefox 97
Type:
defect

Tracking

()

Status:
UNCONFIRMED

People

(Reporter: marc, Unassigned, NeedInfo)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-triaged])

User Agent: Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.111 Safari/537.36

Steps to reproduce:

Here is my workaround, this code has worked since IE7:

https://github.com/tinspin/fuse/commit/41118f1cb0f4493e80ae13d8a995f295325312e2

Actual results:

You are calling onreadystatechange with readyState == 3 many times per chunk in a comet-stream.

Expected results:

Like it works in Chrome, ONE onreadystatechange with readyState == 3 per chunk.

marc, thanks for filing this!

Olli, Kershaw, have we changed readystatechange behavior recently? Can it fire more than once per chunk in Fx at the moment?

(This is a follow-up from discussion in bug 447689.)

Component: Untriaged → DOM: Networking
Flags: needinfo?(kershaw)
Flags: needinfo?(bugs)
Product: Firefox → Core

There is an easy way to reproduce this:

Go to http://fuse.rupy.se and login on two browsers.

Click the firefox username and press the other name in the user list.

Then press join to send an invite, if you send from chrome -> firefox you'll se an arbitrary amount of poll|join|<name>||hejsan in the console.

From firefox -> chrome/safari you always get one!

It can be my bug but I'm pretty sure it's not because I have no differentiation in the code between modern browsers, only IE is treated with that.

Ok, you should be able to edit comment here..

*Click the other name in the user list.

I am not aware of any recent change about readystatechange.

I also tried to reproduce this with the steps in comment #2, but I can't reproduce this. I only saw one poll|join|<name>||hejsan in the console.
Could you try to help us by using mozregression to find out which firefox version breaks this?
Or could you provide a simple test sample that shows this issue?

Thanks.

Flags: needinfo?(kershaw) → needinfo?(marc)

I'm using 97.

You mean you never saw this: http://move.rupy.se/file/ff.png <-- this is wrong the message comes twice...

That is surprising.

I'll wait a few upgrades and try again.

Flags: needinfo?(marc)

Have you tested this on Windows 7?

Blocks: fetch
Severity: -- → S4
Priority: -- → P3
Whiteboard: [necko-triaged]
You need to log in before you can comment on or make changes to this bug.