WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED FIXED
264905
Sync 'RTCRtpScriptTransform.idl' with WebIDL Specification
https://bugs.webkit.org/show_bug.cgi?id=264905
Summary
Sync 'RTCRtpScriptTransform.idl' with WebIDL Specification
Ahmad Saleem
Reported
2023-11-15 15:59:34 PST
Hi Team, While looking into browser specific failures in WebRTC, I noticed that we fail few 'idlharness' tests for 'RTCRtpScriptTransform' and while looking into web specification, I noticed two things, which in my local testing progress failing tests: Web-Spec:
https://www.w3.org/TR/webrtc-encoded-transform/#RTCRtpScriptTransformer-interfaces
WebKit Source: Source/WebCore/Modules/mediastream/RTCRtpScriptTransform.idl Current: ] interface RTCRtpScriptTransform : EventTarget { To: ] interface RTCRtpScriptTransform { and Current: [CallWith=CurrentGlobalObject] constructor(Worker worker, any options, optional sequence<object> transfer); To: [CallWith=CurrentGlobalObject] constructor(Worker worker, optional any options, optional sequence<object> transfer); ____ Just wanted to raise, so we can fix it. Only Firefox Nightly 121 passes them yet, so not adding 'BrowserCompat' tag. Thanks!
Attachments
Add attachment
proposed patch, testcase, etc.
Ahmad Saleem
Comment 1
2023-11-15 16:00:35 PST
Test Case:
https://wpt.fyi/results/webrtc-encoded-transform/idlharness.https.window.html?label=master&label=experimental&aligned
= Test Case Live Link:
https://wpt.live/webrtc-encoded-transform/idlharness.https.window.html
Radar WebKit Bug Importer
Comment 2
2023-11-22 16:00:14 PST
<
rdar://problem/118728503
>
youenn fablet
Comment 3
2023-11-30 22:59:54 PST
Pull request:
https://github.com/WebKit/WebKit/pull/21145
EWS
Comment 4
2023-12-01 06:34:12 PST
Committed
271380@main
(91ff5d82ae24): <
https://commits.webkit.org/271380@main
> Reviewed commits have been landed. Closing PR #21145 and removing active labels.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug