Thanks for the doc. It sounds like the design evolved during the OT based on fedback from the WG, and at least one partner was satisfied with the shape and functionality.

LGTM1

On 5/14/24 8:05 AM, Guido Urdaneta wrote:
Here is a doc with feedback from the Origin Trial. There were two registrants reported with a large usage scale. We received feedback from one of them and will update the doc if/when we get feedback from the other one.

https://docs.google.com/document/d/1QSYbrlgE_6ZZag_VRd8Mn863Slb5-GLlJ_-X4WOiym0/edit?usp=sharing


On Wed, May 8, 2024 at 5:45 PM Alex Russell <slightly...@chromium.org> wrote:

    Hey Guido,

    This is a cool feature! The Milestones section shows that an OT
    was run; is there a summary someplace of what we learned from the OT?

    Best,

    Alex

    On Thursday, May 2, 2024 at 4:40:31 AM UTC-7 Guido Urdaneta wrote:


                Contact emails


                h...@chromium.org, gui...@chromium.org,
                agpa...@chromium.org


                Explainer


                
https://github.com/guidou/webrtc-extensions/blob/main/constructor-explainer.md


                Specification


                
https://w3c.github.io/webrtc-encoded-transform/#dom-rtcencodedvideoframe-constructor


                
https://w3c.github.io/webrtc-encoded-transform/#dom-rtcencodedaudioframe-constructor


                Summary


                Allow WebRTC Encoded Transform API to create encoded
                audio and video frames specifying custom metadata.
                This is achieved by introducing constructors for
                encoded frames that take the original frame and custom
                metadata as input. This supports use cases that
                involve manipulation of not only the payload of
                encoded video / audio frames but also its metadata.
                Some examples: * Changing the mime type of the frame
                if the transform changes the type of the payload *
                Forwarding of media to a new peer connection set up to
                use different metadata values * Altering the timestamp
                of a frame to introduce a delay
                Use cases:
                https://w3c.github.io/webrtc-nv-use-cases/#live-encoded-media
                https://w3c.github.io/webrtc-nv-use-cases/#stored-encoded-media
                https://w3c.github.io/webrtc-nv-use-cases/#auction
                Issue link:
                https://github.com/w3c/webrtc-nv-use-cases/issues/77



        This change has consensus in the WebRTC Working Group and has
        been merged into the WebRTC Encoded Transform spec.


                Blink component


                Blink>WebRTC
                
<https://bugs.chromium.org/p/chromium/issues/list?q=component:Blink%3EWebRTC>


                TAG review


                TAG review request for this specific change:
                https://github.com/w3ctag/design-reviews/issues/942


                The original version of the full spec was reviewed by
                TAG here:
                https://github.com/w3ctag/design-reviews/issues/531


                TAG review status


                Pending


                Chromium Trial Name


                RTCEncodedFrameSetMetadata


                Origin Trial documentation link


                
https://github.com/palak8669/webrtc-encoded-transform/blob/create-encoded-explainer/create-encoded-explainer.md


                WebFeature UseCounter name


                RTCEncodedFrameSetMetadata


                Risks




                Interoperability and Compatibility


                Interoperability risk: There is always the risk that
                other browsers will not implement this feature. This
                risk is mitigated by alignment across browser vendors
                in the W3C WebRTC Working Group around the spec.
                Compatibility risk: This is a new feature intended to
                support new use cases. It introduces no breaking
                changes, so we do not expect any compatibility issues.


                /Gecko/: No signal
                (https://github.com/mozilla/standards-positions/issues/1009)
                During WebRTC WG meetings, Mozilla has shown positive
                signals and agreed with merging the PR in the main
                spec. See the exchange in
                https://github.com/w3c/webrtc-encoded-transform/pull/223

                /WebKit/: Positive
                (https://github.com/WebKit/standards-positions/issues/334)

                Marked as "Invalid" in the position request because
                this is a small change and the position was addressed
                in the PR review. The comments from WebKit in the PR
                review and during WebRTC WG meetings are positive and
                they have agreed with merging the PR in the main spec.
                See the exchange in
                https://github.com/w3c/webrtc-encoded-transform/pull/223

                /Web developers/: Positive

                /Other signals/:


                Ergonomics


                This feature is an extension to WebRTC Encoded
                Transform, which itself is an extension to
                WebRTC/RTCPeerConnection.



                Activation


                No significant risks identified.



                Security


                No new security risks identified.



                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?

                No



                Debuggability


                N/A



                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/tentative/RTCEncodedAudioFrame-metadata.https.html?label=master&label=experimental&aligned
                
<https://wpt.fyi/results/webrtc-encoded-transform/tentative/RTCEncodedAudioFrame-metadata.https.html?label=master&label=experimental&aligned>
                
https://wpt.fyi/results/webrtc-encoded-transform/tentative/RTCEncodedVideoFrame-metadata.https.html?label=master&label=experimental&aligned
                
<https://wpt.fyi/results/webrtc-encoded-transform/tentative/RTCEncodedVideoFrame-metadata.https.html?label=master&label=experimental&aligned>




                Flag name on chrome://flags




                Finch feature name


                RTCEncodedFrameSetMetadata


                Non-finch justification


                Guarded by a Blink RuntimeEnabledFeature.



                Requires code in //chrome?


                False


                Tracking bug


                https://issues.chromium.org/issues/40248396


                Estimated milestones


                Shipping on desktop     126
                Origin trial desktop first      118
                Origin trial desktop last       126
                Origin trial extension 1 end milestone  126

                Shipping on Android     126
                OriginTrial Android last        126
                OriginTrial Android first       118

                Shipping on WebView     126
                OriginTrial webView last        126
                OriginTrial webView first       118



                Anticipated spec changes


                None for the changes covered by this intent.


                Link to entry on the Chrome Platform Status


                
https://chromestatus.com/feature/5116073827893248?gate=5122458632323072


                Links to previous Intent discussions


                Intent to prototype:
                
https://groups.google.com/a/chromium.org/g/blink-dev/c/x2ZACgXrqp0 Intent
                to Experiment:
                
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CA%2BBuZxazRts59rCgrOHm2yDKwpGkXqsd-_5Wkurxid34FknDiQ%40mail.gmail.com
                Intent to Extend Experiment 1:
                
https://groups.google.com/a/chromium.org/g/blink-dev/c/dA4TndGG4VQ


                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 on the web visit https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CA%2BBuZxa8OyREc7SGWbwSB9xPkwrbpwjkw%2Bhz%2BPJtPezDpOmu_w%40mail.gmail.com <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CA%2BBuZxa8OyREc7SGWbwSB9xPkwrbpwjkw%2Bhz%2BPJtPezDpOmu_w%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 on the web visit 
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/1c75f07c-75d7-49f7-89d4-d0df3f93b67b%40chromium.org.

Reply via email to