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)
Tracking
()
VERIFIED
FIXED
People
(Reporter: dveditz, Assigned: smontagu)
References
()
Details
(Keywords: regression, verified1.9.0.12, verified1.9.1)
Attachments
(1 file)
(deleted),
patch
|
roc
:
review+
roc
:
superreview+
dveditz
:
approval1.9.0.12+
|
Details | Diff | Splinter Review |
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
Reporter | ||
Updated•16 years ago
|
Reporter | ||
Comment 1•16 years ago
|
||
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.
Reporter | ||
Updated•16 years ago
|
Assignee: nobody → dbaron
Flags: blocking1.9.0.12? → blocking1.9.0.12+
Updated•15 years ago
|
Flags: blocking1.9.0.11?
Comment 3•15 years ago
|
||
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.
Assignee | ||
Comment 5•15 years ago
|
||
(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+
Assignee | ||
Updated•15 years ago
|
Attachment #382473 -
Flags: approval1.9.0.12?
Reporter | ||
Updated•15 years ago
|
Attachment #382473 -
Flags: approval1.9.0.12? → approval1.9.0.12+
Reporter | ||
Comment 6•15 years ago
|
||
Comment on attachment 382473 [details] [diff] [review]
Patch
Approved for 1.9.0.12, a=dveditz for release-drivers
Assignee | ||
Comment 7•15 years ago
|
||
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
Updated•15 years ago
|
Keywords: fixed1.9.1
Comment 8•15 years ago
|
||
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.
Description
•