Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix visibility change and loss of full activity events #595

Merged
merged 3 commits into from
Feb 10, 2025

Conversation

reillyeon
Copy link
Contributor

@reillyeon reillyeon commented Oct 3, 2022

This change fixes the steps for handling visibility change and loss of full activity to find the global object for the Document which experienced the event and from there identify the Bluetooth object for which we need to take action.

This introduces the concept of an "associated Bluetooth", similar to an "associated Document" and adds a "Bluetooth task source" which is now used whenever queuing a task.

Fixed #593.


Preview | Diff

This change fixes the steps for handling visibility change and loss of
full activity to find the global object for the Document which
experienced the event and from there identify the Bluetooth object for
which we need to take action.

This introduces the concept of an "associated Bluetooth", similar to an
"associated Document" and adds a "Bluetooth task source" which is now
used whenever queuing a task.

Fixed #593.
@reillyeon reillyeon requested a review from jyasskin October 3, 2022 22:15
@reillyeon
Copy link
Contributor Author

Oh no, this change never got merged and now I need to rebase and reconcile it with #643.

@reillyeon reillyeon requested review from scheib and removed request for jyasskin February 8, 2025 01:46
@reillyeon
Copy link
Contributor Author

This is a follow-up to my last PR which further cleans up the spec language and fixes the referenced issue.

Copy link
Member

@jyasskin jyasskin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice if all the changes to queuing tasks were in a separate PR, but I think they're all right.

index.bs Outdated Show resolved Hide resolved
index.bs Outdated Show resolved Hide resolved
@reillyeon reillyeon requested a review from jyasskin February 10, 2025 22:48
@reillyeon reillyeon merged commit 030375d into main Feb 10, 2025
2 checks passed
@reillyeon reillyeon deleted the relevant_global branch February 10, 2025 22:56
github-actions bot added a commit that referenced this pull request Feb 10, 2025
SHA: 030375d
Reason: push, by reillyeon

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Feb 10, 2025
SHA: 030375d
Reason: push, by reillyeon

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Feb 10, 2025
SHA: 030375d
Reason: push, by reillyeon

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Feb 10, 2025
SHA: 030375d
Reason: push, by reillyeon

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"current settings object's relevant global object" is not defined
2 participants