RUM-14784: Network headers common instrumentation

[kikoveiga]

What does this PR do?

• Add trackResourceHeaders api in RumNetworkInstrumentationConfiguration.
• Create ResourceHeadersExtractor#toResourceAttributes for better logic encapsulation and some cleanup.
• Instrument both Sample App's okHttpClient and CronetEngine, both tested and working well.

Example Cronet resource with headers from the sample app here

Motivation

We want to support tracking resource headers for all network libraries (OkHttp & Cronet), configured with the new instrumentation coming in #3180.

This is the Part 2 of this feature, after instrumenting for OkHttp in DatadogInterceptor #3204.

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)

[codecov-commenter]

Codecov Report

❌ Patch coverage is 86.66667% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.97%. Comparing base (e40c7d7) to head (15d75f6).

Files with missing lines	Patch %	Lines
...roid/rum/internal/net/RumNetworkInstrumentation.kt	80.00%	0 Missing and 2 partials ⚠️
...g/android/rum/resource/ResourceHeadersExtractor.kt	89.47%	0 Missing and 2 partials ⚠️
...otlin/com/datadog/android/rum/_RumInternalProxy.kt	0.00%	1 Missing ⚠️
...n/com/datadog/android/okhttp/DatadogInterceptor.kt	90.91%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #3338      +/-   ##
===========================================
- Coverage    72.01%   71.97%   -0.04%     
===========================================
  Files          960      960              
  Lines        35298    35309      +11     
  Branches      5848     5856       +8     
===========================================
- Hits         25417    25412       -5     
- Misses        8281     8290       +9     
- Partials      1600     1607       +7     

Files with missing lines	Coverage Δ
...guration/RumNetworkInstrumentationConfiguration.kt	100.00% <100.00%> (ø)
...otlin/com/datadog/android/rum/_RumInternalProxy.kt	65.22% <0.00%> (+2.72%)	⬆️
...n/com/datadog/android/okhttp/DatadogInterceptor.kt	81.11% <90.91%> (-0.86%)	⬇️
...roid/rum/internal/net/RumNetworkInstrumentation.kt	83.87% <80.00%> (-0.83%)	⬇️
...g/android/rum/resource/ResourceHeadersExtractor.kt	92.65% <89.47%> (-1.10%)	⬇️

... and 33 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]

@codex review

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a9e7e11a8a

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ae4f9d3f36

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".