[webkit-reviews] review granted: [Bug 137454] [Mac] We are spending a lot of time loading fonts when loading weather.com : [Attachment 239450] Patch

[bugzilla-daemon at webkit.org]

Darin Adler <darin at apple.com> has granted Chris Dumez <cdumez at apple.com>'s
request for review:
Bug 137454: [Mac] We are spending a lot of time loading fonts when loading
weather.com
https://bugs.webkit.org/show_bug.cgi?id=137454

Attachment 239450: Patch
https://bugs.webkit.org/attachment.cgi?id=239450&action=review

------- Additional Comments from Darin Adler <darin at apple.com>
View in context: https://bugs.webkit.org/attachment.cgi?id=239450&action=review


> Source/WebCore/platform/graphics/mac/FontCacheMac.mm:104
> +    if (knownFamilies.get().size() == maxCacheSize)

I’d also assert that size is <= maxCacheSize.