Closed Bug 751955 Opened 13 years ago Closed 13 years ago

Lion fullscreen check should be more versatile

Categories

(Firefox :: General, defect)

All
macOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Firefox 15

People

(Reporter: zpao, Assigned: zpao)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

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)
Attached patch Patch v0.1Splinter Review
Assignee: nobody → paul
Status: NEW → ASSIGNED
Attachment #621105 - Flags: review?(gavin.sharp)
Attachment #621105 - Flags: review?(gavin.sharp) → review+
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 15
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: