Closed Bug 493652 Opened 16 years ago Closed 15 years ago

First-line style applies to two lines when first-letter style also used

Categories

(Core :: Layout, defect)

1.9.0 Branch
x86
macOS
defect
Not set
normal

Tracking

()

VERIFIED FIXED

People

(Reporter: dveditz, Assigned: smontagu)

References

()

Details

(Keywords: regression, verified1.9.0.12, verified1.9.1)

Attachments

(1 file)

The patch for crash bug 429969 (which was actually in bug 429968) caused a regression in the reftest for bug 23604 -- with both a first-letter and first-line style the first-line style applies to the first TWO lines. After a dynamic change (as in the 23604-1.html test) things appear OK, or if you remove the first-letter style. 1.9.0 branch, 1.9.1 seems OK. Correct: http://mxr.mozilla.org/mozilla/source/layout/reftests/bugs/23604-1.html Incorrect: http://mxr.mozilla.org/mozilla/source/layout/reftests/bugs/23604-1-ref.html
I would imagine this is a common combination.
Flags: wanted1.9.0.x+
Flags: blocking1.9.0.12?
Flags: blocking1.9.0.11?
The best thing to do would be to figure out which patches in 1.9.1 fix the bug and pull them over.
Assignee: nobody → dbaron
Flags: blocking1.9.0.12? → blocking1.9.0.12+
Flags: blocking1.9.0.11?
Why did this get assigned to me?
Sorry, I don't know. Dan, is there anyone else who could do this? We just need to apply the patch for bug 429968 to where 1.9.1 split from 1.9.0, and then bisect along 1.9.1 to find out where the bug gets fixed.
Attached patch Patch (deleted) — Splinter Review
(In reply to comment #4) > We just need > to apply the patch for bug 429968 to where 1.9.1 split from 1.9.0, and then > bisect along 1.9.1 to find out where the bug gets fixed. Or we could just look at the changes and try to figure out what other changes might interact with them :-P This is taken from bug 441418, for the record.
Assignee: dbaron → smontagu
Attachment #382473 - Flags: superreview?(roc)
Attachment #382473 - Flags: review?(roc)
Attachment #382473 - Flags: superreview?(roc)
Attachment #382473 - Flags: superreview+
Attachment #382473 - Flags: review?(roc)
Attachment #382473 - Flags: review+
Attachment #382473 - Flags: approval1.9.0.12?
Attachment #382473 - Flags: approval1.9.0.12? → approval1.9.0.12+
Comment on attachment 382473 [details] [diff] [review] Patch Approved for 1.9.0.12, a=dveditz for release-drivers
Checking in layout/reftests/bugs/reftest.list; /cvsroot/mozilla/layout/reftests/bugs/reftest.list,v <-- reftest.list new revision: 1.477; previous revision: 1.476 done Checking in layout/generic/nsTextFrameThebes.cpp; /cvsroot/mozilla/layout/generic/nsTextFrameThebes.cpp,v <-- nsTextFrameThebes.cpp new revision: 3.192; previous revision: 3.191 done
Status: NEW → RESOLVED
Closed: 15 years ago
Flags: in-testsuite+
Keywords: fixed1.9.0.12
Resolution: --- → FIXED
v. Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.0.12) Gecko/2009070611 Firefox/3.0.12 Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1) Gecko/20090624 Firefox/3.5 Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.2a1pre) Gecko/20090716 Minefield/3.6a1pre
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: