LGTM3

On Wed, Jul 19, 2023 at 8:54 AM Daniel Bratell <bratel...@gmail.com> wrote:

> LGTM2
>
> /Daniel
> On 2023-07-18 22:06, 'Rick Byers' via blink-dev wrote:
>
> Sounds good, thanks! I defer to your judgement on the future compat risk.
> Worst case "*" has to be redefined not to include "executionContexts", but
> that wouldn't be the ugliest wart in the web :-)
>
>  LGTM1
>
> On Tue, Jul 18, 2023 at 2:28 PM Ari Chivukula <aric...@chromium.org>
> wrote:
>
>> Replies inline.
>> ~ Ari Chivukula (Their/There/They're)
>>
>>
>> On Tue, Jul 18, 2023 at 1:59 PM Rick Byers <rby...@google.com> wrote:
>>
>>> Seems like a pretty tiny addition to an already shipped feature. Just
>>> one question on future compat:
>>>
>>> On Mon, Jul 17, 2023 at 7:34 AM Ari Chivukula <aric...@chromium.org>
>>> wrote:
>>>
>>>> Contact emails
>>>>
>>>> aric...@chromium.org, miketa...@chromium.org, yoavwe...@chromium.org
>>>>
>>>> Specification
>>>>
>>>> https://w3c.github.io/webappsec-clear-site-data/
>>>>
>>>> Summary
>>>>
>>>> Websites will now be able to clear all storage targets (“cookies”,
>>>> “cache”, and “storage”) by sending `Clear-Site-Data: “*”`. Note that Chrome
>>>> does not support clearing “executionContexts” at the moment, but if we
>>>> added it in the future any header targeting “*” would then clear them too.
>>>>
>>>
>>> What's the risk we'll find we can't actually do that for web compat
>>> reasons? Eg. sites deploying "*" today without appreciating the
>>> implications of asking for pages to be reloaded, then when we add
>>> "executionContexts" support in the future the user experience degrades
>>> enough that we can't actually ship it as part of the "*" set? Do we have
>>> guidance for developers on when to use "*" vs. a specific list?
>>>
>>
>> That's a reasonable question, so I could see adding a note in the spec so
>> developers are forewarned. The danger seems less compatibility and more
>> performance (if new methods take significant time) as anyone sending "*"
>> should assume no prior state for that page is retained.
>>
>>
>>>
>>> Note: This was proposed in 2017, but never launched in Chrome.
>>>>
>>>> Blink component
>>>>
>>>> Blink>Storage
>>>> <https://bugs.chromium.org/p/chromium/issues/list?q=component%3ABlink%3EStorage>
>>>>
>>>> Motivation
>>>>
>>>> If a website really wanted to clear all data they would have to list
>>>> all possible targets in the header and be sure to check if any were added
>>>> in the future. By using “*” as the target, a website can be sure all data
>>>> the browser supports to clear via the header will be cleared.
>>>>
>>>> TAG review
>>>>
>>>> https://github.com/w3ctag/design-reviews/issues/62
>>>>
>>>>
>>>> Compatibility & Interoperability
>>>>
>>>> We would be the first to implement if approved.
>>>>
>>>
>>> Does Firefox (and older Chrome) just ignore the "*" token today? I.e.
>>> can developers list a set of tokens along with "*" in order to use this
>>> compatibly on both Chrome and Firefox?
>>>
>>
>> Firefox seems to support it
>> https://github.com/mozilla/gecko-dev/blame/master/toolkit/components/clearsitedata/ClearSiteData.cpp#L243
>> but Chrome ignores it
>> https://source.chromium.org/chromium/chromium/src/+/main:content/browser/browsing_data/clear_site_data_handler.cc;drc=6b23c6773893d03431973aaa53ec9305c02bfeb8;l=268.
>> In both cases you can list * with other targets for backwards compatibility.
>>
>>
>>>
>>> Gecko: Positive
>>>> <https://github.com/mozilla/standards-positions/issues/90>
>>>>
>>>>
>>>> WebKit: No current support
>>>> <https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Clear-Site-Data>
>>>>
>>>>
>>>> Web Developers: None so far
>>>>
>>>> Is this feature fully tested by web-platform-tests?
>>>>
>>>>
>>>> https://wpt.fyi/results/client-hints/clear-site-data?label=experimental&label=master&aligned
>>>>
>>>>
>>>> https://wpt.fyi/results/clear-site-data?label=experimental&label=master&aligned
>>>>
>>>>
>>>> Tracking bug
>>>>
>>>> https://crbug.com/1464260
>>>>
>>>> Link to entry on the Chrome Platform Status
>>>>
>>>> https://chromestatus.com/feature/5074132743487488
>>>>
>>>> --
>>>> 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/CAGpy5DLiBdF2iFZxhr7Uu0HEm6NtickKF2T_D1nvymYCCuV%3DHQ%40mail.gmail.com
>>>> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAGpy5DLiBdF2iFZxhr7Uu0HEm6NtickKF2T_D1nvymYCCuV%3DHQ%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/CAFUtAY8S4erdt0A6NnZ%2B%3Dkqj589%3Dxb7Oumk%3D2j4_3aQ9p5%2Bitg%40mail.gmail.com
> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAFUtAY8S4erdt0A6NnZ%2B%3Dkqj589%3Dxb7Oumk%3D2j4_3aQ9p5%2Bitg%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/fed5dc9b-4ea3-185d-4039-c1cd33578abf%40gmail.com
> <https://groups.google.com/a/chromium.org/d/msgid/blink-dev/fed5dc9b-4ea3-185d-4039-c1cd33578abf%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 on the web visit 
https://groups.google.com/a/chromium.org/d/msgid/blink-dev/CAOMQ%2Bw9i75Ym4A%2B7u0wykJTPfusBxEdGDtb-Hqbo204RqtTJ9w%40mail.gmail.com.

Reply via email to