Utility functions to recognize OS X 10.13

RESOLVED FIXED in Firefox 57

Status

()

Product:
Core
Component:
Graphics
Importance:
P1
normal
Status:
RESOLVED FIXED
Reported:
a year ago
Modified:
a year ago

People

(Reporter: milan, Assigned: milan)

Tracking

Version:
unspecified
Target:
mozilla58
Points:
---

Firefox Tracking Flags

(firefox57 fixed, firefox58 fixed)

Details

(Whiteboard: [gfx-noted])

Attachments

(1 attachment)

Bug 1403945: Add utility functions to recognize OS X 10.13.
59 bytes, text/x-review-board-request
mstange
: review+
sylvestre
: approval-mozilla-beta+
Details
(Assignee)

Description

•
a year ago 
Not making any decisions based on these versions, but making sure the underlying utility functions are in place.
Assignee: nobody → milan
status-firefox57: --- → affected
status-firefox58: --- → affected
Priority: -- → P1
Whiteboard: [gfx-noted]
Comment hidden (mozreview-request)

Comment 2

•
a year ago
mozreview-review 
Comment on attachment 8913221 [details]
Bug 1403945: Add utility functions to recognize OS X 10.13.

https://reviewboard.mozilla.org/r/184624/#review189812
Attachment #8913221 - Flags: review?(mstange) → review+

Comment 3

•
a year ago 
Pushed by msreckovic@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/8063ebcb25db
Add utility functions to recognize OS X 10.13. r=mstange

Comment 4

•
a year ago
bugherder 
https://hg.mozilla.org/mozilla-central/rev/8063ebcb25db
Status: NEW → RESOLVED
Last Resolved: a year ago
status-firefox58: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla58

Comment 5

•
a year ago 
Do we want to uplift that to 57?
Flags: needinfo?(milan)
 (Assignee)

Comment 6

•
a year ago 
Comment on attachment 8913221 [details]
Bug 1403945: Add utility functions to recognize OS X 10.13.

Approval Request Comment
This gives us flexibility in case something like bug 1400057 gets a 10.13 specific solution.  Right now, the code is unused, so it is low risk, but if we need to gate specific functionality, or add downloadable blocklist item for 10.13, we need this patch.
Flags: needinfo?(milan)
Attachment #8913221 - Flags: approval-mozilla-beta?

Comment 7

•
a year ago 
Comment on attachment 8913221 [details]
Bug 1403945: Add utility functions to recognize OS X 10.13.

In the (likely) case we need that, taking it. Should be in 57b5
Attachment #8913221 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

Comment 8

•
a year ago
bugherderuplift 
https://hg.mozilla.org/releases/mozilla-beta/rev/9bd05768af69
status-firefox57: affected → fixed
You need to log in before you can comment on or make changes to this bug.