Skip to content

RUM-10350: Add telemetry to track WebView instrumentation usage#3087

Merged
kikoveiga merged 2 commits into
developfrom
kikoveiga/RUM-10350/add-webview-telemetry
Dec 31, 2025
Merged

RUM-10350: Add telemetry to track WebView instrumentation usage#3087
kikoveiga merged 2 commits into
developfrom
kikoveiga/RUM-10350/add-webview-telemetry

Conversation

@kikoveiga

@kikoveiga kikoveiga commented Dec 30, 2025

Copy link
Copy Markdown
Contributor

What does this PR do?

Adds usage telemetry that sends a TrackWebView event when WebViewTracking.enable() is called.

Motivation

We currently lack visibility into how many orgs use WebView (WV) tracking.

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)

@kikoveiga kikoveiga self-assigned this Dec 30, 2025
@kikoveiga
kikoveiga force-pushed the kikoveiga/RUM-10350/add-webview-telemetry branch 3 times, most recently from 266878f to aee0b0f Compare December 30, 2025 16:55
@datadog-official

datadog-official Bot commented Dec 30, 2025

Copy link
Copy Markdown

🎯 Code Coverage
Patch Coverage: 75.00%
Overall Coverage: 66.51% (+0.08%)

View detailed report

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

@codecov-commenter

codecov-commenter commented Dec 30, 2025

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 81.81818% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.39%. Comparing base (bcbe8d3) to head (a94eeba).
⚠️ Report is 267 commits behind head on develop.

Files with missing lines Patch % Lines
...droid/internal/telemetry/InternalTelemetryEvent.kt 0.00% 2 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3087      +/-   ##
===========================================
- Coverage    71.40%   71.39%   -0.01%     
===========================================
  Files          881      881              
  Lines        32397    32405       +8     
  Branches      5462     5463       +1     
===========================================
+ Hits         23131    23135       +4     
- Misses        7722     7723       +1     
- Partials      1544     1547       +3     
Files with missing lines Coverage Δ
...n/kotlin/com/datadog/android/api/InternalLogger.kt 94.12% <ø> (ø)
...ndroid/telemetry/internal/TelemetryEventHandler.kt 87.96% <100.00%> (+1.01%) ⬆️
...lin/com/datadog/android/webview/WebViewTracking.kt 84.87% <100.00%> (+0.53%) ⬆️
...droid/internal/telemetry/InternalTelemetryEvent.kt 23.68% <0.00%> (-1.32%) ⬇️

... and 35 files with indirect coverage changes

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

@kikoveiga
kikoveiga marked this pull request as ready for review December 30, 2025 17:52
@kikoveiga
kikoveiga requested a review from a team as a code owner December 30, 2025 17:52
0xnm
0xnm previously approved these changes Dec 31, 2025
@kikoveiga
kikoveiga force-pushed the kikoveiga/RUM-10350/add-webview-telemetry branch from 68405ea to a94eeba Compare December 31, 2025 10:39
@kikoveiga
kikoveiga merged commit 8f86c4b into develop Dec 31, 2025
26 of 27 checks passed
@kikoveiga
kikoveiga deleted the kikoveiga/RUM-10350/add-webview-telemetry branch December 31, 2025 11:38
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