Closed
Bug 1045087
Opened 10 years ago
Closed 10 years ago
Remove Product Announcements from Fennec settings and pref broadcaster
Categories
(Firefox for Android Graveyard :: Settings and Preferences, defect)
Tracking
(firefox33 verified, firefox34 verified)
VERIFIED
FIXED
Firefox 34
People
(Reporter: rnewman, Assigned: rnewman)
References
Details
Attachments
(1 file)
16.69 KB,
patch
|
mfinkle
:
review+
Sylvestre
:
approval-mozilla-beta+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•10 years ago
|
||
Note that there are two omissions here: * We don't clean up the Fennec pref that tracks whether announcements are enabled. Neither do we clean up the product announcements prefs themselves (e.g., last fetch). * We no longer track last startup. I need to audit to make sure we're not using that elsewhere.
Attachment #8463551 -
Flags: review?(mark.finkle)
Comment 2•10 years ago
|
||
Comment on attachment 8463551 [details] [diff] [review] Remove Product Announcements integration points from Fennec. v1 This part looks good
Attachment #8463551 -
Flags: review?(mark.finkle) → review+
Assignee | ||
Comment 3•10 years ago
|
||
https://hg.mozilla.org/integration/fx-team/rev/ad7d3c6f8bde
https://hg.mozilla.org/mozilla-central/rev/ad7d3c6f8bde
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Target Milestone: --- → Firefox 34
Assignee | ||
Comment 5•10 years ago
|
||
Comment on attachment 8463551 [details] [diff] [review] Remove Product Announcements integration points from Fennec. v1 Trivial dependency of Bug 1045085, which is approved for Beta. This just removes the Settings screen, the prefs broadcaster, **one string**, and the manifest includes for the service that we delete in Bug 1045085.
Attachment #8463551 -
Flags: approval-mozilla-beta?
Updated•10 years ago
|
Attachment #8463551 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 6•10 years ago
|
||
https://hg.mozilla.org/releases/mozilla-beta/rev/c0ba357c4c89 With string changes omitted to appease the l10n gods.
status-firefox33:
--- → fixed
status-firefox34:
--- → fixed
Comment 7•10 years ago
|
||
'Show product announcements' doesn't appear anymore in Settings -> Mozilla, so I'll mark this as verified fixed. Builds: Firefox for Android 33 Firefox for Android 34 Beta 2 Device: Asus Transformer Pad TF300T (Android 4.2.1)
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•