[webkit-reviews] review granted: [Bug 223985] [Cocoa] REGRESSION(r272999): User-installed fonts no longer work in Mail : [Attachment 424873] Patch

[bugzilla-daemon at webkit.org]

Per Arne Vollan <pvollan at apple.com> has granted Myles C. Maxfield
<mmaxfield at apple.com>'s request for review:
Bug 223985: [Cocoa] REGRESSION(r272999): User-installed fonts no longer work in
Mail
https://bugs.webkit.org/show_bug.cgi?id=223985

Attachment 424873: Patch

https://bugs.webkit.org/attachment.cgi?id=424873&action=review




--- Comment #5 from Per Arne Vollan <pvollan at apple.com> ---
Comment on attachment 424873
  --> https://bugs.webkit.org/attachment.cgi?id=424873
Patch

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

Thanks! R=me.

> Source/WebKit/WebProcess/WebPage/WebPage.h:-2275
> -
> -#if HAVE(STATIC_FONT_REGISTRY)
> -    RefPtr<SandboxExtension> m_fontExtension;
> -#endif

You may also be able to remove the font extension member in WebProcess, since
you can consume the extension permanently.