Open Bug 808136 Opened 9 years ago Updated 8 months 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
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]
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.