Bug 98061 - Disable hinting when webkit-font-smoothing:antialiased is used on Mac.
Summary: Disable hinting when webkit-font-smoothing:antialiased is used on Mac.
Status: RESOLVED FIXED
Alias: None
Product: WebKit
Classification: Unclassified
Component: Platform (show other bugs)
Version: 528+ (Nightly build)
Hardware: Mac All
: P2 Normal
Assignee: bungeman
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2012-10-01 12:53 PDT by bungeman
Modified: 2012-10-09 13:06 PDT (History)
4 users (show)

See Also:


Attachments
Patch (1.98 KB, patch)
2012-10-01 13:02 PDT, bungeman
no flags Details | Formatted Diff | Diff
Patch (2.01 KB, patch)
2012-10-05 13:17 PDT, bungeman
no flags Details | Formatted Diff | Diff
Patch (2.02 KB, patch)
2012-10-06 09:14 PDT, bungeman
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description bungeman 2012-10-01 12:53:54 PDT
Disable hinting when webkit-font-smoothing:antialiased is used on Mac.
Comment 1 bungeman 2012-10-01 13:02:56 PDT
Created attachment 166529 [details]
Patch
Comment 2 Mike Reed 2012-10-02 07:55:12 PDT
are we putting this on hold, pending the search for other mechanisms that are more explicit?
Comment 3 bungeman 2012-10-05 13:17:48 PDT
Created attachment 167371 [details]
Patch
Comment 4 bungeman 2012-10-05 13:33:38 PDT
USE(SKIA_ON_MAC_CHROMIUM) is currently always true when FontSkia.cpp is included. However, this documents for the future merging of the various 'Font' implementations that this is Mac specific. Requesting re-evaluation now that the Skia changes have settled down.
Comment 5 Mike Reed 2012-10-05 13:48:28 PDT
works for me
Comment 6 Stephen White 2012-10-05 14:18:37 PDT
Comment on attachment 167371 [details]
Patch

I think we prefer #if PLATFORM(CHROMIUM) && OS(DARWIN) in platform/graphics/skia.  SKIA_ON_MAC_CHROMIUM only seems to be used in cross-platform code, and should be replaced with the above now anyway.
Comment 7 Build Bot 2012-10-05 14:47:13 PDT
Comment on attachment 167371 [details]
Patch

Attachment 167371 [details] did not pass mac-ews (mac):
Output: http://queues.webkit.org/results/14183532

New failing tests:
http/tests/workers/terminate-during-sync-operation.html
Comment 8 bungeman 2012-10-06 09:14:54 PDT
Created attachment 167457 [details]
Patch
Comment 9 bungeman 2012-10-06 09:34:23 PDT
Changed test to '#if OS(DARWIN)' since this is the desired behavior, Chromium or not (though Chromium is the only one currently using this file).

It would be nice if the test could be more like '#if SKIA_FONT_HOST(CORE_GRAPHICS)' since that's what we're really getting at here. The webkit-font-smoothing css property is just exposing a CoreGraphics API. It just happens to be that currently 'OS(DARWIN)' will have the same value as the ephemeral 'SKIA_FONT_HOST(CORE_GRAPHICS)'. That's why I proposed using SKIA_ON_MAC_CHROMIUM, as it seemed closer to 'SKIA_FONT_HOST(CORE_GRAPHICS)'. The test '#if OS(DARWIN)' is fine for now, but I have modified the comment to clarify the situation.
Comment 10 Stephen White 2012-10-09 13:00:10 PDT
Comment on attachment 167457 [details]
Patch

OK.  r=me
Comment 11 WebKit Review Bot 2012-10-09 13:06:18 PDT
Comment on attachment 167457 [details]
Patch

Clearing flags on attachment: 167457

Committed r130800: <http://trac.webkit.org/changeset/130800>
Comment 12 WebKit Review Bot 2012-10-09 13:06:22 PDT
All reviewed patches have been landed.  Closing bug.