LGTM2

On Wed, Aug 27, 2025, 9:52 AM Daniel Bratell <bratel...@gmail.com> wrote:

> LGTM1
>
> Regardless of how you handle those tests for now, I assume this is just
> the first step and once the spec is complete, it will all be good.
>
> /Daniel
> On 2025-08-25 18:45, Guido Urdaneta wrote:
>
> Since those tests are included in Interop 2025 I believe we cannot
> unilaterally make them tentative. However, it is likely that some parts of
> the generateKeyFrame WPTs will change after the upcoming discussion.
> Since we are leaving generateKeyFrame out of this launch I think the
> tentative status of those tests should not affect this I2S.
>
> Note also that Chromium already supports the functionality of generating
> key frames via the setParameters API (see
> https://w3c.github.io/webrtc-extensions/#dom-rtcencodingoptions, I2S:
> https://groups.google.com/a/chromium.org/g/blink-dev/c/pd3Hksi3jq0). This
> is a better API for this functionality that has the advantage of being
> decoupled from Encoded Transform.
>
>
>
> On Mon, Aug 25, 2025 at 10:52 AM Yoav Weiss (@Shopify) <
> yoavwe...@chromium.org> wrote:
>
>> Can the failing tests be renamed to `.tentative` until those discussions
>> settle?
>>
>> On Thu, Aug 21, 2025 at 7:42 PM Guido Urdaneta <gui...@chromium.org>
>> wrote:
>>
>>> The multi-browser failures in wpt.fyi are mainly due to their including
>>> features that are still under discussion (SFrame and generateKeyFrame).
>>> SFrame has not been shipped by any browser yet (it is not covered in the
>>> Interop 2025 effort). Safari and Firefox do have generateKeyFrame, but they
>>> are not completely compatible and there are open issues that are being
>>> discussed in the W3C WebRTC WG. We are not including these features in this
>>> launch.
>>> The generateKeyFrame issues are expected to be resolved after the WG
>>> discussion, and we plan to ship it shortly after.
>>>
>>> The results of Chrome and Edge will improve with this launch, in
>>> particular script-metadata-transform.https.html
>>> <https://wpt.fyi/results/webrtc-encoded-transform/script-metadata-transform.https.html?label=experimental&label=master&aligned>
>>> , script-transform-sendKeyFrameRequest.https.html
>>> <https://wpt.fyi/results/webrtc-encoded-transform/script-transform-sendKeyFrameRequest.https.html?label=experimental&label=master&aligned>
>>>  and
>>> RTCRtpScriptTransform-encoded-transform.https.html
>>> <https://wpt.live/webrtc-encoded-transform/RTCRtpScriptTransform-encoded-transform.https.html>
>>> will pass fully, which will put them on par with Firefox and Safari.
>>>
>>>
>>> On Wed, Aug 20, 2025 at 5:05 PM Philip Jägenstedt <foo...@chromium.org>
>>> wrote:
>>>
>>>> Very happy to see us approaching interoperability on this feature!
>>>>
>>>> Looking at
>>>> https://wpt.fyi/results/webrtc-encoded-transform?label=experimental&label=master&aligned
>>>> there are quite a lot of failures in all browsers. Does wpt.fyi reflect
>>>> the test results we'll have after enabling this, and are there a problem
>>>> with the test suite that needs to be addressed here?
>>>>
>>>> On Wed, Aug 13, 2025 at 7:21 PM Chromestatus <
>>>> ad...@cr-status.appspotmail.com> wrote:
>>>>
>>>>> Contact emails gui...@chromium.org, h...@chromium.org,
>>>>> topr...@chromium.org, alonsomo...@google.com, gui...@google.com
>>>>>
>>>>> Explainer None
>>>>>
>>>>> Specification https://github.com/w3c/webrtc-encoded-transform
>>>>>
>>>>> Summary
>>>>>
>>>>> This API allows processing of encoded media flowing through an
>>>>> RTCPeerConnection. Chromium shipped an early version of this API in 2020.
>>>>> Since then, the spec has changed and other browsers have shipped the
>>>>> updated version of the spec (Safari in 2022 and Firefox in 2023). This
>>>>> launch refers to the latest spec version and is part of Interop 2025. This
>>>>> launch does not cover the generateKeyFrame method, which is still under
>>>>> discussion https://github.com/w3c/webrtc-encoded-transform/issues/143
>>>>> https://github.com/w3c/webrtc-encoded-transform/issues/271
>>>>> https://github.com/w3c/webrtc-encoded-transform/pull/269
>>>>>
>>>>>
>>>>> Blink component Blink>WebRTC>PeerConnection
>>>>> <https://issues.chromium.org/issues?q=customfield1222907:%22Blink%3EWebRTC%3EPeerConnection%22>
>>>>>
>>>>> TAG review None
>>>>>
>>>>> TAG review status Issues addressed
>>>>>
>>>>> Risks
>>>>>
>>>>>
>>>>> Interoperability and Compatibility
>>>>>
>>>>> Interoperability: This change improves interoperability since it
>>>>> implements an API already implemented by other browsers. Interoperability
>>>>> risks are relatively low and derive from slight differences in
>>>>> implementation. We have seen that WPTs don't have the same pass rate for
>>>>> all implementations, but we have observed that in most cases this is due 
>>>>> to
>>>>> the WPTs testing unspecified behavior. Compatibility: Existing 
>>>>> applications
>>>>> using the old version of the API will continue to work without changes.
>>>>> Existing applications using the new version of the API should start 
>>>>> working
>>>>> with Chromium.
>>>>>
>>>>>
>>>>> *Gecko*: Shipped/Shipping
>>>>>
>>>>> *WebKit*: Shipped/Shipping
>>>>>
>>>>> *Web developers*: Positive
>>>>>
>>>>> *Other signals*:
>>>>>
>>>>> WebView application risks
>>>>>
>>>>> Does this intent deprecate or change behavior of existing APIs, such
>>>>> that it has potentially high risk for Android WebView-based applications?
>>>>>
>>>>> N/A
>>>>>
>>>>>
>>>>> Debuggability
>>>>>
>>>>> None
>>>>>
>>>>>
>>>>> Will this feature be supported on all six Blink platforms (Windows,
>>>>> Mac, Linux, ChromeOS, Android, and Android WebView)? Yes
>>>>>
>>>>> Is this feature fully tested by web-platform-tests
>>>>> <https://chromium.googlesource.com/chromium/src/+/main/docs/testing/web_platform_tests.md>
>>>>> ? Yes
>>>>>
>>>>>
>>>>> https://wpt.fyi/results/webrtc-encoded-transform?label=experimental&label=master&aligned
>>>>>
>>>>>
>>>>> Flag name on about://flags RTCRtpScriptTransform
>>>>>
>>>>> Finch feature name RTCRtpScriptTransform
>>>>>
>>>>> Rollout plan Will ship enabled for all users
>>>>>
>>>>> Requires code in //chrome? False
>>>>>
>>>>> Tracking bug https://issues.chromium.org/issues/354881878
>>>>>
>>>>> Availability expectation Feature available on all major browsers.
>>>>>
>>>>> Adoption expectation Feature is used by specific partner(s) to
>>>>> provide functionality within 12 months of launch in Chrome
>>>>>
>>>>> Non-OSS dependencies
>>>>>
>>>>> Does the feature depend on any code or APIs outside the Chromium open
>>>>> source repository and its open-source dependencies to function?
>>>>> None
>>>>>
>>>>> Sample links
>>>>>
>>>>> https://webrtc.github.io/samples/src/content/insertable-streams/endtoend-encryption
>>>>>
>>>>> Estimated milestones
>>>>> Shipping on desktop 141
>>>>> Shipping on Android 141
>>>>> Shipping on WebView 141
>>>>>
>>>>> Anticipated spec changes
>>>>>
>>>>> Open questions about a feature may be a source of future web compat or
>>>>> interop issues. Please list open issues (e.g. links to known github issues
>>>>> in the project for the feature specification) whose resolution may
>>>>> introduce web compat/interop risk (e.g., changing to naming or structure 
>>>>> of
>>>>> the API in a non-backward-compatible way).
>>>>> None
>>>>>
>>>>> Link to entry on the Chrome Platform Status
>>>>> https://chromestatus.com/feature/5175278159265792?gate=5141653464285184
>>>>>
>>>>> This intent message was generated by Chrome Platform Status
>>>>> <https://chromestatus.com>.
>>>>> --
>>>>> You received this message because you are subscribed to the Google
>>>>> Groups "blink-dev" group.
>>>>> To unsubscribe from this group and stop receiving emails from it, send
>>>>> an email to blink-dev+unsubscr...@chromium.org.
>>>>> To view this discussion visit
>>>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/689cc98b.050a0220.b43f3.1226.GAE%40google.com
>>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/689cc98b.050a0220.b43f3.1226.GAE%40google.com?utm_medium=email&utm_source=footer>
>>>>> .
>>>>>
>>>> --
>>> You received this message because you are subscribed to the Google
>>> Groups "blink-dev" group.
>>> To unsubscribe from this group and stop receiving emails from it, send
>>> an email to blink-dev+unsubscr...@chromium.org.
>>> To view this discussion visit
>>> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CA%2BBuZxY0oMTxiSQMjZifiusK%3DL7yFd6BNggn6emi1Pxha%3DRjEQ%40mail.gmail.com
>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CA%2BBuZxY0oMTxiSQMjZifiusK%3DL7yFd6BNggn6emi1Pxha%3DRjEQ%40mail.gmail.com?utm_medium=email&utm_source=footer>
>>> .
>>>
>> --
> You received this message because you are subscribed to the Google Groups
> "blink-dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to blink-dev+unsubscr...@chromium.org.
> To view this discussion visit
> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CA%2BBuZxYeUckh5fv8S3DckUW6%2BRGiL38M9ZArp17frvYMJbLtaQ%40mail.gmail.com
> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CA%2BBuZxYeUckh5fv8S3DckUW6%2BRGiL38M9ZArp17frvYMJbLtaQ%40mail.gmail.com?utm_medium=email&utm_source=footer>
> .
>
> --
> You received this message because you are subscribed to the Google Groups
> "blink-dev" group.
> To unsubscribe from this group and stop receiving emails from it, send an
> email to blink-dev+unsubscr...@chromium.org.
> To view this discussion visit
> https://groups.google.com/a/chromium.org/d/msgid/blink-dev/d1dbe684-2f57-4d5c-8dc6-a7a70bdbcdf7%40gmail.com
> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/d1dbe684-2f57-4d5c-8dc6-a7a70bdbcdf7%40gmail.com?utm_medium=email&utm_source=footer>
> .
>

-- 
You received this message because you are subscribed to the Google Groups 
"blink-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to blink-dev+unsubscr...@chromium.org.
To view this discussion visit 
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw_kFcvKDfe9XRFoRuFOTLrtO_quFhqskmbcHqe%3DDHeR9Q%40mail.gmail.com.

Reply via email to