Extract frame draw detection to its own component

[bidetofevil]

Goal

Extract the code to detect when a app startup is ended by the first activity drawing its first frame into its own class so it can be used for the UI Load trace as well.

[bidetofevil]

• Make FirstDrawDetector a singleton #1796
• Add unregistering of draw callback #1795
• Add render detection to UI Load traces #1794
• Extract frame draw detection to its own component #1793 👈 (View in Graphite)
• Restructure assertions to remove magic numbers #1791
• API and integration tests for manually ending UI load traces #1790
• Allow all activities to ge traced by default #1789
• Remove the reset event #1788
• Change start time of UI Load traces #1787
• Align event emitter names with lifecycle methods #1786
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[codecov]

Codecov Report

Attention: Patch coverage is 43.75000% with 36 lines in your changes missing coverage. Please review.

Project coverage is 85.52%. Comparing base (8048b99) to head (a204c36).
Report is 2 commits behind head on main.

Files with missing lines	Patch %	Lines
...ndroid/embracesdk/internal/ui/FirstDrawDetector.kt	26.53%	31 Missing and 5 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1793      +/-   ##
==========================================
+ Coverage   85.44%   85.52%   +0.08%     
==========================================
  Files         467      468       +1     
  Lines       10964    10965       +1     
  Branches     1629     1630       +1     
==========================================
+ Hits         9368     9378      +10     
+ Misses        863      857       -6     
+ Partials      733      730       -3     

Files with missing lines	Coverage Δ
...racesdk/internal/capture/startup/StartupTracker.kt	90.24% <100.00%> (+39.69%)	⬆️
...internal/injection/DataCaptureServiceModuleImpl.kt	95.74% <100.00%> (+0.09%)	⬆️
...d/embracesdk/internal/logging/InternalErrorType.kt	100.00% <100.00%> (ø)
...ndroid/embracesdk/internal/ui/FirstDrawDetector.kt	26.53% <26.53%> (ø)

... and 3 files with indirect coverage changes

[fractalwrench]

LGTM

[bidetofevil]

Merge activity

• Jan 8, 11:04 AM EST: A user started a stack merge that includes this pull request via Graphite.
• Jan 8, 11:23 AM EST: Graphite rebased this pull request as part of a merge.
• Jan 8, 11:34 AM EST: A user merged this pull request with Graphite.

[github-actions]

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

OpenSSF Scorecard

Package	Version	Score	Details

Scanned Files