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

Deprecate SbTime APIs (partial) #2012

Merged
merged 1 commit into from
Dec 15, 2023
Merged

Deprecate SbTime APIs (partial) #2012

merged 1 commit into from
Dec 15, 2023

Conversation

gbournou
Copy link
Contributor

@gbournou gbournou commented Nov 28, 2023

  • Add emulation of POSIX APIs for Windows platforms
  • Provide common helper functions to simplify getting usec timestamps
  • Replace usages of SbTimeGetNow and SbTimeGetMonotonicNow in cobalt

b/302733082

Test-On-Device: true

@gbournou gbournou force-pushed the sbtime branch 6 times, most recently from 9dfa2d7 to eaf27fe Compare November 29, 2023 07:43
Copy link

codecov bot commented Nov 29, 2023

Codecov Report

Attention: 114 lines in your changes are missing coverage. Please review.

Comparison is base (7a34f86) 58.54% compared to head (cf0d093) 58.53%.
Report is 5 commits behind head on main.

Files Patch % Lines
cobalt/browser/web_module.cc 0.00% 17 Missing ⚠️
starboard/shared/modular/posix_time_wrappers.cc 0.00% 16 Missing ⚠️
cobalt/dom/performance_lifecycle_timing.cc 0.00% 12 Missing ⚠️
cobalt/dom/serialized_algorithm_runner.h 0.00% 10 Missing ⚠️
cobalt/xhr/xml_http_request.cc 0.00% 9 Missing ⚠️
cobalt/dom/source_buffer_metrics.cc 0.00% 7 Missing ⚠️
cobalt/dom/html_element.cc 50.00% 6 Missing ⚠️
cobalt/media_session/media_session_client.cc 0.00% 5 Missing ⚠️
cobalt/ui_navigation/scroll_engine/scroll_engine.h 0.00% 4 Missing ⚠️
starboard/common/time.cc 73.33% 4 Missing ⚠️
... and 13 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2012      +/-   ##
==========================================
- Coverage   58.54%   58.53%   -0.01%     
==========================================
  Files        1911     1914       +3     
  Lines       94379    94432      +53     
==========================================
+ Hits        55252    55276      +24     
- Misses      39127    39156      +29     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@gbournou gbournou force-pushed the sbtime branch 4 times, most recently from ba9c5e2 to 539ac20 Compare December 13, 2023 11:39
@gbournou gbournou force-pushed the sbtime branch 5 times, most recently from f619af1 to fed0f28 Compare December 14, 2023 23:40
@gbournou
Copy link
Contributor Author

The 45 failing tests all seem unrelated (see http://b/316393043).

starboard/elf_loader/exported_symbols.cc Outdated Show resolved Hide resolved
starboard/build/config/modular/BUILD.gn Outdated Show resolved Hide resolved
starboard/build/config/base_configuration.gni Outdated Show resolved Hide resolved
- Add emulation of POSIX APIs for Windows platforms
- Provide common helper functions to simplify getting usec timestamps
- Replace usages of SbTimeGetNow and SbTimeGetMonotonicNow in `cobalt`

b/302733082

Test-On-Device: true
@gbournou gbournou merged commit f615309 into youtube:main Dec 15, 2023
323 of 376 checks passed
Rongo-JL pushed a commit to Rongo-JL/cobalt that referenced this pull request Dec 19, 2023
- Add emulation of POSIX APIs for Windows platforms
- Provide common helper functions to simplify getting usec timestamps
- Replace usages of SbTimeGetNow and SbTimeGetMonotonicNow in `cobalt`

b/302733082

Test-On-Device: true
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants