[webkit-changes] [WebKit/WebKit] 5f7add: RealtimeMediaSource error logging is wrong for zoo...

youennf noreply at github.com
Tue Feb 20 05:46:20 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/WebKit/WebKit
  Commit: 5f7adde53ff63535d548a3c038b2a84b89afde3a
      https://github.com/WebKit/WebKit/commit/5f7adde53ff63535d548a3c038b2a84b89afde3a
  Author: Youenn Fablet <youennf at gmail.com>
  Date:   2024-02-20 (Tue, 20 Feb 2024)

  Changed paths:
    A LayoutTests/fast/mediastream/applyConstraints-bad-constraints-expected.txt
    A LayoutTests/fast/mediastream/applyConstraints-bad-constraints.html
    M Source/WebCore/platform/mediastream/RealtimeMediaSource.cpp
    M Source/WebCore/platform/mediastream/RealtimeMediaSourceCapabilities.h

  Log Message:
  -----------
  RealtimeMediaSource error logging is wrong for zoom and frame rate constraints
rdar://121949844
https://bugs.webkit.org/show_bug.cgi?id=269700

Reviewed by Eric Carlson.

Debug ASSERTs were changed into RELEASE_ASSERT in https://github.com/WebKit/WebKit/commit/f685afa11acafdf44fd60af91b837cb1f3bf85f9.
But these asserts are hit since our loggings are mistakenly calling longRange() for double constraints.
Update our logging code to use double constraints and add corresponding tests.
We downgrade the two RELEASE_ASSERT to ASSERT like it was the case before https://github.com/WebKit/WebKit/commit/f685afa11acafdf44fd60af91b837cb1f3bf85f9.
A future patch should further refactor this code to remove these ASSERTS.

* LayoutTests/fast/mediastream/applyConstraints-bad-constraints-expected.txt: Added.
* LayoutTests/fast/mediastream/applyConstraints-bad-constraints.html: Added.
* Source/WebCore/platform/mediastream/RealtimeMediaSource.cpp:
(WebCore::RealtimeMediaSource::hasInvalidSizeFrameRateAndZoomConstraints):

Canonical link: https://commits.webkit.org/275036@main



To unsubscribe from these emails, change your notification settings at https://github.com/WebKit/WebKit/settings/notifications


More information about the webkit-changes mailing list