- From: Ayu Ishii <notifications@github.com>
- Date: Thu, 28 May 2020 15:41:51 -0700
- To: w3ctag/design-reviews <design-reviews@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
Received on Thursday, 28 May 2020 22:42:06 UTC
Thanks @kenchris for the thorough advice. For the `set()` method change, I agree, I think this seems more straight forward without having 2 dict options. I filed a tracking [bug](https://crbug.com/1087580) for track Chrome's implementation. Removing optional from CookieStoreGetOptions seems reasonable as well. Currently the behavior is with no options, `get()`returns the first cookie (or `null` if no cookies are set). This is fairly arbitrary, and only makes sense from the lens of being consistent with `getAll()`. ([Bug](https://crbug.com/1087583) for tracking Chrome's implementation). Thanks! -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: https://github.com/w3ctag/design-reviews/issues/469#issuecomment-635646163
Received on Thursday, 28 May 2020 22:42:06 UTC