<html>
    <head>
      <base href="https://bugs.webkit.org/" />
    </head>
    <body><span class="vcard"><a class="email" href="mailto:mrobinson@webkit.org" title="Martin Robinson <mrobinson@webkit.org>"> <span class="fn">Martin Robinson</span></a>
</span> changed
              <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GTK] Fonts loaded via @font-face look bad"
   href="https://bugs.webkit.org/show_bug.cgi?id=140994">bug 140994</a>
        <br>
             <table border="1" cellspacing="0" cellpadding="8">
          <tr>
            <th>What</th>
            <th>Removed</th>
            <th>Added</th>
          </tr>

         <tr>
           <td style="text-align:right;">Attachment #247101 Flags</td>
           <td>review?, commit-queue?
           </td>
           <td>review-
           </td>
         </tr></table>
      <p>
        <div>
            <b><a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GTK] Fonts loaded via @font-face look bad"
   href="https://bugs.webkit.org/show_bug.cgi?id=140994#c19">Comment # 19</a>
              on <a class="bz_bug_link 
          bz_status_NEW "
   title="NEW - [GTK] Fonts loaded via @font-face look bad"
   href="https://bugs.webkit.org/show_bug.cgi?id=140994">bug 140994</a>
              from <span class="vcard"><a class="email" href="mailto:mrobinson@webkit.org" title="Martin Robinson <mrobinson@webkit.org>"> <span class="fn">Martin Robinson</span></a>
</span></b>
        <pre>Comment on <span class=""><a href="attachment.cgi?id=247101&action=diff" name="attach_247101" title="[GTK] Fonts loaded via @font-face look bad">attachment 247101</a> <a href="attachment.cgi?id=247101&action=edit" title="[GTK] Fonts loaded via @font-face look bad">[details]</a></span>
[GTK] Fonts loaded via @font-face look bad

View in context: <a href="https://bugs.webkit.org/attachment.cgi?id=247101&action=review">https://bugs.webkit.org/attachment.cgi?id=247101&action=review</a>

Looks good, though I have some minor suggestions.

<span class="quote">> Source/WebCore/ChangeLog:9
> +        Reviewed by NOBODY (OOPS!).
> +
> +        No new tests because I haven't found a freely-distributable font that triggers the bug.
> +</span >

This is a great change, but I think it deserves a full explanation in the changelog which describes the change and why it fixes the problem.

<span class="quote">> Source/WebCore/platform/graphics/freetype/FontPlatformDataFreeType.cpp:129
> +    RefPtr<FcPattern> pattern = adoptRef(FcPatternCreate());
> +    FcConfigSubstitute(nullptr, pattern.get(), FcMatchPattern);
> +    FcDefaultSubstitute(pattern.get());
> +    FcPatternRemove(pattern.get(), FC_FAMILY, 0);
> +    FcConfigSubstitute(nullptr, pattern.get(), FcMatchFont);
> +
> +    return pattern;</span >

We can probably allocate this once and simply keep returning it from a static variable.

<span class="quote">> Source/WebCore/platform/graphics/freetype/FontPlatformDataFreeType.cpp:186
> -    : m_fallbacks(nullptr)
> +    : m_pattern(getDefaultFontconfigOptions())
> +    , m_fallbacks(nullptr)</span >

Instead of making a new pattern per font, I think you should explicitly handle a null pattern in FontPlatformData::initializeWithFontFace. The way you have this now, it looks like it makes two separate instances with otherwise equal data, fail the equality test.</pre>
        </div>
      </p>
      <hr>
      <span>You are receiving this mail because:</span>
      
      <ul>
          <li>You are the assignee for the bug.</li>
      </ul>
    </body>
</html>