Title: [240900] trunk/Tools
Revision
240900
Author
ddkil...@apple.com
Date
2019-02-02 12:20:36 -0800 (Sat, 02 Feb 2019)

Log Message

Leak of NSArray (4.25 Kbytes) in com.apple.WebKit.WebContent running WebKit layout tests on iOS Simulator
<https://webkit.org/b/194178>
<rdar://problem/47753473>

Reviewed by Alexey Proskuryakov.

* WebKitTestRunner/InjectedBundle/cocoa/ActivateFontsCocoa.mm:
(WTR::uninstallFakeHelvetica): Pass nullptr into
CTFontManagerUnregisterFontsForURLs() to ignore any errors.

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (240899 => 240900)


--- trunk/Tools/ChangeLog	2019-02-02 20:05:22 UTC (rev 240899)
+++ trunk/Tools/ChangeLog	2019-02-02 20:20:36 UTC (rev 240900)
@@ -1,3 +1,15 @@
+2019-02-02  David Kilzer  <ddkil...@apple.com>
+
+        Leak of NSArray (4.25 Kbytes) in com.apple.WebKit.WebContent running WebKit layout tests on iOS Simulator
+        <https://webkit.org/b/194178>
+        <rdar://problem/47753473>
+
+        Reviewed by Alexey Proskuryakov.
+
+        * WebKitTestRunner/InjectedBundle/cocoa/ActivateFontsCocoa.mm:
+        (WTR::uninstallFakeHelvetica): Pass nullptr into
+        CTFontManagerUnregisterFontsForURLs() to ignore any errors.
+
 2019-02-02  Zalan Bujtas  <za...@apple.com>
 
         [LFC] Initialize ICB's style with fixed width/height.

Modified: trunk/Tools/WebKitTestRunner/InjectedBundle/cocoa/ActivateFontsCocoa.mm (240899 => 240900)


--- trunk/Tools/WebKitTestRunner/InjectedBundle/cocoa/ActivateFontsCocoa.mm	2019-02-02 20:05:22 UTC (rev 240899)
+++ trunk/Tools/WebKitTestRunner/InjectedBundle/cocoa/ActivateFontsCocoa.mm	2019-02-02 20:20:36 UTC (rev 240900)
@@ -154,8 +154,7 @@
         if ([[url lastPathComponent] hasPrefix:@"FakeHelvetica"])
             [fontsToRemove addObject:url];
     }
-    CFArrayRef errors = nullptr;
-    CTFontManagerUnregisterFontsForURLs(static_cast<CFArrayRef>(fontsToRemove), kCTFontManagerScopeProcess, &errors);
+    CTFontManagerUnregisterFontsForURLs(static_cast<CFArrayRef>(fontsToRemove), kCTFontManagerScopeProcess, nullptr);
 }
 
 }
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to