Open Bug 808136 Opened 12 years ago Updated 3 years ago

fullscreen window.innerHeight broken on OSX

Categories

(Core :: DOM: Core & HTML, defect, P5)

All
macOS
defect

Tracking

()

Tracking Status
firefox19 - affected

People

(Reporter: vlad, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [games:p?])

Attachments

(1 file)

When going fullscreen on a Mac with a 1680x1050 display, window.innerHeight reports 974, and not 1050.  On Win7, on a 1920x1200 display, innerHeight is correctly 1200.

There's a simple testcase for this at http://people.mozilla.com/~vladimir/misc/fs.html .

Note that bug 712225 might be relevant -- that also has a similar testcase at http://diogogmt.github.com/mozilla-central/windowInnerOuterHeight.html (move the mouse to get the numbers to update).

We should fix this and figure out what it affects, as it can have a pretty huge impact on gaming viability (since fullscreen elements need to be resized to get 1:1 pixel ratio).
Attached file Testcase (deleted) —
What you're seeing is the "mozfullscreenchange" event firing before the transition to fullscreen is complete. Specifically before the window manager believes the chrome menubar is hidden.

If you check window.inner* after the "mozfullscreenchange" event (as this attached testcase does) you'll see we return the correct value at some time after "mozfullscreenchange".

I was working in bug 756334 to address delay dispatching "mozfullscreenchange" until the transition was complete, but got bogged down with mochitest timeouts on tinderbox. It may be a month or two before I get back to working on that...
Is this a regression? Why should we track for FF19 specifically?
This is not a regression. This issue has existed for as long as we've had a fullscreen API.
Whiteboard: [games:p1] → [games:p2]
Blocks: gecko-games
Whiteboard: [games:p2] → [games:p?]

Bulk-downgrade of unassigned, 4 years untouched DOM/Storage bugs' priority.

If you have reason to believe this is wrong (especially for the severity), please write a comment and ni :jstutte.

Severity: normal → S4
Priority: -- → P5
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: