Closed rakuco closed 2 years ago
cc @miketaylr
@miketaylr / @anssiko friendly ping. Is there anything else you'd like me to address here?
@anssiko thanks for reviewing. I don't have write access to this repository; can you land this one for me? It's also fine if you're waiting for Mike's feedback.
@miketaylr please let us know if you'd like to adjust the "given document" part. I'll merge this now, and we'll update based on your suggestion as needed.
Apologies for showing up late. :)
I think it would be nice to formally define "given document" as the active document (or just use that in place), and tie the active document to a browsing context (something like "this's browsing context's active document").
This has been removed from HTML. Instead, retrieve the document's relevant settings object (which is an environment as specified in HTML) and run the "is a secure context" check that is defined in HTML on it.
Fixes #432.