1403945 - Utility functions to recognize OS X 10.13

Status: RESOLVED FIXED

(Core :: Graphics, enhancement, P1)

Description

[Milan Sreckovic [:milan] (needinfo for best results)]

Not making any decisions based on these versions, but making sure the underlying utility functions are in place.

Comment 1

[Milan Sreckovic [:milan] (needinfo for best results)]

Attachment: Bug 1403945: Add utility functions to recognize OS X 10.13.

Review commit: https://reviewboard.mozilla.org/r/184624/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/184624/

Comment 2

[Markus Stange [:mstange]]

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

https://reviewboard.mozilla.org/r/184624/#review189812

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8063ebcb25db
Add utility functions to recognize OS X 10.13. r=mstange

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/8063ebcb25db

Comment 5

[Sylvestre Ledru [:Sylvestre]]

Do we want to uplift that to 57?

Comment 6

[Milan Sreckovic [:milan] (needinfo for best results)]

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.

Comment 7

[Sylvestre Ledru [:Sylvestre]]

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

Comment 8

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/releases/mozilla-beta/rev/9bd05768af69