Changeset 256584 in webkit

Timestamp:

Feb 13, 2020 7:48:34 PM (4 years ago)

Author:

jer.noble@apple.com

Message:

MediaToolbox may not be available; check before calling MTOverrideShouldPlayHDRVideo()
​https://bugs.webkit.org/show_bug.cgi?id=207733
<rdar://problem/59441647>

Reviewed by Maciej Stachowiak.

• platform/mac/PlatformScreenMac.mm:

(WebCore::setShouldOverrideScreenSupportsHighDynamicRange):

Location:

trunk/Source/WebCore

Files:

• ChangeLog (modified) (1 diff)
• platform/mac/PlatformScreenMac.mm (modified) (1 diff)

trunk/Source/WebCore/ChangeLog

@@ +1 @@
+2020-02-13  Jer Noble  <jer.noble@apple.com>
+
+        MediaToolbox may not be available; check before calling MTOverrideShouldPlayHDRVideo()
+        https://bugs.webkit.org/show_bug.cgi?id=207733
+        <rdar://problem/59441647>
+
+        Reviewed by Maciej Stachowiak.
+
+        * platform/mac/PlatformScreenMac.mm:
+        (WebCore::setShouldOverrideScreenSupportsHighDynamicRange):
+
 2020-02-13  Kate Cheney  <katherine_cheney@apple.com>
 

trunk/Source/WebCore/platform/mac/PlatformScreenMac.mm

@@ -159 +159 @@
 void setShouldOverrideScreenSupportsHighDynamicRange(bool shouldOverride, bool supportsHighDynamicRange)
 {
-    if (PAL::canLoad_MediaToolbox_MTOverrideShouldPlayHDRVideo())
+    if (PAL::isMediaToolboxFrameworkAvailable() && PAL::canLoad_MediaToolbox_MTOverrideShouldPlayHDRVideo())
         PAL::softLink_MediaToolbox_MTOverrideShouldPlayHDRVideo(shouldOverride, supportsHighDynamicRange);
 }