Re: [blink-dev] Intent to Prototype and Ship: MessagePort.onclose

2023-11-28 Thread Yoav Weiss
I ran into a Chromium-internal case where having a `close` signal would've helped me reduce complexity significantly. So I'm very excited about this work. I also believe there's an equivalent mechanism in Electron. Might be worthwhile

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

[blink-dev] Re: Intent to Prototype: Extending Storage Access API (SAA) to non-cookie storage

2023-11-28 Thread Jesper van den Ende
Hi all, I noticed the TODOs for shared workers were removed. My application relies on cross origin shared workers and to be honest this was the main part of the proposal I was looking forward to haha. I'm currently relying on the origin trial for disabling third party storage partitioning. Are t

[blink-dev] Re: Intent to Prototype: Extending Storage Access API (SAA) to non-cookie storage

2023-11-28 Thread Jesper van den Ende
Awesome, thanks for the info! A broadcast channel doesn't work for my use case unfortunately. I'd like to create a MessageChannel and then transfer the two MessagePorts to both tabs. Since a broadcast channel doesn't allow transfe

[blink-dev] Re: Intent to Prototype: Extending Storage Access API (SAA) to non-cookie storage

2023-11-28 Thread Ari Chivukula
Testing locally, event.source seems always to be null when using Broadcast Channel. This seems to align with the spec which doesn't mention setting source at all for the MessageEvent: https://html.spec.whatwg.org/multipage/web-messaging.html#broadcasting-to-other-browsing-contexts and the chrome co

[blink-dev] Re: pylint 2.17 available in depot_tools

2023-11-28 Thread 'Josip Sokcevic' via infra-announce
Hi all, We identified pylint 2.13 incompatibilities with py3.11 ( https://crbug.com/1504663 has more details). So, pylint 2.13 was removed and pylint 2.17 was added. To use pylint 2.17 in presubmit, pass version='2.17' to input_api.canned_checks.RunPylint or input_api.canned_checks.GetPylint. Ap

[blink-dev] Re: Intent to Prototype: Extending Storage Access API (SAA) to non-cookie storage

2023-11-28 Thread Ari Chivukula
Thanks for letting us know about your use case. Yes, Chrome has decided not to pursue Shared Worker access via the handle returned by requestStorageAccess given it would allow access to SameSite=Strict cookies to a third-party iframe via that shared worker. We are examining an alternate option wher

Re: [blink-dev] Intent to Prototype and Ship: MessagePort.onclose

2023-11-28 Thread Torne (Richard Coles)
Android WebView exposes the ability to create and use MessagePorts to the host app through a Java API, and currently one of the limitations of this API is the lack of a signal about the channel being closed, which in some cases can lead to apps holding on to Java objects longer than intended. So,

[blink-dev] Re: Intent to Ship: Storage Buckets API

2023-11-28 Thread Evan Stade
It depends on the use case. Sometimes developers may be OK with missing functionality. For example, if some data such as downloaded media should be persisted whereas other data such as logs should be evictable, then not having buckets available probably just means that the site does not play as nic

[blink-dev] Intent to Prototype and Ship: MessagePort.onclose

2023-11-28 Thread Nonoka Muraki
Contact emails murakinon...@chromium.org, m...@chromium.org, rak...@chromium.org Explainer https://github.com/fergald/explainer-messageport-close Specification https://github.com/whatwg/

Re: [blink-dev] Intent to Ship: CSS font-palette property animation

2023-11-28 Thread 'Munira Tursunova' via blink-dev
Thank you for LGTMs. FYI, TAG review was positive https://github.com/w3ctag/design-reviews/issues/915#issuecomment-1828305940 . On Tuesday, November 28, 2023 at 7:53:33 AM UTC+1 rby...@chromium.org wrote: > LGTM3 > > On Sun, Nov 26, 2023 at 3