Closed
Bug 317561
Opened 19 years ago
Closed 15 years ago
Hang CPU loop on google "view as HTML"
Categories
(Core :: Layout: Positioned, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: w.langdon, Unassigned)
References
()
Details
(Keywords: perf, regression, Whiteboard: [reflow-refactor])
Attachments
(2 files)
64.33 KB,
application/zip
|
Details | |
4.67 KB,
patch
|
bzbarsky
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051119 Firefox/1.6a1 Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051119 Firefox/1.6a1 Search google for: statistics "standard error for ratios (note placement of double quote) google page comes up as usual. Click on view as HTML Deerpark alpha 2 hangs The cursor turns to the hour glass. Within a few seconds firefox is consuming all the CPU it can get its hands on. It never comes back. Repeated three or more times whilst trying to report this problem. Error report sent via desktop icons to microsoft (my reading of talkback web pages suggests it cannot be used with this sort of fault). Reproducible: Always Actual Results: see screenshot google URL is http://216.239.59.104/search?q=cache:3Mq3Vq_p6RgJ:www.cdc.gov/nchs/data/series/sr_02/sr02_125.pdf+statistics+%22standard+error+for+ratios%22&hl=en if this is loaded via usual then browser hands in same way I tried 2 other browsers, both are ok Netscape 7.2 Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.7.2) Gecko/20040804 Netscape/7.2 (ax) IE 6
screen shot is at http://www.cs.ucl.ac.uk/staff/W.Langdon/317561.gif code downloaded (via Netscape 7.2) is at http://www.cs.ucl.ac.uk/staff/W.Langdon/317561.html
Comment 2•19 years ago
|
||
Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a1) Gecko/20051123 Firefox/1.6a1 ID:2005112305 This is a regression somewhere between 25-Nov-2004 and 05-Dec-2004.
Comment 3•19 years ago
|
||
I can see the regression between 2004-11-26 and 2004-11-28: http://bonsai.mozilla.org/cvsquery.cgi?treeid=default&module=all&branch=HEAD&branchtype=match&dir=&file=&filetype=match&who=&whotype=match&sortby=Date&hours=2&date=explicit&mindate=2004-11-26+06%3A00%3A00&maxdate=2004-11-28+09%3A00%3A00&cvsroot=%2Fcvsroot So I think this could be a regression from one of the check-ins from bug 201897. This one exactly: http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&subdir=mozilla/layout/html/base/src&command=DIFF_FRAMESET&file=nsAbsoluteContainingBlock.cpp&rev1=1.54&rev2=1.55&root=/cvsroot
Blocks: 201897
Component: General → Layout: R & A Pos
Product: Firefox → Core
QA Contact: general → layout.r-and-a-pos
Version: unspecified → Trunk
(In reply to comment #3) Can I delete http://www.cs.ucl.ac.uk/staff/W.Langdon/317561.html ? Have you taken a copy of it? I dont like to leave broken www pages on my home page. Thanks again Bill
Comment 5•19 years ago
|
||
It can be attached to the bug.
Updated•19 years ago
|
Status: UNCONFIRMED → NEW
Ever confirmed: true
OS: Windows XP → All
Comment 6•19 years ago
|
||
Bug 160665 is also mentions Google "view as HTML", so that bug is probably related to this.
Comment 7•19 years ago
|
||
Some statistics on the first testcase: 2004-11-26-08: 2 seconds to render 2005-01-11-07: 13 minutes 2005-11-25-02: 8.5 minutes Number of calls to nsAbsoluteContainingBlock::ReflowAbsoluteFrame: Aug 2004 (roughly) debug build: 22135 current debug build: 13686788 The reason is the second reflow introduced in bug 201897 for shrink-to-fit calculation. This is by design and not an error. Of course, this results in bad performance in the case we have nested shrink-to-fit abs. blocks, in the worst case 2^N where N is the depth of the tree. In the first testcase there are several bogus end tags: <div style="position:absolute;top:476;left:55"><nobr>Health St<b style="color:black;background-color:#ffff66">atistics</nobr></</b>div> So the DIV becomes an ancestor for the rest of the elements and there are 11 of these errors generated by Google's PDF to HTML translator. For the short term, I think the best solution is to make Google fix their bug - the page renders within a few seconds with the above markup errors fixed. I filed Evangelism bug 319081 on that.
Keywords: perf
Comment 9•17 years ago
|
||
Did you mean to close this, or are you keeping it open for performance profiling?
Updated•17 years ago
|
Keywords: regression
Comment 11•15 years ago
|
||
A lot of open abs pos div elements. Fx2: hang, Fx3+: works perfectly
Attachment #373491 -
Flags: review?(bzbarsky)
Status: NEW → RESOLVED
Closed: 15 years ago
Flags: in-testsuite?
Resolution: --- → FIXED
Updated•15 years ago
|
Attachment #373491 -
Flags: review?(bzbarsky) → review+
Comment 12•15 years ago
|
||
Pushed the test as http://hg.mozilla.org/mozilla-central/rev/1f9b126ecec0
You need to log in
before you can comment on or make changes to this bug.
Description
•