Closed
Bug 868274
Opened 12 years ago
Closed 12 years ago
Generate FHR documents
Categories
(Firefox Health Report Graveyard :: Client: Android, defect, P1)
Tracking
(firefox23 fixed)
RESOLVED
FIXED
Firefox 24
Tracking | Status | |
---|---|---|
firefox23 | --- | fixed |
People
(Reporter: rnewman, Assigned: rnewman)
References
Details
Attachments
(1 file)
3.09 KB,
patch
|
nalexander
:
review+
lsblakk
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
Working on this as I build storage (Bug 858742).
Assignee | ||
Updated•12 years ago
|
Priority: -- → P1
Assignee | ||
Comment 1•12 years ago
|
||
Like most of the rest, this is in rnewman/storage in Git.
Attachment #751922 -
Flags: review?(nalexander)
Comment 2•12 years ago
|
||
Comment on attachment 751922 [details] [diff] [review]
Proposed patch. v1 (from git)
Review of attachment 751922 [details] [diff] [review]:
-----------------------------------------------------------------
::: mobile/android/base/background/healthreport/HealthReportGenerator.java
@@ +46,4 @@
> * The document consists of:
> *
> * * Basic metadata: last ping time, current ping time, version.
> * * A map of environments: 'current' and others named by hash. 'current' is fully specified,
nit: format this is an HTML list.
Attachment #751922 -
Flags: review?(nalexander) → review+
Assignee | ||
Comment 3•12 years ago
|
||
Target Milestone: --- → Firefox 24
Comment 4•12 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 5•12 years ago
|
||
Comment on attachment 751922 [details] [diff] [review]
Proposed patch. v1 (from git)
[Approval Request Comment]
Bug caused by (feature/regressing bug #):
User impact if declined:
Testing completed (on m-c, etc.):
Risk to taking this patch (and alternatives if risky):
String or IDL/UUID changes made by this patch:
[Approval Request Comment]
Bug caused by (feature/regressing bug #):
New work.
User impact if declined:
No FHR in 23.
Testing completed (on m-c, etc.):
Baking on m-c for some time. QA overview written up; waiting for QA to execute. Manual testing and automated unit tests.
Risk to taking this patch (and alternatives if risky):
None: new work.
String or IDL/UUID changes made by this patch:
None.
Attachment #751922 -
Flags: approval-mozilla-aurora?
Comment 6•12 years ago
|
||
Comment on attachment 751922 [details] [diff] [review]
Proposed patch. v1 (from git)
Approving for uplift as part of the body of work for FHR's first revision on Android.
Attachment #751922 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Assignee | ||
Comment 7•12 years ago
|
||
Assignee | ||
Updated•12 years ago
|
status-firefox23:
--- → fixed
Updated•6 years ago
|
Product: Firefox Health Report → Firefox Health Report Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•