don't let postMessage() flood the main thread

NEW
Unassigned

Status

()

defect
P3
normal
3 years ago
3 days ago

People

(Reporter: bkelly, Unassigned)

Tracking

(Blocks 2 bugs)

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Posted patch wipSplinter Review
This builds on the timer ThrottledEventQueue work in bug 1300659.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=c36464ef409498f90ad4ee2b4c232137086b561e

The back pressure part of this patch is not quite right.  I may not implement back pressure immediately since we never stop message events today.
(Reporter)

Updated

3 years ago
Assignee: nobody → bkelly
Status: NEW → ASSIGNED
One thing people have asked is as fast postMessage (between windows) as possible, not being blocked but other stuff.
Just something to keep in mind.
Priority: -- → P3
(Reporter)

Updated

10 months ago
Assignee: ben → nobody
Status: ASSIGNED → NEW
Component: DOM → DOM: Core & HTML
Product: Core → Core

Updated

20 days ago
Blocks: eviltraps

Comment 2

20 days ago

It's very similar to bug 1514413, this one is postMessage, that bug is Http request. If the parent gets flooded by those events, the browser will completely hang, some extreme case may even force user to reboot the machine.

Chrome doesn't have this problem.

You need to log in before you can comment on or make changes to this bug.