RUMM-2142: Allow global context update if view is stopped in a new session

[0xnm]

What does this PR do?

This change should fix having a telemetry entry saying that current view ID doesn't match view ID in the global RUM context. This may happen if we update the context when new session started - this means we also change view ID for the current view.

This PR fixes that allowing update of the global context in case of the session ID mismatch.

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)

[mariusc83]

good catch

[xgouchet]

If the session id is different then it might mean that another view is active, which you could be erasing here. What's your logic for doing this check?

[0xnm]

Normally it shouldn't be another view at this point, because the current one should be stopped first (if we are saying that stopView is called). If stopView is not called for current view and another view starts (via startView), then stopped flag will be true after onStartView with event having key of another view, so we won't even go in this logical branch.

Anyway, such override was a case even before, when GlobalRum.updateRumContext was called in onStopView without such verification, so at least we are not making things worse.

[xgouchet]

If the session id is different then it might mean that another view is active, which you could be erasing here. What's your logic for doing this check?

[codecov-commenter]

Codecov Report

Merging #913 (e04f942) into develop (be3cb96) will decrease coverage by 0.12%.
The diff coverage is 88.24%.

@@             Coverage Diff             @@
##           develop     #913      +/-   ##
===========================================
- Coverage    83.07%   82.95%   -0.12%     
===========================================
  Files          267      267              
  Lines         9047     9054       +7     
  Branches      1453     1455       +2     
===========================================
- Hits          7515     7510       -5     
- Misses        1137     1148      +11     
- Partials       395      396       +1     

Impacted Files	Coverage Δ
.../android/rum/internal/domain/scope/RumViewScope.kt	96.55% <88.24%> (-0.44%)	⬇️
...ndroid/core/internal/persistence/file/EventMeta.kt	80.00% <0.00%> (-10.00%)	⬇️
...android/rum/internal/ndk/DatadogNdkCrashHandler.kt	83.91% <0.00%> (-4.60%)	⬇️
...rc/main/java/com/datadog/opentracing/DDTracer.java	55.65% <0.00%> (-0.42%)	⬇️
...tadog/android/core/internal/net/CurlInterceptor.kt	83.33% <0.00%> (+2.38%)	⬆️