Re: [blink-dev] Intent to Implement and Ship: VideoEncoderConfig.contentHint

2023-11-28 Thread Mike Taylor
LGTM3 On 11/27/23 10:50 PM, Yoav Weiss wrote: LGTM2 On Tue, Nov 28, 2023 at 7:50 AM Rick Byers wrote: All the other gates are now approved. I see there's a larger TAG review open which includes this feature, but it sounds like

Re: [blink-dev] Intent to Implement and Ship: VideoEncoderConfig.contentHint

2023-11-27 Thread Yoav Weiss
LGTM2 On Tue, Nov 28, 2023 at 7:50 AM Rick Byers wrote: > All the other gates are now approved. I see there's a larger TAG review > open which includes > this feature, but it sounds like this generic (not codec-specific) feature > is the

Re: [blink-dev] Intent to Implement and Ship: VideoEncoderConfig.contentHint

2023-11-27 Thread Rick Byers
All the other gates are now approved. I see there's a larger TAG review open which includes this feature, but it sounds like this generic (not codec-specific) feature is the direction TAG is encouraging. I don't think we need to block this

Re: [blink-dev] Intent to Implement and Ship: VideoEncoderConfig.contentHint

2023-11-08 Thread Daniel Bratell
They seem to be in state "preparing" which is the stage before they are actually requested. The system is a bit new so maybe there is some confusing UI or bug, but they are not yet requested. jrobbins, do you know what might have gone awry? /Daniel On 2023-11-08 05:04, 'Eugene Zemtsov' via

Re: [blink-dev] Intent to Implement and Ship: VideoEncoderConfig.contentHint

2023-11-07 Thread 'Eugene Zemtsov' via blink-dev
Done for Privacy, Security, Enterprise, Debuggability and Testing. On Mon, Nov 6, 2023 at 7:19 AM Mike Taylor wrote: > Hi Eugene, > > Could you please request reviews for all the other review gates in your > chromestatus entry? > > thanks, > Mike > On 11/1/23 6:32 PM, 'Eugene Zemtsov' via

Re: [blink-dev] Intent to Implement and Ship: VideoEncoderConfig.contentHint

2023-11-06 Thread Mike Taylor
Hi Eugene, Could you please request reviews for all the other review gates in your chromestatus entry? thanks, Mike On 11/1/23 6:32 PM, 'Eugene Zemtsov' via blink-dev wrote: Contact emails ezemt...@google.com Specification

[blink-dev] Intent to Implement and Ship: VideoEncoderConfig.contentHint

2023-11-01 Thread 'Eugene Zemtsov' via blink-dev
Contact emailsezemt...@google.com Specification https://www.w3.org/TR/webcodecs/#dom-videoencoderconfig-contenthint Explainerhttps://gist.github.com/Djuffin/c3742404b7c53ada227849c8b2b76b4c Summary Adding a contentHint field to VideoEncoderConfig Content hint takes values that are already used