Closed Bug 865798 Opened 9 years ago Closed 9 years ago
Date/update Date are not added to social manifests
install/update should stamp the manifest so the addon wrapper can properly provide that information for the addon manager and fhr. The addon wrapper in SocialService.jsm does have the getters, but the AddonInstaller doesn't add the stamp.
Assignee: nobody → mixedpuppy
Attachment #742002 - Flags: review?(felipc)
Attachment #742002 - Flags: review?(felipc) → review+
adding tests and try https://tbpl.mozilla.org/?tree=Try&rev=058a9c2495e3
Comment on attachment 743193 [details] [diff] [review] manifest timestamps.patch r+ with the potential removal of the "!Services.prefs.prefHasUserValue(prefname)" check, if you think it's ok to do. (run tests again?) The comment didn't help me much to understand the if condition. I think you could add something like "... is set at a user-level by checking the existence of the updateDate field, which is not present in the default pref value".
Attachment #743193 - Flags: review?(felipc) → review+
patch with discussed change, carry forward r+, retested locally https://hg.mozilla.org/integration/mozilla-inbound/rev/2d02910b859e
Comment on attachment 743378 [details] [diff] [review] manifest timestamps.patch [Approval Request Comment] Bug caused by (feature/regressing bug #): 851653 User impact if declined: fhr reporting is slightly flawed Testing completed (on m-c, etc.): on m-c Risk to taking this patch (and alternatives if risky): low String or IDL/UUID changes made by this patch: none
Attachment #743378 - Flags: approval-mozilla-aurora?
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 23
Comment on attachment 743378 [details] [diff] [review] manifest timestamps.patch In support of FHR
Attachment #743378 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Does this bug require any qa testing? It seems have patch landed for it.
Looks like this has tests in-testsuite so I'm tagging this [qa-]. Please remove this tag and add the verifyme keyword if you feel this needs QA verification.
Flags: in-testsuite? → in-testsuite+
Whiteboard: [qa?] → [qa-]
You need to log in before you can comment on or make changes to this bug.