Closed Bug 838879 Opened 9 years ago Closed 9 years ago
Decouple FHR from desktop-specific code
Uploading, UX triggering, pref management should all be abstracted in such a way that we can ship only relevant code on Android, and implement the equivalents in Java.
Shortest term: allow Android to only write prefs, not call a JS function, when enabling or disabling.
OS: Mac OS X → All
Hardware: x86 → All
Assignee: nobody → rnewman
Status: NEW → ASSIGNED
This splits out all of the logic that isn't concerned with uploading and prefs. This is step 1, with subsequent steps being to conditionalize the definition of HealthReporter and its bagheeraclient imports, define a class suitable for use on Android, and conditionally use that instead, and fix tests appropriately. I was planning to define MOZ_USE_EXTERNAL_HEALTHREPORTER to control this; do you have an opinion?
Attachment #713194 - Flags: review?(gps)
Comment on attachment 713194 [details] [diff] [review] Part 1: split out AbstractHealthReporter. v1 Review of attachment 713194 [details] [diff] [review]: ----------------------------------------------------------------- I didn't verify that the chunks moved to the new prototype didn't actually change. I trust you to not try pull off something mischievous :) When I had this refactoring designed in my head, I arrived at a slightly different API design. I won't talk about it unless you are interested. You got there first. You win :)
Attachment #713194 - Flags: review?(gps) → review+
Whiteboard: [fixed in services][leave open]
Whiteboard: [fixed in services][leave open] → [leave open]
I'm going to resolve this, because it crosses a release boundary. Will open another bug if necessary.
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [leave open]
Target Milestone: --- → mozilla21
Component: Metrics and Firefox Health Report → Client: Desktop
Product: Mozilla Services → Firefox Health Report
Target Milestone: mozilla21 → Firefox 21
Product: Firefox Health Report → Firefox Health Report Graveyard
You need to log in before you can comment on or make changes to this bug.