Title: [245948] trunk/Source/WebCore
- Revision
- 245948
- Author
- [email protected]
- Date
- 2019-05-30 21:57:06 -0700 (Thu, 30 May 2019)
Log Message
Fix AVVideoCaptureSource::setFrameRateWithPreset logging
https://bugs.webkit.org/show_bug.cgi?id=198392
Reviewed by Eric Carlson.
Move logging from setFrameRateWithPreset to setSessionSizeAndFrameRate which does the actual job.
This ensures to not log in case of preset being null.
No change of behavior.
* platform/mediastream/mac/AVVideoCaptureSource.mm:
(WebCore::AVVideoCaptureSource::setFrameRateWithPreset):
(WebCore::AVVideoCaptureSource::setSessionSizeAndFrameRate):
Modified Paths
Diff
Modified: trunk/Source/WebCore/ChangeLog (245947 => 245948)
--- trunk/Source/WebCore/ChangeLog 2019-05-31 04:46:39 UTC (rev 245947)
+++ trunk/Source/WebCore/ChangeLog 2019-05-31 04:57:06 UTC (rev 245948)
@@ -1,3 +1,18 @@
+2019-05-30 Youenn Fablet <[email protected]>
+
+ Fix AVVideoCaptureSource::setFrameRateWithPreset logging
+ https://bugs.webkit.org/show_bug.cgi?id=198392
+
+ Reviewed by Eric Carlson.
+
+ Move logging from setFrameRateWithPreset to setSessionSizeAndFrameRate which does the actual job.
+ This ensures to not log in case of preset being null.
+ No change of behavior.
+
+ * platform/mediastream/mac/AVVideoCaptureSource.mm:
+ (WebCore::AVVideoCaptureSource::setFrameRateWithPreset):
+ (WebCore::AVVideoCaptureSource::setSessionSizeAndFrameRate):
+
2019-05-30 Jer Noble <[email protected]>
Video playback in Safari should continue when CarPlay is plugged in
Modified: trunk/Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.mm (245947 => 245948)
--- trunk/Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.mm 2019-05-31 04:46:39 UTC (rev 245947)
+++ trunk/Source/WebCore/platform/mediastream/mac/AVVideoCaptureSource.mm 2019-05-31 04:57:06 UTC (rev 245948)
@@ -285,8 +285,6 @@
void AVVideoCaptureSource::setFrameRateWithPreset(double requestedFrameRate, RefPtr<VideoPreset> preset)
{
- ALWAYS_LOG_IF(loggerPtr(), LOGIDENTIFIER, SizeAndFrameRate { preset->size.width(), preset->size.height(), requestedFrameRate });
-
auto* avPreset = preset ? downcast<AVVideoPreset>(preset.get()) : nullptr;
m_currentPreset = avPreset;
m_currentFrameRate = requestedFrameRate;
@@ -303,6 +301,8 @@
if (!avPreset)
return;
+ ALWAYS_LOG_IF(loggerPtr(), LOGIDENTIFIER, SizeAndFrameRate { m_currentPreset->size.width(), m_currentPreset->size.height(), m_currentFrameRate });
+
ASSERT(avPreset->format);
NSError *error = nil;
_______________________________________________
webkit-changes mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-changes
