Skip to content

RUM-3761 prevent crashing the host app in the ViewOnDrawInterceptor#1951

Merged
xgouchet merged 1 commit into
developfrom
xgouchet/RUM-3761/crash_viewondrawinterceptor
Mar 28, 2024
Merged

RUM-3761 prevent crashing the host app in the ViewOnDrawInterceptor#1951
xgouchet merged 1 commit into
developfrom
xgouchet/RUM-3761/crash_viewondrawinterceptor

Conversation

@xgouchet

Copy link
Copy Markdown
Contributor

What does this PR do?

Prevent a crash previously unnoticed

Motivation

What inspired you to submit this pull request?

Additional Notes

Anything else we should know when reviewing?

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@xgouchet
xgouchet requested review from a team as code owners March 28, 2024 10:01
@codecov-commenter

codecov-commenter commented Mar 28, 2024

Copy link
Copy Markdown

Codecov Report

Merging #1951 (39d95ac) into develop (aab05c6) will decrease coverage by 0.05%.
The diff coverage is 87.88%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #1951      +/-   ##
===========================================
- Coverage    83.14%   83.09%   -0.05%     
===========================================
  Files          486      486              
  Lines        17517    17545      +28     
  Branches      2600     2602       +2     
===========================================
+ Hits         14563    14578      +15     
- Misses        2258     2262       +4     
- Partials       696      705       +9     
Files Coverage Δ
...dog/android/sessionreplay/SessionReplayRecorder.kt 96.36% <100.00%> (+0.49%) ⬆️
...nreplay/internal/recorder/ViewOnDrawInterceptor.kt 90.48% <87.10%> (-5.36%) ⬇️

... and 31 files with indirect coverage changes

@xgouchet
xgouchet force-pushed the xgouchet/RUM-3761/crash_viewondrawinterceptor branch from e958318 to 39d95ac Compare March 28, 2024 12:23
@xgouchet
xgouchet requested a review from 0xnm March 28, 2024 12:25
@xgouchet
xgouchet merged commit 736e81c into develop Mar 28, 2024
@xgouchet
xgouchet deleted the xgouchet/RUM-3761/crash_viewondrawinterceptor branch March 28, 2024 13:59
@xgouchet xgouchet added this to the 2.8.x milestone Apr 5, 2024
@0xnm 0xnm mentioned this pull request Sep 16, 2025
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants