| 03:47 | <jmdyck> | In https://html.spec.whatwg.org/#obtain-an-embedder-policy, step 6.2, should the reference to "policy's endpoint" actually be "policy's report only reporting endpoint"? That's where the link goes. |
| 03:47 | <jmdyck> | (And maybe 4.2 should say "reporting endpoint") |
| 06:05 | <annevk> | jmdyck: yeah, both should have the full name indeed. Instead of parsedItem using (value, parameters) might also be an improvement here. |
| 08:56 | <Noam Rosenthal> | Added |
| 09:01 | <Noam Rosenthal> | https://github.com/whatwg/html/issues?q=is%3Aissue%20state%3Aopen%20%22navigation%20api%22%20label%3A%22topic%3A%20navigation%20API%22 |
| 12:56 | <Noam Rosenthal> | annevk, zcorpan I've fixed the pending issues in the fragment parsing PR stack (https://github.com/whatwg/html/pull/12583) if you want to put your LLMs to work on them again to see if you find more stuff (or just finish the review for the bottom one, whichever works for you). |
| 12:57 | <Noam Rosenthal> | I'm working on extracting a couple of cleanup fixes to separate PRs but they're not going to affect the first two |
| 13:02 | <zcorpan> | Checking |
| 13:24 | <jmdyck> | I'll add the name-changes to the 'fixes' PR I'm working on. Re (value, parameters), yeah that looks like an improvement, but |
| 13:39 | <jmdyck> | Also, there's some inconsistency re whether "report-only" has a hyphen or not. (Hyphens are in the majority.) |
| 14:09 | <annevk> | jmdyck: adding a hyphen makes sense to me. Should probably be done for the other fields as well then. |
| 14:14 | <sfarre> | Noam Rosenthal: after investigating https://github.com/whatwg/html/issues/12574, I think the provided test there is not good enough to actually prove the race condition in the spec, I think we just by sheer luck trigger it in chrome and gecko (but its harder for me to reason about chrome actually racing there). However, in inner navigate event firing algorithm we seem to just YOLO, if So maybe by spec'ing in the null-case there, would solve this. Maybe. |
| 14:16 | <Noam Rosenthal> | I will hopefully get to it tomorrow. Need to put my brain in navigation API race bug mode after a few months break from that |
| 14:16 | <zcorpan> | annevk: should https://github.com/whatwg/html/pull/12823 be merged without squashing? |
| 14:18 | <annevk> | Yeah I think that would be clearer. Tests still need to be reviewed though. |
| 14:19 | <sb3nder> | Hi! Does anyone know if the Windows WebKit build on build.webkit.org is expected to come back, or if it has been abandoned? It looks like it's been failing to compile for the past few months. |
| 14:19 | <zcorpan> | Just did :) ok |
| 18:20 | <Noam Rosenthal> | Not much new came from zcorpan's LLM run (one editorial fix in trusted types, 4 false positives) |