Closed
Bug 1303751
Opened 9 years ago
Closed 1 year ago
Large etherpad documents become unusable in Firefox
Categories
(Core :: DOM: Core & HTML, defect, P3)
Core
DOM: Core & HTML
Tracking
()
RESOLVED
INCOMPLETE
People
(Reporter: jrmuizel, Unassigned)
References
Details
(Keywords: perf, Whiteboard: domcore-bugbash-triaged)
This document is painful to edit in Firefox. It works fine in Chrome.
Here's a profile: https://cleopatra.io/#report=0208026f364e6b67d1982440f9d8ecb361f615d3
Much of the time is spent re-spellchecking the entire document. (bug 1303749)
| Comment hidden (obsolete) |
Comment 2•9 years ago
|
||
Does the profile point to a Mac-specific issue (I am probably just reading it improperly but: http://searchfox.org/mozilla-central/source/toolkit/xre/MacApplicationDelegate.mm#99)?
FWIW the editing seems pretty painless on Windows in Aurora.
Flags: needinfo?(mstange)
Comment 3•9 years ago
|
||
It's true, the profile shows a lot of Mac event loop overhead in the parent process. It would be nice to find out why for example setWindowsNeedUpdate is called so frequently; it seems to be called for every IPC message that the parent process receives from the content process.
But the real problem in that profile is happening in the content process, and that's almost exclusively bug 1303749.
There's also a bit of slow JS execution in there, for example in this part: https://cleopatra.io/#report=0208026f364e6b67d1982440f9d8ecb361f615d3&javascriptOnly=true&filter=%5B%7B%22type%22%3A%22RangeSampleFilter%22,%22start%22%3A357951,%22end%22%3A359149%7D%5D .
Though 5.5% of JS execution is in CodeGeneratorShared::generateCompactTrackedOptimizationsMap, which is maybe only called when you're profiling.
Flags: needinfo?(mstange)
Updated•9 years ago
|
Priority: -- → P3
| Assignee | ||
Updated•7 years ago
|
Component: DOM → DOM: Core & HTML
Updated•3 years ago
|
Severity: normal → S3
Comment 4•1 year ago
|
||
The test case is gone, not much we can do... Please file a new bug with new profiles.
Whiteboard: domcore-bugbash-triaged
Updated•1 year ago
|
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → INCOMPLETE
You need to log in
before you can comment on or make changes to this bug.
Description
•