-
Notifications
You must be signed in to change notification settings - Fork 3
feat: support system info driven by pressure api #11
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
Conversation
brycetham
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some small comments. Overall the API looks good :)
brycetham
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved, with a small comment that doesn't need to be addressed (just wanted to put my thoughts out there).
| * | ||
| * @returns True if the Compute Pressure API is supported, false otherwise. | ||
| */ | ||
| static isPressureObserverSupported(): boolean { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a thought I had... every time I look at this API, I wonder to myself how useful exposing it is. But I guess it doesn't hurt to keep it.
No description provided.