Attachment 361767 Details for Bug 194529 – Patch v1

[patch] Patch v1

bug-194529-20190211213831.patch (text/plain), 6.91 KB, created by David Kilzer (:ddkilzer) on 2019-02-11 21:38:33 PST

(hide)

Description:

Filename:

MIME Type:

Creator: David Kilzer (:ddkilzer)

Created: 2019-02-11 21:38:33 PST

Size: 6.91 KB

patch

obsolete

>Subversion Revision: 241288
>diff --git a/Source/WTF/ChangeLog b/Source/WTF/ChangeLog
>index da5eece0d14cf1df21135d43f7d08485470f613b..be184245019da85b7b7536d13d99bc34ff2145c8 100644
>--- a/Source/WTF/ChangeLog
>+++ b/Source/WTF/ChangeLog
>@@ -1,3 +1,30 @@
>+2019-02-11  David Kilzer  <ddkilzer@apple.com>
>+
>+        REGRESSION (r238955, r240494): Soft-linking optional Lookup.framework triggers release assertion when missing
>+        <https://webkit.org/b/194529>
>+        <rdar://problem/47924449>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * wtf/cocoa/SoftLinking.h:
>+        (SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_IS_OPTIONAL): Rename
>+        SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_ASSERTION() to this
>+        and change `assertion` argument to `isOptional`.  Pass
>+        `isOptional` to framework##Library() method to control assertion
>+        behavior.  Only check RELEASE_ASSERT() if `!isOptional`, else
>+        that code should be optimized out by the compiler.  This fixes
>+        the crash.
>+        (NO_ASSERT): Remove macro since it's no longer used.
>+        (SOFT_LINK_IS_OPTIONAL): Add macro to use for soft-linking
>+        optional classes.
>+        (SOFT_LINK_IS_NOT_OPTIONAL): Add macro to use for soft-linking
>+        non-optional classes.
>+        (SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT): Update to use new
>+        SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_IS_OPTIONAL() macro.
>+        (SOFT_LINK_CLASS_FOR_SOURCE_OPTIONAL_WITH_EXPORT): Ditto.
>+        (SOFT_LINK_CLASS_FOR_SOURCE): Ditto.
>+        (SOFT_LINK_CLASS_FOR_SOURCE_OPTIONAL): Ditto.
>+
> 2019-02-11  Myles C. Maxfield  <mmaxfield@apple.com>
> 
>         [Cocoa] Ask platform for generic font family mappings
>diff --git a/Source/WebCore/ChangeLog b/Source/WebCore/ChangeLog
>index 18ee72539d313a2c7d68ca5782b3dc550a3e6eef..df3e6ca1a4ccdc98422db04fcafa49a52c576bf9 100644
>--- a/Source/WebCore/ChangeLog
>+++ b/Source/WebCore/ChangeLog
>@@ -1,3 +1,17 @@
>+2019-02-11  David Kilzer  <ddkilzer@apple.com>
>+
>+        REGRESSION (r238955, r240494): Soft-linking optional Lookup.framework triggers release assertion when missing
>+        <https://webkit.org/b/194529>
>+        <rdar://problem/47924449>
>+
>+        Reviewed by NOBODY (OOPS!).
>+
>+        * platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm:
>+        - Change use of SOFT_LINK_CLASS_FOR_SOURCE() macros to
>+          SOFT_LINK_CLASS() since the latter can only be used with
>+          SOFT_LINK_FRAMEWORK_OPTIONAL(AVFoundation).  This broke after
>+          the fix for <wtf/SoftLinking.h> was applied.
>+
> 2019-02-11  Myles C. Maxfield  <mmaxfield@apple.com>
> 
>         [Cocoa] Ask platform for generic font family mappings
>diff --git a/Source/WTF/wtf/cocoa/SoftLinking.h b/Source/WTF/wtf/cocoa/SoftLinking.h
>index c028cd3f665ec655ce5ca7861ddb575a8b8f9e14..e008322375a72933263b0bd4ecae6abfc78db539 100644
>--- a/Source/WTF/wtf/cocoa/SoftLinking.h
>+++ b/Source/WTF/wtf/cocoa/SoftLinking.h
>@@ -389,7 +389,7 @@
>     } \
>     }
> 
>-#define SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_ASSERTION(functionNamespace, framework, className, export, assertion) \
>+#define SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_IS_OPTIONAL(functionNamespace, framework, className, export, isOptional) \
>     @class className; \
>     namespace functionNamespace { \
>     static Class init##className(); \
>@@ -405,28 +405,30 @@
>     { \
>         static dispatch_once_t once; \
>         dispatch_once(&once, ^{ \
>-            framework##Library(); \
>+            framework##Library(isOptional); \
>             class##className = objc_getClass(#className); \
>-            assertion(class##className); \
>+            if (!isOptional) \
>+                RELEASE_ASSERT(class##className); \
>             get##className##Class = className##Function; \
>         }); \
>         return class##className; \
>     } \
>     }
> 
>-#define NO_ASSERT(assertion) (void(0))
>+#define SOFT_LINK_IS_OPTIONAL true
>+#define SOFT_LINK_IS_NOT_OPTIONAL false
> 
> #define SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT(functionNamespace, framework, className, export) \
>-    SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_ASSERTION(functionNamespace, framework, className, export, RELEASE_ASSERT)
>+    SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_IS_OPTIONAL(functionNamespace, framework, className, export, SOFT_LINK_IS_NOT_OPTIONAL)
> 
> #define SOFT_LINK_CLASS_FOR_SOURCE_OPTIONAL_WITH_EXPORT(functionNamespace, framework, className, export) \
>-    SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_ASSERTION(functionNamespace, framework, className, export, NO_ASSERT)
>+    SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_IS_OPTIONAL(functionNamespace, framework, className, export, SOFT_LINK_IS_OPTIONAL)
> 
> #define SOFT_LINK_CLASS_FOR_SOURCE(functionNamespace, framework, className) \
>-    SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_ASSERTION(functionNamespace, framework, className, , RELEASE_ASSERT)
>+    SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_IS_OPTIONAL(functionNamespace, framework, className, , SOFT_LINK_IS_NOT_OPTIONAL)
> 
> #define SOFT_LINK_CLASS_FOR_SOURCE_OPTIONAL(functionNamespace, framework, className) \
>-    SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_ASSERTION(functionNamespace, framework, className, , NO_ASSERT)
>+    SOFT_LINK_CLASS_FOR_SOURCE_WITH_EXPORT_AND_IS_OPTIONAL(functionNamespace, framework, className, , SOFT_LINK_IS_OPTIONAL)
> 
> #define SOFT_LINK_CONSTANT_FOR_HEADER(functionNamespace, framework, variableName, variableType) \
>     namespace functionNamespace { \
>diff --git a/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm b/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm
>index fd484d7340edf80fe56f2b0c05c27a66ed5620ca..770f637e11fc2365bb7ccb3e737ea2b5dfe25f5a 100644
>--- a/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm
>+++ b/Source/WebCore/platform/graphics/avfoundation/objc/MediaPlayerPrivateAVFoundationObjC.mm
>@@ -154,14 +154,14 @@ SOFT_LINK_FRAMEWORK_OPTIONAL(AVFoundation)
> 
> SOFT_LINK_FRAMEWORK_OPTIONAL(CoreImage)
> 
>-SOFT_LINK_CLASS_FOR_SOURCE(WebCore, AVFoundation, AVPlayer)
>-SOFT_LINK_CLASS_FOR_SOURCE(WebCore, AVFoundation, AVPlayerItem)
>-SOFT_LINK_CLASS_FOR_SOURCE(WebCore, AVFoundation, AVPlayerItemVideoOutput)
>-SOFT_LINK_CLASS_FOR_SOURCE(WebCore, AVFoundation, AVPlayerLayer)
>-SOFT_LINK_CLASS_FOR_SOURCE(WebCore, AVFoundation, AVURLAsset)
>-SOFT_LINK_CLASS_FOR_SOURCE(WebCore, AVFoundation, AVAssetImageGenerator)
>-SOFT_LINK_CLASS_FOR_SOURCE(WebCore, AVFoundation, AVMetadataItem)
>-SOFT_LINK_CLASS_FOR_SOURCE(WebCore, AVFoundation, AVAssetCache)
>+SOFT_LINK_CLASS(AVFoundation, AVPlayer)
>+SOFT_LINK_CLASS(AVFoundation, AVPlayerItem)
>+SOFT_LINK_CLASS(AVFoundation, AVPlayerItemVideoOutput)
>+SOFT_LINK_CLASS(AVFoundation, AVPlayerLayer)
>+SOFT_LINK_CLASS(AVFoundation, AVURLAsset)
>+SOFT_LINK_CLASS(AVFoundation, AVAssetImageGenerator)
>+SOFT_LINK_CLASS(AVFoundation, AVMetadataItem)
>+SOFT_LINK_CLASS(AVFoundation, AVAssetCache)
> 
> SOFT_LINK_CLASS(CoreImage, CIContext)
> SOFT_LINK_CLASS(CoreImage, CIImage)

Actions: View | Formatted Diff | Diff

Attachments on bug 194529: 361767 | 361771 | 361790 | 361801