Closed
Bug 1873967
Opened 9 months ago
Closed 9 months ago
Simplify postMessage throttling code a bit.
Categories
(Core :: DOM: postMessage, task)
Core
DOM: postMessage
Tracking
()
RESOLVED
FIXED
123 Branch
People
(Reporter: emilio, Assigned: emilio)
References
Details
Attachments
(2 files)
48 bytes,
text/x-phabricator-request
|
dmeehan
:
approval-mozilla-release+
|
Details | Review |
48 bytes,
text/x-phabricator-request
|
dmeehan
:
approval-mozilla-release+
|
Details | Review |
No description provided.
Assignee | ||
Comment 1•9 months ago
|
||
Updated•9 months ago
|
Assignee: nobody → emilio
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•9 months ago
|
||
This is somewhat complex, untested, never shipped, and predates
fission.
Remove it to simplify the code.
Depends on D198183
Pushed by ealvarez@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/22fa0caa658b
Simplify dom_separate_event_queue_for_post_message_enabled checks. r=sefeng
https://hg.mozilla.org/integration/autoland/rev/0c7e3ebf56ed
Remove DocGroup::TryToLoadIframesInBackground. r=sefeng
Comment 4•9 months ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/22fa0caa658b
https://hg.mozilla.org/mozilla-central/rev/0c7e3ebf56ed
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
status-firefox123:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 123 Branch
Assignee | ||
Comment 5•9 months ago
|
||
Comment on attachment 9372049 [details]
Bug 1873967 - Simplify dom_separate_event_queue_for_post_message_enabled checks. r=smaug,sefeng
Beta/Release Uplift Approval Request
- User impact if declined: Necessary for bug 1840762
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce: N/A
- List of other uplifts needed: none
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Trivial dead code removal + clean-up.
- String changes made/needed: none
- Is Android affected?: Yes
Attachment #9372049 -
Flags: approval-mozilla-release?
Assignee | ||
Updated•9 months ago
|
Attachment #9372050 -
Flags: approval-mozilla-release?
Comment 6•9 months ago
|
||
Comment on attachment 9372049 [details]
Bug 1873967 - Simplify dom_separate_event_queue_for_post_message_enabled checks. r=smaug,sefeng
Approved for 122.0 RC2
Attachment #9372049 -
Flags: approval-mozilla-release? → approval-mozilla-release+
Comment 7•9 months ago
|
||
Comment on attachment 9372050 [details]
Bug 1873967 - Remove DocGroup::TryToLoadIframesInBackground. r=smaug,sefeng
Approved for 122.0 RC2
Attachment #9372050 -
Flags: approval-mozilla-release? → approval-mozilla-release+
Updated•9 months ago
|
status-firefox122:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•