Bug 109535 – [Cairo] Anti-aliasing should not be always disabled for lines

• Status: RESOLVED
• Resolution: FIXED
• Component: WebCore Misc.

[Laszlo Gombos]

GraphicsContextCairo.cpp::drawLineOnCairoContext() sets anti-aliasing to CAIRO_ANTIALIAS_NONE with no condition. This is not desired and no other ports are handling anti-aliasing the same way. Antialiasing is desired e.g. when the page is scaled.

[Viatcheslav Ostapenko]

Created attachment 194547 [details]
Unscaled page sample with underlines.

[Viatcheslav Ostapenko]

Created attachment 194550 [details]
Scaled page sample.

Notice, that "Global Attributes in HTML" underline is too bold and "Event Attribute in HTML" underline disappeared.

[Viatcheslav Ostapenko]

Created attachment 194693 [details]
Patch

[Martin Robinson]

Comment on attachment 194693 [details]
Patch

View in context: https://bugs.webkit.org/attachment.cgi?id=194693&action=review

> LayoutTests/ChangeLog:10
> +        * fast/css3-text/css3-text-decoration/text-decoration-line-scaled-expected.txt: Added.

What platform is this result for? I imagine that few platforms will be able to share this result.

[Viatcheslav Ostapenko]

(In reply to comment #4)
> (From update of attachment 194693 [details])
> View in context: https://bugs.webkit.org/attachment.cgi?id=194693&action=review
> 
> > LayoutTests/ChangeLog:10
> > +        * fast/css3-text/css3-text-decoration/text-decoration-line-scaled-expected.txt: Added.
> 
> What platform is this result for? I imagine that few platforms will be able to share this result.

IMHO, GTK should be able to share. And other cairo platforms. Chromium has somewhat strange code there and still has a problem similar to what EFL has without patch (like on scaled page sample).

[Martin Robinson]

(In reply to comment #5)
 
> > What platform is this result for? I imagine that few platforms will be able to share this result.
> 
> IMHO, GTK should be able to share. And other cairo platforms. Chromium has somewhat strange code there and still has a problem similar to what EFL has 
without patch (like on scaled page sample).

Sorry my comment was pretty unclear. The result in your patch is placed in the platform-independent directory, but it's almost certain that Mac, Qt, Chromium and all other ports will required separate baselines.

[Viatcheslav Ostapenko]

(In reply to comment #6)
> (In reply to comment #5)
> 
> > > What platform is this result for? I imagine that few platforms will be able to share this result.
> > 
> > IMHO, GTK should be able to share. And other cairo platforms. Chromium has somewhat strange code there and still has a problem similar to what EFL has 
> without patch (like on scaled page sample).
> 
> Sorry my comment was pretty unclear. The result in your patch is placed in the platform-independent directory, but it's almost certain that Mac, Qt, Chromium and all other ports will required separate baselines.

Test results that are in common directory are generated for chromium build. Should I move it to chromium platform results?

[Martin Robinson]

(In reply to comment #7)

> Test results that are in common directory are generated for chromium build. Should I move it to chromium platform results?

Sorry for the late reply. I think these need to be moved to Chromium specific directories. I'm not sure if they need to be specific to Chromium linux or all Chromium ports.

[Viatcheslav Ostapenko]

Created attachment 200444 [details]
Patch for test results

[Viatcheslav Ostapenko]

Created attachment 200447 [details]
Patch for test results

[Viatcheslav Ostapenko]

Created attachment 200451 [details]
Failing test to get pixel test results from EWS. Not for review!!!

[Viatcheslav Ostapenko]

Created attachment 200469 [details]
Patch for landing

[WebKit Commit Bot]

Comment on attachment 200469 [details]
Patch for landing

Clearing flags on attachment: 200469

Committed r149540: <http://trac.webkit.org/changeset/149540>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.