Skip to content

RUMS-5352 RUM-13527: Fix issue with OnWindowFocusChanged#3083

Merged
jonathanmos merged 1 commit into
developfrom
jmoskovich/rum-13527/fix-onwindowfocuschanged
Dec 30, 2025
Merged

RUMS-5352 RUM-13527: Fix issue with OnWindowFocusChanged#3083
jonathanmos merged 1 commit into
developfrom
jmoskovich/rum-13527/fix-onwindowfocuschanged

Conversation

@jonathanmos

Copy link
Copy Markdown
Member

What does this PR do?

We were missing a call to the OnWindowFocusChanged callback in SR, and as a result were blocking this callback from returning.

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)

@jonathanmos
jonathanmos force-pushed the jmoskovich/rum-13527/fix-onwindowfocuschanged branch from c87be23 to 08ea8b8 Compare December 30, 2025 10:23
@datadog-official

datadog-official Bot commented Dec 30, 2025

Copy link
Copy Markdown

🎯 Code Coverage
Patch Coverage: 100.00%
Overall Coverage: 66.41% (+0.02%)

View detailed report

This comment will be updated automatically if new data arrives.
🔗 Commit SHA: 08ea8b8 | Docs | Datadog PR Page | Was this helpful? Give us feedback!

@codecov-commenter

codecov-commenter commented Dec 30, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.38%. Comparing base (96f099f) to head (08ea8b8).
⚠️ Report is 964 commits behind head on develop.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3083      +/-   ##
===========================================
+ Coverage    71.35%   71.38%   +0.02%     
===========================================
  Files          881      881              
  Lines        32388    32389       +1     
  Branches      5462     5462              
===========================================
+ Hits         23110    23118       +8     
+ Misses        7732     7724       -8     
- Partials      1546     1547       +1     
Files with missing lines Coverage Δ
...ternal/recorder/callback/RecorderWindowCallback.kt 92.13% <100.00%> (+0.09%) ⬆️

... and 36 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jonathanmos
jonathanmos marked this pull request as ready for review December 30, 2025 12:53
@jonathanmos
jonathanmos requested a review from a team as a code owner December 30, 2025 12:53
@jonathanmos
jonathanmos merged commit 714d915 into develop Dec 30, 2025
26 of 27 checks passed
@jonathanmos
jonathanmos deleted the jmoskovich/rum-13527/fix-onwindowfocuschanged branch December 30, 2025 13:07
@allspain allspain changed the title RUM-13527: Fix issue with OnWindowFocusChanged RUMS-5352 RUM-13527: Fix issue with OnWindowFocusChanged Feb 3, 2026
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.

4 participants