-
Notifications
You must be signed in to change notification settings - Fork 116
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
Add Starboard thread_sampler APIs to Android #2689
Add Starboard thread_sampler APIs to Android #2689
Conversation
Change-Id: Ifb64271ced9df64cce67ad3e545fe8a59fe813c1
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2689 +/- ##
==========================================
+ Coverage 58.27% 58.90% +0.63%
==========================================
Files 1795 1812 +17
Lines 84862 87479 +2617
==========================================
+ Hits 49450 51533 +2083
- Misses 35412 35946 +534 ☔ View full report in Codecov by Sentry. |
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.
LGTM.
The orignal implementation is https://lbshell-internal-review.git.corp.google.com/c/starboard/+/123104
Please wait for x86 on device tests to be verified. |
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.
looks ready to land
Merging |
Swaps Starboard's thread_sampler APIs for Android TV from stub to shared implementations (`//starboard/shared/stub/thread_sampler_*` to `//starboard/shared/pthread/thread_sampler_*`). Tested on Android TV (x64), Google TV Emulator (arm64), and a real Google TV device (arm). Test-On-Device: true b/330564529 Co-authored-by: Ahmed Elzeiny <[email protected]> (cherry picked from commit 7984c7a)
…2721) Refer to the original PR: #2689 Swaps Starboard's thread_sampler APIs for Android TV from stub to shared implementations (`//starboard/shared/stub/thread_sampler_*` to `//starboard/shared/pthread/thread_sampler_*`). Tested on Android TV (x64), Google TV Emulator (arm64), and a real Google TV device (arm). Test-On-Device: true b/330564529 Co-authored-by: Ahmed Elzeiny <[email protected]>
Swaps Starboard's thread_sampler APIs for Android TV from stub to shared implementations (
//starboard/shared/stub/thread_sampler_*
to//starboard/shared/pthread/thread_sampler_*
).Tested on Android TV (x64), Google TV Emulator (arm64), and a real Google TV device (arm).
Test-On-Device: true
b/330564529