Consider to add a separate queue for window.postMessage and process it with lower priority during page load
Categories
(Core :: DOM: Core & HTML, enhancement, P3)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox69 | --- | fixed |
People
(Reporter: smaug, Assigned: sefeng)
References
(Blocks 1 open bug)
Details
Attachments
(3 files)
firefox_vs_chrome_yahoo_news.png
Facebook seems to use postMessage rather early during pageload, and it has some heavy event listener for the message event.
|
Reporter | |
Comment 1•6 years ago
|
||
Perhaps the postMessage queue which bug 1512388 needs can be used here.
|
||
Updated•6 years ago
|
|
||
Updated•6 years ago
|
|
|
Reporter | |
Comment 2•6 years ago
|
||
Just for testing, to see how much this might affect to page loads
Raptor with a test patch: https://treeherder.mozilla.org/#/jobs?repo=try&revision=21d4ec2101750dfca50005a6806ac5a64b60fd03
Base: https://treeherder.mozilla.org/#/jobs?repo=try&revision=e182aae79a79c5cd3c00d7e057e8925839843b8a
|
|
Reporter | |
Comment 3•6 years ago
|
||
Looks like this could affect positively to google-mail and yahoo-news. So worth to look at some more.
|
|
Reporter | |
Comment 4•6 years ago
|
||
Another raptor run to get new numbers with some of the recent patches.
Base: https://treeherder.mozilla.org/#/jobs?repo=try&revision=5768c81a3a4fb01c2c3c69b27fc98c46f59cef8d
patch: https://treeherder.mozilla.org/#/jobs?repo=try&revision=b693e3ba21d4c52d13cb1539b641e00b319b4628
|
Assignee | |
Updated•6 years ago
|
|
|
Assignee | |
Comment 5•6 years ago
|
||
|
|
Assignee | |
Comment 6•6 years ago
|
||
Try push with flag turned on: https://treeherder.mozilla.org/#/jobs?repo=try&revision=d5d9f8beaee7b39a0c18b9e5136cdb2fb3628010
|
|
Assignee | |
Comment 7•6 years ago
|
||
WPT bug that needs to be fixed first: https://github.com/web-platform-tests/wpt/issues/16540
|
||
Comment 9•6 years ago
|
||
Backed out changeset 7fa8e33635fc (Bug 1534012) for wpt failures at promise-rejection-events.html.
Backout: https://hg.mozilla.org/integration/autoland/rev/b35cf8000e7a708377696f3859ae6bfa0fa8f554
Push that started the failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=pending%2Crunning%2Csuccess%2Ctestfailed%2Cbusted%2Cexception&revision=7fa8e33635fcd787ff275d92fd4e9c8df6127d78&selectedJob=245624422
Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=245624422&repo=autoland&lineNumber=17735
|
|
Assignee | |
Comment 10•6 years ago
|
||
PostMessage Event uses "PostMessage task source" which is different
than "DOM manipulation task queue"; Update the code to use detail
object instead.
Depends on D27386
|
|
Assignee | |
Comment 11•5 years ago
|
||
|
||
Comment 12•5 years ago
|
||
|
||
Comment 15•5 years ago
|
||
| bugherder | ||
https://hg.mozilla.org/mozilla-central/rev/ef5bf4372b76
https://hg.mozilla.org/mozilla-central/rev/55c360c47f31
|
||
Comment 17•5 years ago
|
||
== Change summary for alert #21015 (as of Tue, 21 May 2019 07:34:34 GMT) ==
Improvements:
26% raptor-tp6-yahoo-news-firefox loadtime windows10-64-shippable-qr opt 481.21 -> 356.17
26% raptor-tp6-yahoo-news-firefox loadtime windows10-64-shippable opt 481.83 -> 357.71
24% raptor-tp6-yahoo-news-firefox loadtime linux64-shippable-qr opt 510.21 -> 386.79
23% raptor-tp6-yahoo-news-firefox loadtime windows7-32-shippable opt 468.71 -> 362.00
21% raptor-tp6-yahoo-news-firefox loadtime linux64-shippable opt 465.96 -> 365.83
7% raptor-tp6-yahoo-news-firefox windows10-64-shippable opt 290.02 -> 268.48
7% raptor-tp6-yahoo-news-firefox windows10-64-shippable-qr opt 289.48 -> 268.46
5% raptor-tp6-yahoo-news-firefox windows7-32-shippable opt 286.52 -> 270.82
5% raptor-tp6-yahoo-news-firefox linux64-shippable opt 294.51 -> 279.56
5% raptor-tp6-yahoo-news-firefox linux64-shippable-qr opt 314.07 -> 298.39
For up to date results, see: https://treeherder.mozilla.org/perf.html#/alerts?id=21015
|
|
Assignee | |
Updated•5 years ago
|
|
|
Assignee | |
Comment 18•5 years ago
|
||
Updated a screenshot to show the performance changes between Firefox and Chrome for Yahoo News.
Description
•