Closed Bug 2030667 Opened 1 month ago Closed 21 days ago

Refactor MarketingAttributionService to allow reusing it for RTAMO

Categories

(Firefox for Android :: Onboarding, task)

All
Android
task

Tracking

()

RESOLVED FIXED
152 Branch
Tracking Status
firefox152 --- fixed

People

(Reporter: petru, Assigned: petru)

References

(Blocks 1 open bug)

Details

(Whiteboard: [fxdroid][group3])

Attachments

(2 files)

Bug 2028867 showed that we already have MarketingAttributionService which uses Google's InstallReferrerClient to infer where Fenix was installed from.

Since InstallReferrerClient is a heavy and sometimes slow service we should not use multiple separate instances of it but rather use the same instance to get the referrer data only once and then use this data for inferring various scenarios.

The goal of this ticket is to support refactoring MarketingAttributionService to a generic service that can be reused for inferring both attribution scenarios

  • marketing scenarios
  • Mozilla's own AMO webpage.

Renamed this class separately from further changes to ensure that git will not
treat the combined actions as producing an entirely new file.

Same functionality and code, just moved to different places.

In preparation of using InstallReferrerHandlingService for multiple scenarios
this patch extract the functionality of inferring the marketing attribution
details to a separate MarketingAttributionHandler.

Pushed by plingurar@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/a11de944b15c https://hg.mozilla.org/integration/autoland/rev/31ff72a11b03 part 1 - Rename MarketingAttributionService to InstallReferrerHandlingService r=android-reviewers,moyin https://github.com/mozilla-firefox/firefox/commit/52b235a6ba7d https://hg.mozilla.org/integration/autoland/rev/01753c82fd77 part 2 - Refactor marketing attribution inference and tests r=android-reviewers,moyin
Status: ASSIGNED → RESOLVED
Closed: 21 days ago
Resolution: --- → FIXED
Target Milestone: --- → 152 Branch
Regressions: 2038350
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: