| 07:38 | <annevk> | krosylight (they/them): any reason https://github.com/web-platform-tests/wpt/blob/master/notifications/icon-url-encoding-euc-kr.tentative.https.html is tentative? This also applies to navigate. |
| 07:40 | <annevk> | Cookie Store and some other APIs too. I really need to flip the default behavior of the UTF-8 argument in WebKit. |
| 07:40 | <krosylight (they/them)> | I think mostly because https://github.com/whatwg/notifications/issues/209 is still open and only gecko passes the test https://wpt.fyi/results/notifications/icon-url-encoding-euc-kr.tentative.https.html?label=experimental&label=master&aligned |
| 07:41 | <krosylight (they/them)> | should do something for fetch steps in notifications... |
| 07:41 | <annevk> | krosylight (they/them): I see. I think it would be fine to make it non-tentative as it matches the specification. tentative is really for when the specification would say to use the document encoding or some such. |
| 07:43 | <krosylight (they/them)> | I think the fetch call in notifications is simply wrong so that's also a part, but I'm all for dropping tentative |
| 07:44 | <krosylight (they/them)> | Do you want to file a PR or should I? |
| 07:44 | <annevk> | How does the fetch call relate to this encoding question? |
| 07:46 | <krosylight (they/them)> | good question, that's actually about URL parse "If options["image"] exists, then parse it using baseURL" already should default to UTF-8 |
| 07:46 | <krosylight (they/them)> | Ok fine |
| 07:47 | <annevk> | https://github.com/web-platform-tests/wpt/pull/59775 |
| 07:48 | <krosylight (they/them)> | annevk: approved. Thanks! (How come are you working on sunday 👀) |