Closed Bug 901622 Opened 10 years ago Closed 10 years ago

Invalidate ProfileInformationCache after fixing add-on uninstallation

Categories

(Firefox Health Report Graveyard :: Client: Android, defect)

Product:
Firefox Health Report Graveyard
Component:
Client: Android
Platform:
All
Android
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(firefox24+ fixed, firefox25 fixed, firefox26 fixed)

Status:
RESOLVED FIXED
Milestone:
Firefox 26
Tracking Flags:
Tracking Status
firefox24 + fixed
firefox25 --- fixed
firefox26 --- fixed

People

(Reporter: rnewman, Assigned: mcomella)

References

Details

Attachments

(1 file)

Patch
2.57 KB, patch
mcomella
: review+
bajaj
: approval-mozilla-aurora+
bajaj
: approval-mozilla-beta+
Details | Diff | Splinter Review 
Just about to uplift Bug 900694 to Beta (pre B1). We should fix existing badness, too.
Assignee: nobody → michael.l.comella
Flaggin' for trackin'.

Release folks: this'll be the follow-on from Bug 900694 -- blowing away cached data to make sure we have an accurate snapshot of the user's add-ons, and are thus recording FHR events correctly.
Status: NEW → ASSIGNED
status-firefox24: --- → affected
status-firefox25: --- → affected
status-firefox26: --- → affected
tracking-firefox24: --- → ?
r+
Flags: needinfo?(rnewman)
https://hg.mozilla.org/mozilla-central/rev/3c188512a17c
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 26
Comment on attachment 786979 [details] [diff] [review]
Patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #):
    Initial landing of FHR and the fix deployed in Bug 900694 (already uplifted to 24).

User impact if declined: 
    Misrecorded addon data will propagate until cache is invalidated (which occurs on version bump - this patch - or corruption)

Testing completed (on m-c, etc.): 
    Tested locally, on m-c one week, unit tests via github (https://github.com/mozilla-services/android-sync/pull/345)

Risk to taking this patch (and alternatives if risky):
    Low risk - a version bump, whose behavior is tested by the unit tests linked above

String or IDL/UUID changes made by this patch:
    None
Attachment #786979 - Flags: approval-mozilla-beta?
Attachment #786979 - Flags: approval-mozilla-aurora?
Attachment #786979 - Flags: approval-mozilla-beta?
Attachment #786979 - Flags: approval-mozilla-beta+
Attachment #786979 - Flags: approval-mozilla-aurora?
Attachment #786979 - Flags: approval-mozilla-aurora+
Keywords: verifyme
Covered by unit tests
Keywords: verifyme
Product: Firefox Health Report → Firefox Health Report Graveyard
You need to log in before you can comment on or make changes to this bug.