[Cocoa] Suppress font download dialog in new matching algorithm
Created attachment 303025 [details] Patch
Comment on attachment 303025 [details] Patch Attachment 303025 [details] did not pass mac-ews (mac): Output: http://webkit-queues.webkit.org/results/3212340 New failing tests: media/modern-media-controls/volume-down-support/volume-down-support.html
Created attachment 303027 [details] Archive of layout-test-results from ews103 for mac-elcapitan The attached test failures were seen while running run-webkit-tests on the mac-ews. Bot: ews103 Port: mac-elcapitan Platform: Mac OS X 10.11.6
Comment on attachment 303025 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=303025&action=review > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:954 > + int one = 1; also static? > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:955 > + static NeverDestroyed<RetainPtr<CFNumberRef>> yes = adoptCF(CFNumberCreate(kCFAllocatorDefault, kCFNumberIntType, &one)); just "static CFNumberRef"
Committed r215314: <http://trac.webkit.org/changeset/215314>
Comment on attachment 303025 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=303025&action=review > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:957 > + CFTypeRef values[] = { yes.get().get(), postScriptNameString.get() }; Would kCFBooleanTrue not work here?!
(In reply to mitz from comment #6) > Comment on attachment 303025 [details] > Patch > > View in context: > https://bugs.webkit.org/attachment.cgi?id=303025&action=review > > > Source/WebCore/platform/graphics/cocoa/FontCacheCoreText.cpp:957 > > + CFTypeRef values[] = { yes.get().get(), postScriptNameString.get() }; > > Would kCFBooleanTrue not work here?! I didn't know about this! Thanks for telling me.
Committed r215331: <http://trac.webkit.org/changeset/215331>