806284 - Resizing a window and reverting it moves some text

Status: RESOLVED FIXED

(Core :: Layout: Block and Inline, defect)

Description

[m101010a]

User Agent: Mozilla/5.0 (X11; Linux x86_64; rv:16.0) Gecko/20100101 Firefox/16.0
Build ID: 20121012050625

Steps to reproduce:

0. Have a window at least 800 pixels wide
1. Visit http://blogs.msdn.com/b/oldnewthing/archive/2012/10/26/10362864.aspx
2. Observe that you can read the entire blockquoted IPv6 address.
3. Resize the window to be 320 pixels wide.
4. Observe that the article content is no longer visible.
5. Revert the window's dimensions.



Actual results:

The blockquoted IPv6 address now hangs off the left side of the blockquote, and is mostly obscured.


Expected results:

The blockquoted IPv6 address should have looked exactly like it did before I resized the window twice.

Comment 1

[Alice0775 White]

Attachment: reduced html

Regression window(m-c)
Good:
http://hg.mozilla.org/mozilla-central/rev/8f658d460c3d
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:12.0a1) Gecko/20111231 Firefox/12.0a1 ID:20111231031039
Bad:
http://hg.mozilla.org/mozilla-central/rev/d77b056ed4bd
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:12.0a1) Gecko/20120101 Firefox/12.0a1 ID:20120101031015
Pushlog:
http://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=8f658d460c3d&tochange=d77b056ed4bd


Regression window(m-i)
Good:
http://hg.mozilla.org/integration/mozilla-inbound/rev/f2610136fb54
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:12.0a1) Gecko/20111230 Firefox/12.0a1 ID:20111230140656
Bad:
http://hg.mozilla.org/integration/mozilla-inbound/rev/b3113b7753c1
Mozilla/5.0 (Windows NT 6.1; WOW64; rv:12.0a1) Gecko/20111230 Firefox/12.0a1 ID:20111230161855
Pushlog:
http://hg.mozilla.org/integration/mozilla-inbound/pushloghtml?fromchange=f2610136fb54&tochange=b3113b7753c1
Suspected: Bug 713856

Comment 2

[(no longer active)]

Jet, can you please see if someone on the layout team can take this?  Should be reasonably easy to debug, given the small size of the regressing patch.

Comment 3

[Jet Villegas (inactive)]

+cc: Mats for a look.

Comment 4

[Mats Palmgren (inactive)]

This bug also occurs in Firefox 3.6.  The bug was hidden by the erroneous code
added in bug 389321 for Firefox 4 and when that error was corrected in bug 713856
the original bug surfaced.

Comment 5

[Mats Palmgren (inactive)]

Attachment: fix

The problem is that the line isn't reflowed due to a bogus optimization
for eResize reflows in nsBlockFrame::PrepareResizeReflow.  I think we
should just skip that optimization for RTL for now and deal with it
in a separate bug.

Comment 6

[Mats Palmgren (inactive)]

Attachment: fix (wdiff)

https://tbpl.mozilla.org/?tree=Try&rev=78f868f92f19

Comment 7

[Mats Palmgren (inactive)]

https://hg.mozilla.org/integration/mozilla-inbound/rev/e71b89c8cc84

Comment 8

[Ed Morley [:emorley]]

https://hg.mozilla.org/mozilla-central/rev/e71b89c8cc84