Closed
Bug 1341561
Opened 8 years ago
Closed 8 years ago
List of installled updates in about:support says "security updates" for Nightly and they are not security fixes
Categories
(Toolkit :: General, defect)
Toolkit
General
Tracking
()
RESOLVED
DUPLICATE
of bug 406738
People
(Reporter: pascalc, Assigned: pascalc)
Details
Attachments
(2 files)
STR:
1/ Use a nightly build
2/ Go to about:support
3/ Click on "Show update history" button
ER:
Nightly updates are labelled as regular updates
AR:
Nightly updates are labelled as "security updates" which they are not (see screenshot)
| Comment hidden (mozreview-request) |
| Comment hidden (mozreview-request) |
| Assignee | ||
Comment 3•8 years ago
|
||
Flod, this patch would introduce a new string, "Regular Update", which would be displayed for Nightly instead of "Security Update".
Flags: needinfo?(francesco.lodolo)
Comment 4•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8839933 [details]
Bug 1341561: List of installled updates in about:support says "security updates" for Nightly and they are not security fixes
https://reviewboard.mozilla.org/r/114500/#review115976
::: browser/components/preferences/in-content/tests/browser_advanced_update.js:140
(Diff revision 2)
> is(updates.length, mockUpdateManager.updateCount, "The update count is incorrect.");
>
> // Test the updates are displayed correctly
> let update = null;
> let updateData = null;
> + let updateType = AppConstants.NIGHTLY_BUILD ? "Regular Update" : "Security Update";
I think we should not limit the change to nightly on only.
For example, aurora & beta are not security updates either.
Comment 5•8 years ago
|
||
(In reply to Pascal Chevrel:pascalc from comment #3)
> Flod, this patch would introduce a new string, "Regular Update", which would
> be displayed for Nightly instead of "Security Update".
Thanks, don't see any issue with this.
Flags: needinfo?(francesco.lodolo)
| Comment hidden (mozreview-request) |
Updated•8 years ago
|
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•