29584 - Exponential time to open text files in the Editor

Status: RESOLVED WORKSFORME

(Core :: DOM: Editor, defect, P3)

Description

[Pierre Saslawsky]

Tested with fairly recent (2 days old) builds on Mac and Windows.

- Create a text file with about 50 small lines of text.
- Give the file a ".html" extension
- Open the file in the Editor
==> It goes fast

- Give the file a ".txt" extension
- open the file in the Editor
==> It's slow (a couple of seconds)

- Increase the size of the file to about 150 lines
- Keep the ".txt" extension and open in the editor
==> It's incredibly slow (10-15 seconds on G4 and PC)

- Rename the file with a ".html" extension
- Open it in Navigator or in the Editor
==> It goes fast

Conclusion: the Editor is VERY slow with ".txt" files compared to ".html" files 
and it seems to be exponentially slower as the size increases, even for files as 
small as 150 lines of text.

Comment 1

[Pierre Saslawsky]

Attachment: The text file I used...

Comment 2

[Kathleen :Brade]

hand over to Simon

Comment 3

[Simon Fraser [no longer active]]

This is covered under 28783, but i'll leave it open in case there are some other 
performance issues.

Comment 4

[Simon Fraser [no longer active]]

M16

Comment 5

[Simon Fraser [no longer active]]

M17 for perf.

Comment 6

[rubydoo123]

setting to m19

Comment 7

[rubydoo123]

please do not nominate for nsbeta3 -- this will be addressed after crashes, 
regressions, correctness and polish bugs have been resolved, all Composer perf 
bugs have been set to m19/m20

Comment 8

[rubydoo123]

moving out ot future, will concentrate on this type of work post rtm

Comment 9

[Simon Fraser [no longer active]]

Performance -> 0.9.1

Comment 10

[Simon Fraser [no longer active]]

-> 0.9.2

Comment 11

[Mike Shaver (:shaver -- probably not reading bugmail closely)]

Does my latest patch in 74656, along with a good environment setting for
$BLOCK_FRAME_LINE_CACHE_THRESHOLD (25?) help this problem at all?

I'll do some testing when I resurrect my build, but I thought I'd throw it out.
 (Kudos to rbs for tipping me off to this bug.)

Comment 12

[Simon Fraser [no longer active]]

This bug really sucks now that we can load big plain text files. I'll try 
shaver's patches.

Comment 13

[Joe Francis]

simon, any results here with the patches?

Comment 14

[rubydoo123]

sorry, this needs to go over to 1.0

Comment 15

[Asa Dotzler [:asa]]

Bugs targeted at mozilla1.0 without the mozilla1.0 keyword moved to mozilla1.0.1 
(you can query for this string to delete spam or retrieve the list of bugs I've 
moved)

Comment 16

[rubydoo123]

removing myself from the cc list

Comment 17

[Simon Fraser [no longer active]]

Pre-sabbatical bug triage. ->jfrancis

Comment 18

[Joe Francis]

The trunk is the wave of the future!

Comment 19

[Joe Francis]

The days of having a half dozen milestones out in front of us to divide bugs 
between seem to be gone, though I dont know why.  Lumping everything together as 
far out as I can.  I'll pull back things that I am working on as I go.

Comment 20

[Joe Francis]

migrating milestones now that there are more available.  i'll pull these back as
I work on them

Comment 21

[Wayne Mery (:wsmwk)]

opens instanteous for me on Thinkpad T40 150 lines. 1200 lines is about 10 secs. 

Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8.0.4) Gecko/20060516 SeaMonkey/1.0.2

similar results with 
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.9a1) Gecko/20060625 SeaMonkey/1.5a

Comment 22

[Wayne Mery (:wsmwk)]

=> WFM