181336 - small-caps are drawn in underline colour

Status: RESOLVED FIXED

(Core :: Layout: Text and Fonts, defect, P1)

Description

[Hixie (not reading bugmail)]

Testcase: http://www.hixie.ch/tests/adhoc/css/fonts/variant/001.xml

If you have font-variant: small-caps set with text-decoration: underline, the
colour of the small caps (not the big caps) is drawn the colour of the
underline, not the colour of the text (like the big caps).

Comment 1

[David Baron :dbaron:]

I'm guessing one chunk of code is doing a SetColor on a rendering context and
not undoing it, and another chunk of code is relying on the color not being changed.

Comment 2

[Boris Zbarsky [:bzbarsky]]

yeah nsTextFrame::RenderString never sets the color on the rendering context
(this is called from PaintTextSlowly) unliks PaintAsciiText, which does set the
color.

Comment 3

[David Baron :dbaron:]

Taking.

Comment 4

[Boris Zbarsky [:bzbarsky]]

Attachment: testcase

OK, this needs a new testcase, since it's now quirks-only, since bug 1777 never
triggers this ickiness.

The issue is that RenderString calls DrawString and DrawTextDecorations in a
loop one after the other.  So the calls to SetColor before the call into
RenderString are not enough; the color needs to be set before every DrawString
in RenderString

Comment 5

[Boris Zbarsky [:bzbarsky]]

Actually, not that simple.  Those SetColor() calls _used_ to be there. Then
anthonyd removed them as "redundant" and "breaking selection colors".  Too bad
there's no bug number cited and no steps to reproduce.... (see
http://bonsai.mozilla.org/cvsview2.cgi?diff_mode=context&whitespace_mode=show&root=/cvsroot&subdir=mozilla/layout/html/base/src&command=DIFF_FRAMESET&file=nsTextFrame.cpp&rev2=1.247&rev1=1.246)

Comment 6

[Boris Zbarsky [:bzbarsky]]

*** Bug 6574 has been marked as a duplicate of this bug. ***

Comment 7

[David Baron :dbaron:]

Attachment: patch

selection colors look fine to me

Comment 8

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 139752 [details] [diff] [review]
patch

Yeah, if selection colors are still good, r+sr=bzbarsky.

Comment 9

[David Baron :dbaron:]

Fix checked in to trunk, 2004-02-03 10:27 -0800.

Comment 10

[Boris Zbarsky [:bzbarsky]]

Hmm.. In a current build if I select the text "HAS: PASS" in that testcase, the
selection colors are off -- the "HAS:" is green while the "PASS" is white.  The
"HAS:" is white up until I select the "P".

Similarly, if I select the "SE" in "PASSED", the "PAS" is painted in white (so
it's white on white).

Reopening; that patch is no good...

Comment 11

[Isaac Hwak Han]

Selection problem mentioned in comment 10 is bug 235101. Adding dependency.

Comment 12

[David Baron :dbaron:]

(In reply to comment #11)
> Selection problem mentioned in comment 10 is bug 235101. Adding dependency.

...so this bug is still fixed.