Open Bug 620650 Opened 10 years ago Updated 10 years ago

If there is big HTML with nested tables and many rows writing in contenteditable freezes browser.

Categories

(Core :: DOM: Editor, defect)

defect
Not set
normal

Tracking

()

UNCONFIRMED

People

(Reporter: akakus, Unassigned)

Details

Attachments

(3 files)

User-Agent:       Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b7) Gecko/20100101 Firefox/4.0b7
Build Identifier: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:2.0b7) Gecko/20100101 Firefox/4.0b7

On pages with a lot of rows, nested tables (or tables wrapped in div elements) and especially combination of them, contenteditable paragraphs at the end of document are unusable - typing is really slow and freezes browser. 

Reproducible: Always

Steps to Reproduce:
1. Open attached testbad.html
2. Go to end of page. Above rows with "Comment" should be cell contaning "Editable segment" 
3. Focus it and start typing. 

Additional way to reproduce it:
1. Open attached testbaddiv.html
2. Go to end of page. Above rows with "Comment" should be cell contaning "Editable segment" 
3. Focus it and start typing. 
[/list]

Actual Results:  
Typing is very slowly and browser freezes (especially in second file which is bigger). 

Expected Results:  
Typing should be normal (same as on top of the document - in first Editable segment). 

There is one more file attached - testgood.html, despite it being twice big as testbad.html it works perfectly fine. Difference is there are many tables and not everything inside one table. Testbaddiv.html works, is almost the same structure as testgood.html but has all tables wrapped in additional div element. It crashes performance completely. 
Bear in mind those are really simple test structures - in final production environment with more complex tables, nesting, it is unusable after about external 30 rows! 
As you noticed in testgood.html it is not because of HTML size or my computer - under any IE or Chrome it works fine - in any underlaying HTML structure so it looks like some really bad DOM traversal algorithm with too much recursion. Worrying is it wasn't fixed in FF 4 beta :(.
Hope you can do something about it.
Attachment #498991 - Attachment mime type: text/plain → text/html
Attachment #498990 - Attachment mime type: text/plain → text/html
Can you please test against a Nightly Build or Beta 8 when it comes out?
http://nightly.mozilla.org/
Component: General → Editor
Product: Firefox → Core
QA Contact: general → editor
Version: unspecified → Trunk
I tested against Minefield and I have to admit it got improved! First file with big table and nesting works fine and can be edited without any problems.  
Second file with many tables wrapped in single div works better (no more browser freezes) but still quite slow (every character appears after 5 - 10 seconds).

On beta 8 it is still not usable. Speed is same as it was on Beta 7.

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