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 this generic (not
codec-specific) feature is the direction TAG is encouraging. I
don't think we need to block this little feature on the larger
discussion of AV1-specific options.

LGTM1



On Thu, Nov 9, 2023 at 12:34 AM Daniel Bratell
 wrote:

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 blink-dev wrote:

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 blink-dev wrote:



Contact emails

ezemt...@google.com


Specification

https://www.w3.org/TR/webcodecs/#dom-videoencoderconfig-contenthint


Explainer

https://gist.github.com/Djuffin/c3742404b7c53ada227849c8b2b76b4c


Summary

Adding a contentHint field to VideoEncoderConfig Content
hint takes values that are already used for
MediaStreamTrack: "motion", "text", "detail". This gives
web developers a way to communicate to VideoEncoder the
expected type video frames they intend to encode.


Blink component

Blink>Media>WebCodecs




TAG review

N/A



Interoperability and Compatibility



/Gecko/: No signal
(https://github.com/mozilla/standards-positions/issues/919)

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

/Web developers/: Positive RTC software vendors
(including Google Meet) are interested in using it for
screen sharing vs live cam scenarios.


Will this feature be supported on all six Blink
platforms (Windows, Mac, Linux, Chrome OS,
Android, and Android WebView)?

Yes


Is this feature fully tested by
web-platform-tests

?

Yes


https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/web_tests/external/wpt/webcodecs/video-encoder-content-hint.https.any.js



Finch feature name

WebCodecsContentHint


Requires code in //chrome?

False


Tracking bug

https://crbug.com/1493588


Estimated milestones

Shipping on desktop 121

Shipping on Android 121


Link to entry on the Chrome Platform Status

https://chromestatus.com/feature/5103493249761280

--
Thanks,
Eugene Zemtsov.
-- 
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/CAK8JDrGKnc0Jc_doyR9ZsGVfcnk8aNfNS_t%2BTNQ-KLL%3DW%3DiZdA%40mail.gmail.com

.




-- 
Thanks,

Eugene Zemtsov.
-- 
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/CAK8JDrHnO-en4d_%2BKcwRBCRWYeVEi8hzcnGUsSkb%3D3dNgh0%2Bdw%40mail.gmail.com

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 direction TAG is encouraging. I don't think we need to block this
> little feature on the larger discussion of AV1-specific options.
>
> LGTM1
>
>
>
> On Thu, Nov 9, 2023 at 12:34 AM Daniel Bratell 
> wrote:
>
>> 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 blink-dev wrote:
>>
>> 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 blink-dev wrote:
>>>
>>> Contact emails ezemt...@google.com
>>>
>>> Specification
>>> https://www.w3.org/TR/webcodecs/#dom-videoencoderconfig-contenthint
>>>
>>> Explainer
>>> https://gist.github.com/Djuffin/c3742404b7c53ada227849c8b2b76b4c
>>>
>>> Summary
>>>
>>> Adding a contentHint field to VideoEncoderConfig Content hint takes
>>> values that are already used for MediaStreamTrack: "motion", "text",
>>> "detail". This gives web developers a way to communicate to VideoEncoder
>>> the expected type video frames they intend to encode.
>>>
>>> Blink component Blink>Media>WebCodecs
>>> 
>>>
>>> TAG review N/A
>>>
>>>
>>>
>>> Interoperability and Compatibility
>>>
>>>
>>>
>>> *Gecko*: No signal (
>>> https://github.com/mozilla/standards-positions/issues/919)
>>>
>>> *WebKit*: Positive (
>>> https://github.com/WebKit/standards-positions/issues/274)
>>>
>>> *Web developers*: Positive RTC software vendors (including Google Meet)
>>> are interested in using it for screen sharing vs live cam scenarios.
>>>
>>>
>>> Will this feature be supported on all six Blink platforms (Windows, Mac,
>>> Linux, Chrome OS, Android, and Android WebView)? Yes
>>>
>>> Is this feature fully tested by web-platform-tests
>>> 
>>> ? Yes
>>>
>>>
>>> https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/web_tests/external/wpt/webcodecs/video-encoder-content-hint.https.any.js
>>>
>>>
>>> Finch feature name WebCodecsContentHint
>>>
>>> Requires code in //chrome? False
>>>
>>> Tracking bug https://crbug.com/1493588
>>>
>>> Estimated milestones
>>> Shipping on desktop 121
>>> Shipping on Android 121
>>> Link to entry on the Chrome Platform Status
>>> https://chromestatus.com/feature/5103493249761280
>>>
>>> --
>>> Thanks,
>>> Eugene Zemtsov.
>>> --
>>> 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/CAK8JDrGKnc0Jc_doyR9ZsGVfcnk8aNfNS_t%2BTNQ-KLL%3DW%3DiZdA%40mail.gmail.com
>>> 
>>> .
>>>
>>>
>>
>> --
>> Thanks,
>> Eugene Zemtsov.
>> --
>> 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/CAK8JDrHnO-en4d_%2BKcwRBCRWYeVEi8hzcnGUsSkb%3D3dNgh0%2Bdw%40mail.gmail.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/6ffa1b86-4f0e-4e07-afb3-f3ad20cb2ebe%40gmail.com
>> 
>> .
>>
> --
> You received this message because you are subscribed to the Google Groups
> "blink-dev" group.
> To unsubscribe from this group

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
little feature on the larger discussion of AV1-specific options.

LGTM1



On Thu, Nov 9, 2023 at 12:34 AM Daniel Bratell  wrote:

> 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 blink-dev wrote:
>
> 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 blink-dev wrote:
>>
>> Contact emails ezemt...@google.com
>>
>> Specification
>> https://www.w3.org/TR/webcodecs/#dom-videoencoderconfig-contenthint
>>
>> Explainer
>> https://gist.github.com/Djuffin/c3742404b7c53ada227849c8b2b76b4c
>>
>> Summary
>>
>> Adding a contentHint field to VideoEncoderConfig Content hint takes
>> values that are already used for MediaStreamTrack: "motion", "text",
>> "detail". This gives web developers a way to communicate to VideoEncoder
>> the expected type video frames they intend to encode.
>>
>> Blink component Blink>Media>WebCodecs
>> 
>>
>> TAG review N/A
>>
>>
>>
>> Interoperability and Compatibility
>>
>>
>>
>> *Gecko*: No signal (
>> https://github.com/mozilla/standards-positions/issues/919)
>>
>> *WebKit*: Positive (
>> https://github.com/WebKit/standards-positions/issues/274)
>>
>> *Web developers*: Positive RTC software vendors (including Google Meet)
>> are interested in using it for screen sharing vs live cam scenarios.
>>
>>
>> Will this feature be supported on all six Blink platforms (Windows, Mac,
>> Linux, Chrome OS, Android, and Android WebView)? Yes
>>
>> Is this feature fully tested by web-platform-tests
>> 
>> ? Yes
>>
>>
>> https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/web_tests/external/wpt/webcodecs/video-encoder-content-hint.https.any.js
>>
>>
>> Finch feature name WebCodecsContentHint
>>
>> Requires code in //chrome? False
>>
>> Tracking bug https://crbug.com/1493588
>>
>> Estimated milestones
>> Shipping on desktop 121
>> Shipping on Android 121
>> Link to entry on the Chrome Platform Status
>> https://chromestatus.com/feature/5103493249761280
>>
>> --
>> Thanks,
>> Eugene Zemtsov.
>> --
>> 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/CAK8JDrGKnc0Jc_doyR9ZsGVfcnk8aNfNS_t%2BTNQ-KLL%3DW%3DiZdA%40mail.gmail.com
>> 
>> .
>>
>>
>
> --
> Thanks,
> Eugene Zemtsov.
> --
> 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/CAK8JDrHnO-en4d_%2BKcwRBCRWYeVEi8hzcnGUsSkb%3D3dNgh0%2Bdw%40mail.gmail.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/6ffa1b86-4f0e-4e07-afb3-f3ad20cb2ebe%40gmail.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

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 blink-dev wrote:

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 blink-dev wrote:



Contact emails

ezemt...@google.com


Specification

https://www.w3.org/TR/webcodecs/#dom-videoencoderconfig-contenthint


Explainer

https://gist.github.com/Djuffin/c3742404b7c53ada227849c8b2b76b4c


Summary

Adding a contentHint field to VideoEncoderConfig Content hint
takes values that are already used for MediaStreamTrack:
"motion", "text", "detail". This gives web developers a way to
communicate to VideoEncoder the expected type video frames they
intend to encode.


Blink component

Blink>Media>WebCodecs




TAG review

N/A



Interoperability and Compatibility



/Gecko/: No signal
(https://github.com/mozilla/standards-positions/issues/919)

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

/Web developers/: Positive RTC software vendors (including Google
Meet) are interested in using it for screen sharing vs live cam
scenarios.


Will this feature be supported on all six Blink platforms
(Windows, Mac, Linux, Chrome OS, Android, and Android
WebView)?

Yes


Is this feature fully tested by web-platform-tests

?

Yes


https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/web_tests/external/wpt/webcodecs/video-encoder-content-hint.https.any.js



Finch feature name

WebCodecsContentHint


Requires code in //chrome?

False


Tracking bug

https://crbug.com/1493588


Estimated milestones

Shipping on desktop 121

Shipping on Android 121


Link to entry on the Chrome Platform Status

https://chromestatus.com/feature/5103493249761280

--
Thanks,
Eugene Zemtsov.
-- 
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/CAK8JDrGKnc0Jc_doyR9ZsGVfcnk8aNfNS_t%2BTNQ-KLL%3DW%3DiZdA%40mail.gmail.com

.




--
Thanks,
Eugene Zemtsov.
--
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/CAK8JDrHnO-en4d_%2BKcwRBCRWYeVEi8hzcnGUsSkb%3D3dNgh0%2Bdw%40mail.gmail.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/6ffa1b86-4f0e-4e07-afb3-f3ad20cb2ebe%40gmail.com.

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 blink-dev wrote:
>
> Contact emails ezemt...@google.com
>
> Specification
> https://www.w3.org/TR/webcodecs/#dom-videoencoderconfig-contenthint
>
> Explainer https://gist.github.com/Djuffin/c3742404b7c53ada227849c8b2b76b4c
>
> Summary
>
> Adding a contentHint field to VideoEncoderConfig Content hint takes values
> that are already used for MediaStreamTrack: "motion", "text", "detail".
> This gives web developers a way to communicate to VideoEncoder the expected
> type video frames they intend to encode.
>
> Blink component Blink>Media>WebCodecs
> 
>
> TAG review N/A
>
>
>
> Interoperability and Compatibility
>
>
>
> *Gecko*: No signal (
> https://github.com/mozilla/standards-positions/issues/919)
>
> *WebKit*: Positive (
> https://github.com/WebKit/standards-positions/issues/274)
>
> *Web developers*: Positive RTC software vendors (including Google Meet)
> are interested in using it for screen sharing vs live cam scenarios.
>
>
> Will this feature be supported on all six Blink platforms (Windows, Mac,
> Linux, Chrome OS, Android, and Android WebView)? Yes
>
> Is this feature fully tested by web-platform-tests
> 
> ? Yes
>
>
> https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/web_tests/external/wpt/webcodecs/video-encoder-content-hint.https.any.js
>
>
> Finch feature name WebCodecsContentHint
>
> Requires code in //chrome? False
>
> Tracking bug https://crbug.com/1493588
>
> Estimated milestones
> Shipping on desktop 121
> Shipping on Android 121
> Link to entry on the Chrome Platform Status
> https://chromestatus.com/feature/5103493249761280
>
> --
> Thanks,
> Eugene Zemtsov.
> --
> 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/CAK8JDrGKnc0Jc_doyR9ZsGVfcnk8aNfNS_t%2BTNQ-KLL%3DW%3DiZdA%40mail.gmail.com
> 
> .
>
>

-- 
Thanks,
Eugene Zemtsov.

-- 
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/CAK8JDrHnO-en4d_%2BKcwRBCRWYeVEi8hzcnGUsSkb%3D3dNgh0%2Bdw%40mail.gmail.com.

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

https://www.w3.org/TR/webcodecs/#dom-videoencoderconfig-contenthint


Explainer

https://gist.github.com/Djuffin/c3742404b7c53ada227849c8b2b76b4c


Summary

Adding a contentHint field to VideoEncoderConfig Content hint takes 
values that are already used for MediaStreamTrack: "motion", "text", 
"detail". This gives web developers a way to communicate to 
VideoEncoder the expected type video frames they intend to encode.



Blink component

Blink>Media>WebCodecs 




TAG review

N/A



Interoperability and Compatibility



/Gecko/: No signal 
(https://github.com/mozilla/standards-positions/issues/919)


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


/Web developers/: Positive RTC software vendors (including Google 
Meet) are interested in using it for screen sharing vs live cam scenarios.



Will this feature be supported on all six Blink platforms
(Windows, Mac, Linux, Chrome OS, Android, and Android WebView)?

Yes


Is this feature fully tested by web-platform-tests

?

Yes

https://source.chromium.org/chromium/chromium/src/+/main:third_party/blink/web_tests/external/wpt/webcodecs/video-encoder-content-hint.https.any.js



Finch feature name

WebCodecsContentHint


Requires code in //chrome?

False


Tracking bug

https://crbug.com/1493588


Estimated milestones

Shipping on desktop 121

Shipping on Android 121


Link to entry on the Chrome Platform Status

https://chromestatus.com/feature/5103493249761280

--
Thanks,
Eugene Zemtsov.
--
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/CAK8JDrGKnc0Jc_doyR9ZsGVfcnk8aNfNS_t%2BTNQ-KLL%3DW%3DiZdA%40mail.gmail.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/eb09de19-f81d-4313-8dfc-9d3e1a3f7cd3%40chromium.org.