270032 - Floating div with overflow auto flickers while changing font size

Status: RESOLVED FIXED

(Core :: Layout: Floats, defect, P2)

Description

[Hadrien Nilsson]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; fr-FR; rv:1.7.5) Gecko/20041108 Firefox/1.0
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US; rv:1.8a5) Gecko/20041113

This bug is very likely to be related to bug 132337. 

Load the example URL, which is the « Simplified testcase » in bug 132337. The
second div flickers on the top left part of the window when your dynamically
change the font size with Ctrl++ and Ctrl+-. The effect is increased when you
are zooming with Ctrl+MouseWheel.


Reproducible: Always
Steps to Reproduce:

Actual Results:  
Flickering of the second div, moving from its position to the top left corner of
the window, then back to its correct position.

Expected Results:  
The second div should go to its correct position without flickering to the top
left corner of the window.

I've been able to same kind of flickering with a floatting iframe :

#Main {	width: 300px; float: left; background: #ff0; }
#Ban { float: left; }

<div id="Main">ABC</div>
<div id="Ban"> <iframe name="google_ads_frame"
src="iframe-content.html"></iframe> </div>

I will attach a complete file.

Comment 1

[Hadrien Nilsson]

Attachment: floatting iframe flickering test case

I reduced even more this test case.

Comment 2

[José Jeria]

Seing this as well with trunk build 20041115, Windows XP

Comment 3

[Boris Zbarsky [:bzbarsky]]

Attachment: Patch

Comment 4

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 167500 [details] [diff] [review]
Patch

The part that fixes the bug is the view update batch; the rest is minor
cleanup...

I'm not sure what I think of the invalidation that
nsPresShell::StyleChangeReflow does.  Perhaps it shouldn't do it?  I suspect
that would also fix the issue here.

Comment 5

[Robert O'Callahan (:roc) (email my personal email if necessary)]

Comment on attachment 167500 [details] [diff] [review]
Patch

sweet.

I didn't know we could have auto-sizing document children

Comment 6

[Boris Zbarsky [:bzbarsky]]

We can't yet, but I keep hoping to fix that sometime. ;)

Comment 7

[Boris Zbarsky [:bzbarsky]]

Fixed on trunk.