This came from: http://forums.mozillazine.org/viewtopic.php?t=328434&highlight= The url testcase fails to render in current trunk builds and in Firefox1.5beta2 This used to work in the 2005-08-24 build, but doesn't work in the 2005-08-25 build: 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=2005-08-24+12%3A00%3A00&maxdate=2005-08-25+09%3A00%3A00&cvsroot=%2Fcvsroot Maybe because of bug 253951?
Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8b5) Gecko/20051011 Firefox/1.4.1 ID:2005101113 Same for Branch
Comment on attachment 199224 [details] [diff] [review] Fix This is just a copy of what we do for <body>. The real issue is that OpenHTML and CloseHTML do nothing of the sort, so we never actually notify on the kids of <html> through the normal codepath! This used to work, sorta, because StartLayout() didn't happen till we hit CloseFrameset(), but that's no longer true.
Requesting blocking; I think it's worth getting this regression fixed...
Comment on attachment 199224 [details] [diff] [review] Fix sr=jst
Fixed on trunk.
To expand on that approval request, this should be fairly safe -- we've been using this exact code for <body> for ages, and the only reason we didn't use it for <frameset> was because it wasn't strictly necessary. Now it is.
Fixed on branch.
It looks like on trunk this got busted by the checkin for bug 311785; I'll try to refix it in bug 315369, probably.