Utility functions to recognize OS X 10.13

RESOLVED FIXED in Firefox 57

Status

()

Product:
Core
Component:
Graphics
Type:
enhancement
Importance:
P1
normal
Status:
RESOLVED FIXED
Opened:
2 years ago
Updated:
2 years 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

•
2 years 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 2

•
2 years 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

•
2 years 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

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

Comment 5

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

Comment 6

•
2 years 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

•
2 years 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+
You need to log in before you can comment on or make changes to this bug.