Attachment 360606 Details for Bug 194039 – ROLLOUT of r240665

[patch] ROLLOUT of r240665

bug-194039-20190130132508.patch (text/plain), 14.86 KB, created by WebKit Commit Bot on 2019-01-30 13:25:09 PST

(hide)

Description:

Filename:

MIME Type:

Creator: WebKit Commit Bot

Created: 2019-01-30 13:25:09 PST

Size: 14.86 KB

patch

obsolete

>Subversion Revision: 240727
>diff --git a/Source/ThirdParty/libwebrtc/ChangeLog b/Source/ThirdParty/libwebrtc/ChangeLog
>index 9bccbbe3dcd08fb7583426fc797cb33035ccc6e5..3f4f1c88cfa5991a599ba623009ca144062b6034 100644
>--- a/Source/ThirdParty/libwebrtc/ChangeLog
>+++ b/Source/ThirdParty/libwebrtc/ChangeLog
>@@ -1,3 +1,17 @@
>+2019-01-30  Commit Queue  <commit-queue@webkit.org>
>+
>+        Unreviewed, rolling out r240665.
>+        https://bugs.webkit.org/show_bug.cgi?id=194039
>+
>+        "Better to postpone SPI adoption" (Requested by youenn on
>+        #webkit).
>+
>+        Reverted changeset:
>+
>+        "Adopt new VCP SPI"
>+        https://bugs.webkit.org/show_bug.cgi?id=193357
>+        https://trac.webkit.org/changeset/240665
>+
> 2019-01-29  Youenn Fablet  <youenn@apple.com>
> 
>         Adopt new VCP SPI
>diff --git a/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/EncoderUtilities.h b/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/EncoderUtilities.h
>index 25f56e764191fdc2cf9cb8a0d577658fb19284b9..5b9bdfda1146ec4a203fe27ae14d158c01c9dcf9 100644
>--- a/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/EncoderUtilities.h
>+++ b/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/EncoderUtilities.h
>@@ -30,7 +30,6 @@
> #if ENABLE_VCP_ENCODER
> 
> #define CompressionSessionRef VCPCompressionSessionRef
>-
> #define CompressionSessionSetProperty webrtc::VCPCompressionSessionSetProperty
> #define CompressionSessionGetPixelBufferPool webrtc::VCPCompressionSessionGetPixelBufferPool
> #define CompressionSessionEncodeFrame webrtc::VCPCompressionSessionEncodeFrame
>@@ -38,23 +37,9 @@
> #define kCodecTypeH264 kVCPCodecType4CC_H264
> #define CompressionSessionInvalidate webrtc::VCPCompressionSessionInvalidate
> 
>-#elif ENABLE_VCP_VTB_ENCODER
>-
>-bool isSupportingH264RTVC();
>-
>-#define CompressionSessionRef VCPCompressionSessionRef
>-
>-#define CompressionSessionSetProperty(compressionSession, ...) (isSupportingH264RTVC() ? VTSessionSetProperty(reinterpret_cast<VTCompressionSessionRef>(compressionSession), __VA_ARGS__) : webrtc::VCPCompressionSessionSetProperty(compressionSession, __VA_ARGS__))
>-#define CompressionSessionGetPixelBufferPool(compressionSession, ...) (isSupportingH264RTVC() ? VTCompressionSessionGetPixelBufferPool(reinterpret_cast<VTCompressionSessionRef>(compressionSession), __VA_ARGS__) : webrtc::VCPCompressionSessionSetProperty(compressionSession, __VA_ARGS__))
>-#define CompressionSessionEncodeFrame(compressionSession, ...) (isSupportingH264RTVC() ? VTCompressionSessionEncodeFrame(reinterpret_cast<VTCompressionSessionRef>(compressionSession), __VA_ARGS__) : webrtc::VCPCompressionSessionEncodeFrame(compressionSession, __VA_ARGS__))
>-#define CompressionSessionCreate(a, b, c, d, e, f, g, h, i, compressionSession) (isSupportingH264RTVC() ? VTCompressionSessionCreate(a, b, c, d, e, f, g, h, i, reinterpret_cast<VTCompressionSessionRef*>(compressionSession)) : webrtc::VCPCompressionSessionCreate(a, b, c, d, e, f, g, h, i, compressionSession))
>-#define kCodecTypeH264 (isSupportingH264RTVC() ? kCMVideoCodecType_H264 : kVCPCodecType4CC_H264)
>-#define CompressionSessionInvalidate(compressionSession) (isSupportingH264RTVC() ? VTCompressionSessionInvalidate(reinterpret_cast<VTCompressionSessionRef>(compressionSession)) : webrtc::VCPCompressionSessionInvalidate(compressionSession))
>-
> #else
> 
> #define CompressionSessionRef VTCompressionSessionRef
>-
> #define CompressionSessionSetProperty VTSessionSetProperty
> #define CompressionSessionGetPixelBufferPool VTCompressionSessionGetPixelBufferPool
> #define CompressionSessionEncodeFrame VTCompressionSessionEncodeFrame
>diff --git a/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/VideoProcessingSoftLink.cpp b/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/VideoProcessingSoftLink.cpp
>index 2e4be53a5b1fcdf1ebc8047ac54c33b6e76f869e..101e87c66777044a1ea557d2a5860e50a9f1336a 100644
>--- a/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/VideoProcessingSoftLink.cpp
>+++ b/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/VideoProcessingSoftLink.cpp
>@@ -25,7 +25,7 @@
> 
> #include "VideoProcessingSoftLink.h"
> 
>-#if ENABLE_VCP_ENCODER || ENABLE_VCP_VTB_ENCODER
>+#if ENABLE_VCP_ENCODER
> 
> #include "rtc_base/logging.h"
> #import <dlfcn.h>
>@@ -67,12 +67,12 @@
> }
> 
> SOFT_LINK_PRIVATE_FRAMEWORK_FOR_SOURCE(webrtc, VideoProcessing)
>-SOFT_LINK_FUNCTION_FOR_SOURCE(webrtc, VideoProcessing, VPModuleInitialize, void, (), ())
> 
> SOFT_LINK_FUNCTION_FOR_SOURCE(webrtc, VideoProcessing, VCPCompressionSessionSetProperty, OSStatus, (VCPCompressionSessionRef session, CFStringRef key, CFTypeRef value), (session, key, value))
> SOFT_LINK_FUNCTION_FOR_SOURCE(webrtc, VideoProcessing, VCPCompressionSessionGetPixelBufferPool, CVPixelBufferPoolRef, (VCPCompressionSessionRef session), (session))
> SOFT_LINK_FUNCTION_FOR_SOURCE(webrtc, VideoProcessing, VCPCompressionSessionEncodeFrame, OSStatus, (VCPCompressionSessionRef session, CVImageBufferRef buffer, CMTime timestamp, CMTime time, CFDictionaryRef dictionary, void* data, VTEncodeInfoFlags* flags), (session, buffer, timestamp, time, dictionary, data, flags))
> SOFT_LINK_FUNCTION_FOR_SOURCE(webrtc, VideoProcessing, VCPCompressionSessionCreate, OSStatus, (CFAllocatorRef allocator1, int32_t value1 , int32_t value2, CMVideoCodecType type, CFDictionaryRef dictionary1, CFDictionaryRef dictionary2, CFAllocatorRef allocator3, VTCompressionOutputCallback callback, void* data, VCPCompressionSessionRef* session), (allocator1, value1, value2, type, dictionary1, dictionary2, allocator3, callback, data, session))
> SOFT_LINK_FUNCTION_FOR_SOURCE(webrtc, VideoProcessing, VCPCompressionSessionInvalidate, void, (VCPCompressionSessionRef session), (session))
>+SOFT_LINK_FUNCTION_FOR_SOURCE(webrtc, VideoProcessing, VPModuleInitialize, void, (), ())
> 
>-#endif // ENABLE_VCP_ENCODER || ENABLE_VCP_VTB_ENCODER
>+#endif // ENABLE_VCP_ENCODER
>diff --git a/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/VideoProcessingSoftLink.h b/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/VideoProcessingSoftLink.h
>index 7c2225b0a7094218a554ff5a7753c7fa16ae7460..cf866d8a8db54a204919df9992fb2c49be6c30fb 100644
>--- a/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/VideoProcessingSoftLink.h
>+++ b/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/VideoProcessingSoftLink.h
>@@ -35,13 +35,10 @@
> 
> #if (defined(TARGET_IPHONE_SIMULATOR)  && TARGET_IPHONE_SIMULATOR)
> #define ENABLE_VCP_ENCODER 0
>-#define ENABLE_VCP_VTB_ENCODER 0
> #elif (defined(TARGET_OS_IPHONE)  && TARGET_OS_IPHONE)
>-#define ENABLE_VCP_ENCODER __MAC_OS_X_VERSION_MAX_ALLOWED < 101500
>-#define ENABLE_VCP_VTB_ENCODER __MAC_OS_X_VERSION_MIN_REQUIRED >= 101500
>+#define ENABLE_VCP_ENCODER 1
> #elif (defined(TARGET_OS_MAC) && TARGET_OS_MAC)
>-#define ENABLE_VCP_ENCODER (__MAC_OS_X_VERSION_MIN_REQUIRED >= 101300 && __MAC_OS_X_VERSION_MAX_ALLOWED < 101500)
>-#define ENABLE_VCP_VTB_ENCODER __MAC_OS_X_VERSION_MIN_REQUIRED >= 101500
>+#define ENABLE_VCP_ENCODER (__MAC_OS_X_VERSION_MIN_REQUIRED >= 101300 && __MAC_OS_X_VERSION_MAX_ALLOWED >= 101304)
> #endif
> 
> #endif
>@@ -50,10 +47,6 @@
> #define ENABLE_VCP_ENCODER 0
> #endif
> 
>-#if !defined(ENABLE_VCP_VTB_ENCODER)
>-#define ENABLE_VCP_VTB_ENCODER 0
>-#endif
>-
> #if !defined(ALWAYS_INLINE)
> #define ALWAYS_INLINE inline
> #endif
>@@ -118,15 +111,12 @@
>         return pointer##name; \
>     }
> 
>-#if ENABLE_VCP_ENCODER || ENABLE_VCP_VTB_ENCODER
>+#if ENABLE_VCP_ENCODER
> 
> #include <VideoProcessing/VideoProcessing.h>
> 
> SOFT_LINK_FRAMEWORK_FOR_HEADER(webrtc, VideoProcessing)
> 
>-SOFT_LINK_FUNCTION_FOR_HEADER(webrtc, VideoProcessing, VPModuleInitialize, void, (), ())
>-#define VPModuleInitialize softLink_VideoProcessing_VPModuleInitialize
>-
> SOFT_LINK_FUNCTION_FOR_HEADER(webrtc, VideoProcessing, VCPCompressionSessionSetProperty, OSStatus, (VCPCompressionSessionRef session, CFStringRef key, CFTypeRef value), (session, key, value))
> #define VCPCompressionSessionSetProperty softLink_VideoProcessing_VCPCompressionSessionSetProperty
> 
>@@ -142,6 +132,9 @@ SOFT_LINK_FUNCTION_FOR_HEADER(webrtc, VideoProcessing, VCPCompressionSessionCrea
> SOFT_LINK_FUNCTION_FOR_HEADER(webrtc, VideoProcessing, VCPCompressionSessionInvalidate, void, (VCPCompressionSessionRef session), (session))
> #define VCPCompressionSessionInvalidate softLink_VideoProcessing_VCPCompressionSessionInvalidate
> 
>-#endif // ENABLE_VCP_ENCODER || ENABLE_VCP_VTB_ENCODER
>+SOFT_LINK_FUNCTION_FOR_HEADER(webrtc, VideoProcessing, VPModuleInitialize, void, (), ())
>+#define VPModuleInitialize softLink_VideoProcessing_VPModuleInitialize
>+
>+#endif // ENABLE_VCP_ENCODER
> 
> #endif // __APPLE__
>diff --git a/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/WebKitUtilities.mm b/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/WebKitUtilities.mm
>index ace8a89261f49133ee41dddf25c69956a8c39a54..f6cebf74e042ae768f3f8a3a86daf92efd9ca4f6 100644
>--- a/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/WebKitUtilities.mm
>+++ b/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/WebKit/WebKitUtilities.mm
>@@ -105,7 +105,7 @@ void setApplicationStatus(bool isActive)
> 
> std::unique_ptr<webrtc::VideoEncoderFactory> createWebKitEncoderFactory(WebKitCodecSupport codecSupport)
> {
>-#if ENABLE_VCP_ENCODER || ENABLE_VCP_VTB_ENCODER
>+#if ENABLE_VCP_ENCODER
>     static std::once_flag onceFlag;
>     std::call_once(onceFlag, [] {
>         webrtc::VPModuleInitialize();
>diff --git a/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/objc/components/video_codec/RTCVideoEncoderH264.mm b/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/objc/components/video_codec/RTCVideoEncoderH264.mm
>index 4499e1ccd959716054d2b57d27ca9f0a52109385..543f4fb5a4ba2e68764e54bd3a905b336aa1a571 100644
>--- a/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/objc/components/video_codec/RTCVideoEncoderH264.mm
>+++ b/Source/ThirdParty/libwebrtc/Source/webrtc/sdk/objc/components/video_codec/RTCVideoEncoderH264.mm
>@@ -40,72 +40,7 @@
> VT_EXPORT const CFStringRef kVTVideoEncoderSpecification_Usage;
> VT_EXPORT const CFStringRef kVTCompressionPropertyKey_Usage;
> 
>-#if ENABLE_VCP_VTB_ENCODER
>-
>-void testCompressionOutputCallback(void *, void *, OSStatus, VTEncodeInfoFlags, CMSampleBufferRef)
>-{
>-}
>-
>-static bool isSupportingH264RTVCValue = false;
>-bool isSupportingH264RTVC()
>-{
>-    static std::once_flag onceFlag;
>-    std::call_once(onceFlag, [] {
>-        const size_t attributesSize = 3;
>-        CFTypeRef keys[attributesSize] = {
>-#if defined(WEBRTC_IOS)
>-            kCVPixelBufferOpenGLESCompatibilityKey,
>-#elif defined(WEBRTC_MAC)
>-            kCVPixelBufferOpenGLCompatibilityKey,
>-#endif
>-            kCVPixelBufferIOSurfacePropertiesKey,
>-            kCVPixelBufferPixelFormatTypeKey
>-        };
>-        CFDictionaryRef ioSurfaceValue = CreateCFTypeDictionary(nullptr, nullptr, 0);
>-        int64_t pixelFormatType = kCVPixelFormatType_420YpCbCr8BiPlanarFullRange;
>-        CFNumberRef pixelFormat = CFNumberCreate(nullptr, kCFNumberLongType, &pixelFormatType);
>-        CFTypeRef values[attributesSize] = {kCFBooleanTrue, ioSurfaceValue, pixelFormat};
>-        CFDictionaryRef sourceAttributes = CreateCFTypeDictionary(keys, values, attributesSize);
>-
>-        CFRelease(ioSurfaceValue);
>-        CFRelease(pixelFormat);
>-
>-        CFMutableDictionaryRef encoderSpecs = CFDictionaryCreateMutable(nullptr, 5, &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks);
>-#if !defined(WEBRTC_IOS)
>-        auto useHardwareEncoder = webrtc::isH264HardwareEncoderAllowed() ? kCFBooleanTrue : kCFBooleanFalse;
>-        // Currently hw accl is supported above 360p on mac, below 360p
>-        // the compression session will be created with hw accl disabled.
>-        CFDictionarySetValue(encoderSpecs, kVTVideoEncoderSpecification_EnableHardwareAcceleratedVideoEncoder, useHardwareEncoder);
>-        CFDictionarySetValue(encoderSpecs, kVTVideoEncoderSpecification_RequireHardwareAcceleratedVideoEncoder, useHardwareEncoder);
>-#endif
>-        CFDictionarySetValue(encoderSpecs, kVTCompressionPropertyKey_RealTime, kCFBooleanTrue);
>-
>-        int usageValue = 1;
>-        auto usage = CFNumberCreate(nullptr, kCFNumberIntType, &usageValue);
>-        CFDictionarySetValue(encoderSpecs, kVTCompressionPropertyKey_Usage, usage);
>-        CFRelease(usage);
>-
>-        CFDictionarySetValue(encoderSpecs, kVTVideoEncoderList_EncoderID, CFSTR("com.apple.videotoolbox.videoencoder.h264.rtvc"));
>-
>-        CompressionSessionRef compressionSession = nullptr;
>-        OSStatus status = VTCompressionSessionCreate(nullptr, 320, 240, kCMVideoCodecType_H264,
>-            encoderSpecs, sourceAttributes, nullptr, testCompressionOutputCallback, nullptr, reinterpret_cast<VTCompressionSessionRef*>(&compressionSession));
>-
>-        if (compressionSession) {
>-            CompressionSessionInvalidate(compressionSession);
>-            CFRelease(compressionSession);
>-        }
>-
>-        CFRelease(sourceAttributes);
>-        CFRelease(encoderSpecs);
>-
>-        isSupportingH264RTVCValue = status == noErr;
>-    });
>-    return isSupportingH264RTVCValue;
>-}
>-#endif
>-
>-#if !ENABLE_VCP_ENCODER && !defined(WEBRTC_IOS) && !ENABLE_VCP_VTB_ENCODER
>+#if !ENABLE_VCP_ENCODER && !defined(WEBRTC_IOS)
> static inline bool isStandardFrameSize(int32_t width, int32_t height)
> {
>     // FIXME: Envision relaxing this rule, something like width and height dividable by 4 or 8 should be good enough.
>@@ -673,7 +608,7 @@ CFStringRef ExtractProfile(webrtc::SdpVideoFormat videoFormat) {
>     CFRelease(pixelFormat);
>     pixelFormat = nullptr;
>   }
>-  CFMutableDictionaryRef encoderSpecs = CFDictionaryCreateMutable(nullptr, 5, &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks);
>+  CFMutableDictionaryRef encoderSpecs = CFDictionaryCreateMutable(nullptr, 4, &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks);
> #if !defined(WEBRTC_IOS)
>   auto useHardwareEncoder = webrtc::isH264HardwareEncoderAllowed() ? kCFBooleanTrue : kCFBooleanFalse;
>   // Currently hw accl is supported above 360p on mac, below 360p
>@@ -683,15 +618,11 @@ CFStringRef ExtractProfile(webrtc::SdpVideoFormat videoFormat) {
> #endif
>   CFDictionarySetValue(encoderSpecs, kVTCompressionPropertyKey_RealTime, kCFBooleanTrue);
> 
>-#if ENABLE_VCP_ENCODER || ENABLE_VCP_VTB_ENCODER
>+#if ENABLE_VCP_ENCODER
>   int usageValue = 1;
>   auto usage = CFNumberCreate(nullptr, kCFNumberIntType, &usageValue);
>   CFDictionarySetValue(encoderSpecs, kVTCompressionPropertyKey_Usage, usage);
>   CFRelease(usage);
>-#endif
>-#if ENABLE_VCP_VTB_ENCODER
>-    if (isSupportingH264RTVC())
>-        CFDictionarySetValue(encoderSpecs, kVTVideoEncoderList_EncoderID, CFSTR("com.apple.videotoolbox.videoencoder.h264.rtvc"));
> #endif
>   OSStatus status =
>       CompressionSessionCreate(nullptr,  // use default allocator
>@@ -732,7 +663,7 @@ CFStringRef ExtractProfile(webrtc::SdpVideoFormat videoFormat) {
>   } else {
>     RTC_LOG(LS_INFO) << "Compression session created with hw accl disabled";
> 
>-#if !ENABLE_VCP_ENCODER && !ENABLE_VCP_VTB_ENCODER && !defined(WEBRTC_IOS)
>+#if !ENABLE_VCP_ENCODER && !defined(WEBRTC_IOS)
>     if (!isStandardFrameSize(_width, _height)) {
>       _disableEncoding = true;
>       RTC_LOG(LS_ERROR) << "Using H264 software encoder with non standard size is not supported";

Actions: View | Formatted Diff | Diff

Attachments on bug 194039: 360606