Title: [227094] trunk/Source/WebCore
Revision
227094
Author
[email protected]
Date
2018-01-17 16:04:49 -0800 (Wed, 17 Jan 2018)

Log Message

Use existing RGB colorspace instead of creating a new one
https://bugs.webkit.org/show_bug.cgi?id=181765
<rdar://problem/36595753>

Reviewed by Dean Jackson.

* platform/mediastream/mac/ScreenDisplayCaptureSourceMac.mm:
(WebCore::ScreenDisplayCaptureSourceMac::createDisplayStream): Use sRGBColorSpaceRef instead
of creating a new static colorspace.

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (227093 => 227094)


--- trunk/Source/WebCore/ChangeLog	2018-01-17 23:49:45 UTC (rev 227093)
+++ trunk/Source/WebCore/ChangeLog	2018-01-18 00:04:49 UTC (rev 227094)
@@ -1,3 +1,15 @@
+2018-01-17  Eric Carlson  <[email protected]>
+
+        Use existing RGB colorspace instead of creating a new one
+        https://bugs.webkit.org/show_bug.cgi?id=181765
+        <rdar://problem/36595753>
+
+        Reviewed by Dean Jackson.
+
+        * platform/mediastream/mac/ScreenDisplayCaptureSourceMac.mm:
+        (WebCore::ScreenDisplayCaptureSourceMac::createDisplayStream): Use sRGBColorSpaceRef instead
+        of creating a new static colorspace.
+
 2018-01-17  Matt Lewis  <[email protected]>
 
         Unreviewed, rolling out r227076.

Modified: trunk/Source/WebCore/platform/mediastream/mac/ScreenDisplayCaptureSourceMac.mm (227093 => 227094)


--- trunk/Source/WebCore/platform/mediastream/mac/ScreenDisplayCaptureSourceMac.mm	2018-01-17 23:49:45 UTC (rev 227093)
+++ trunk/Source/WebCore/platform/mediastream/mac/ScreenDisplayCaptureSourceMac.mm	2018-01-18 00:04:49 UTC (rev 227094)
@@ -149,7 +149,6 @@
         if (!m_captureQueue)
             m_captureQueue = adoptOSObject(dispatch_queue_create("ScreenDisplayCaptureSourceMac Capture Queue", DISPATCH_QUEUE_SERIAL));
 
-        static CGColorSpaceRef deviceRGBColorSpace = CGColorSpaceCreateDeviceRGB();
         double frameTime = 1 / frameRate();
         auto frameTimeCF = adoptCF(CFNumberCreate(nullptr,  kCFNumberDoubleType,  &frameTime));
         int depth = screenQueueMaximumLength;
@@ -163,7 +162,7 @@
         CFTypeRef values[] = {
             frameTimeCF.get(),
             depthCF.get(),
-            deviceRGBColorSpace,
+            sRGBColorSpaceRef(),
             kCFBooleanTrue,
         };
         auto streamOptions = adoptCF(CFDictionaryCreate(kCFAllocatorDefault, keys, values, WTF_ARRAY_LENGTH(keys), &kCFTypeDictionaryKeyCallBacks, &kCFTypeDictionaryValueCallBacks));
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to