Lion fullscreen check should be more versatile

RESOLVED FIXED in Firefox 15

Status

()

Firefox
General
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: zpao, Assigned: zpao)

Tracking

(Blocks: 1 bug)

Trunk
Firefox 15
All
Mac OS X
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

The check that we have in FullScreen.useLionFullScreen is limited to only checking if the user is on Lion. With Mountain Lion coming, that check needs to be expanded.

I think using parseFloat (or even parseInt) on Services.sysinfo.getProperty("version") should be good enough.

10.6.x -> 10.x (Snow Leopard)
10.7.y -> 11.y (Lion)
10.8.z -> 12.z (Mountain Lion)
Created attachment 621105 [details] [diff] [review]
Patch v0.1
Assignee: nobody → paul
Status: NEW → ASSIGNED
Attachment #621105 - Flags: review?(gavin.sharp)

Updated

5 years ago
Attachment #621105 - Flags: review?(gavin.sharp) → review+
https://hg.mozilla.org/integration/fx-team/rev/6016301d6d16
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 15
https://hg.mozilla.org/mozilla-central/rev/6016301d6d16
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]

Updated

5 years ago
Duplicate of this bug: 775325
You need to log in before you can comment on or make changes to this bug.