Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Platforms affected
Motivation and Context
On iOS, no
batterystatus
events are emitted until the battery level changes. This is a problem because an app should be able to find out what the battery level is without waiting an arbitrarily long time for it to change. (If a device stays plugged in and at 100%, it might never change at all.) Android, by comparison, emits immediately.Description
Calls
updateBatteryStatus
every timestart
runs, so that new listeners get a value immediately.Testing
Confirmed using that
window.addEventListener('batterystatus', v => console.log(v);
produces immediate output, which it did not do before the change. Also validated in our app.Checklist
(platform)
if this change only applies to one platform (e.g.(android)
)