- Go to URL http://slip/projects/marvin/css/ (netscape internal location - behind firewall) - viewer starts to list a directory of all the files in this folder, then locks up and dies.

I made some performance improvements to the image frame code and it helped some, but the real problem is that the page has some 430 images without width and height tags. That means we end up doing 430 incremental reflows and since the block/inline code isn't optimized yet that means a whole heck of a lot of work. Rick, this can't be fixed by Monday. It's not a very big problem either because it's not common to see pages with so many images without width/height tags A related problem which we need to address at some point is that because there are only five or so unique images we get the size info quickly and then generate 430 reflow commands which we then continually process until we're finished. That's why it appears like we're hung.

*** Bug 1489 has been marked as a duplicate of this bug. ***

I updated the title to reflect the real problem. Also, I'm waiting on a bug fix in the parser (HR's closing PRE tags; bug #1731) before tackling the performance issue because the parser bug makes the issue much worse

Setting all current Open/Normal to M4.

Putting on [Perf] radar.

per leger, assigning QA contacts to all open bugs without QA contacts according to list at http://bugzilla.mozilla.org/describecomponents.cgi?product=Browser

*** Bug 7328 has been marked as a duplicate of this bug. ***

On my Win98 machine with build 1999080312, this problem does not occur. It does take a while to load the entire listing of directories, but it finishes loading fine and i am able to scroll through the entire list.

I've changed the cc list to include troy; this is the ideal testcase for the reflow-dirty optimizations we've been talking about.

With all of the current work in the content sink, block reflow optimization and so on, the page in question now loads pretty well. So I'm changing the milestone to later since its no longer a pressing issue.

Updating to default Layout Assignee...kipp no longer with us :-(

Nisheeth, another example of the need to reflow command coalescing

Setting milestone to M13...

Bulk add of "perf" to new keyword field. This will replace the [PERF] we were using in the Status Summary field.

This bug is fixed now that blocks and inlines are coalescing reflow commands. The number of reflow commands generated for this page has reduced from 257 to 6. There is a perceptible decrease in layout time of about 1-2 seconds (on a Pentium 3, 733 Mhz, 192 MB RAM).

Fixed in the Feb 22 build.