[chromium] linux: prefer the "hinting" fontconfig preference.

[Adam Langley]

fontconfig can give contradictory hinting information, setting "hinting" to zero and "hintstyle" to non-zero. In this case we should take the "hinting" preference.

[Adam Langley]

Created attachment 51641 [details]
patch

[Evan Martin]

LGTM

[David Levin]

Comment on attachment 51641 [details]
patch


> diff --git a/WebCore/platform/graphics/chromium/FontPlatformDataLinux.cpp b/WebCore/platform/graphics/chromium/FontPlatformDataLinux.cpp
> +    default:
> +        paint->setHinting((SkPaint::Hinting) m_style.hintStyle);

Please use C++ style casting like this: 
  paint->setHinting(static_cast<SkPaint::Hinting>(m_style.hintStyle));

[Eric Seidel (no email)]

Attachment 51641 [details] was posted by a committer and has review+, assigning to Adam Langley for commit.

[Eric Seidel (no email)]

Unsure of the status of this patch.  It's been in pending-commit for over a month.  Updates?

[Tony Chang]

(In reply to comment #5)
> Unsure of the status of this patch.  It's been in pending-commit for over a month.  Updates?

This was landed in http://trac.webkit.org/changeset/56800.