Skip to content

PermissionStatus update steps touch DOM objects in parallel #421

@jan-ivar

Description

@jan-ivar

The spec says:

image

Some questions:

  1. What does "it asynchronously runs" mean compared to "it runs"?
  2. What object does "this" refer to here? I suspect we mean "status"?
  3. (As an optimization?) It seems to reference associated Document and make a determination about fully active on a background thread. Isn't that racy (e.g. with navigation)?

By the time the queued task runs, the document may no longer be "fully active", and we fire the event on a non-fully active document, which seems wrong.

It would seem more correct to move these checks into the queued task.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions