Martijn Wargers (dead) Reporter
	Description • 17 years ago

See testcase, the text should not overlap the green block, but it does in current trunk build.

With the url, I sometimes see this happening on first load/shift-reload with the heading overlapping the image.

This regressed between 2006-12-07 and 2006-12-08, regression from bug 300030.

	Martijn Wargers (dead) Reporter
	Comment 1 • 17 years ago

I deduced this testcase from http://www.nydailynews.com/sports/more_sports/2007/09/10/2007-09-10_roger_federers_last_challenge_is_pete_sa.html

	Robert O'Callahan (:roc) (email my personal email if necessary) Assignee
	Comment 2 • 17 years ago

This turned out to be a lot simpler than I feared. It's a very simple bug in nsIntervalSet::IncludeInterval. Basically we have one interval already, let's say [100,200] and we add an overlapping interval, say [150,250] ... the new interval gets put in the front of the list, then it subsumes the old interval, and the current code just updates the endpoint of the new interval instead of doing a full union.

	David Baron :dbaron:
	Comment 3 • 17 years ago

Comment on attachment 304659 [details] [diff] [review]
fix

Er, yikes.  How'd I manage that?

	David Baron :dbaron:
	Comment 4 • 17 years ago

(Believe it or not, this is the first substantive change to the file since I landed the initial version over six years ago.  Not sure how this slipped through that long...)

	Robert O'Callahan (:roc) (email my personal email if necessary) Assignee
	Comment 5 • 17 years ago

checked in with reftest