Title: [284358] trunk/Source/WebCore
Revision
284358
Author
[email protected]
Date
2021-10-18 02:39:43 -0700 (Mon, 18 Oct 2021)

Log Message

[MediaStream] Mock video source background not fully filled for custom video resolution
https://bugs.webkit.org/show_bug.cgi?id=231863

Patch by Philippe Normand <[email protected]> on 2021-10-18
Reviewed by Xabier Rodriguez-Calvar.

The video frame buffer matches the captureSize, so its background should do the same.
this->size() is the final size of the media source, applied to the frame buffer before
notification to observers.

* platform/mock/MockRealtimeVideoSource.cpp:
(WebCore::MockRealtimeVideoSource::generateFrame):

Modified Paths

Diff

Modified: trunk/Source/WebCore/ChangeLog (284357 => 284358)


--- trunk/Source/WebCore/ChangeLog	2021-10-18 08:42:23 UTC (rev 284357)
+++ trunk/Source/WebCore/ChangeLog	2021-10-18 09:39:43 UTC (rev 284358)
@@ -1,3 +1,17 @@
+2021-10-18  Philippe Normand  <[email protected]>
+
+        [MediaStream] Mock video source background not fully filled for custom video resolution
+        https://bugs.webkit.org/show_bug.cgi?id=231863
+
+        Reviewed by Xabier Rodriguez-Calvar.
+
+        The video frame buffer matches the captureSize, so its background should do the same.
+        this->size() is the final size of the media source, applied to the frame buffer before
+        notification to observers.
+
+        * platform/mock/MockRealtimeVideoSource.cpp:
+        (WebCore::MockRealtimeVideoSource::generateFrame):
+
 2021-10-17  Dean Jackson  <[email protected]>
 
         [WebXR] Stubs for WebXR Hand Input Module

Modified: trunk/Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp (284357 => 284358)


--- trunk/Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp	2021-10-18 08:42:23 UTC (rev 284357)
+++ trunk/Source/WebCore/platform/mock/MockRealtimeVideoSource.cpp	2021-10-18 09:39:43 UTC (rev 284358)
@@ -454,7 +454,7 @@
     GraphicsContext& context = buffer->context();
     GraphicsContextStateSaver stateSaver(context);
 
-    auto size = this->size();
+    auto size = this->captureSize();
     FloatRect frameRect(FloatPoint(), size);
 
     context.fillRect(FloatRect(FloatPoint(), size), m_fillColor);
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to