701714 - (full-screen-api-sm) Add support for DOM full-screen

Status: NEW

Description

[Jens Hatlak (:InvisibleSmiley)]

Once we have proper Full Screen support (i.e. with hiding all bars by default, which is bug 610509) we should port and enable DOM full-screen support.

Pref:
full-screen-api.enabled

Test:
http://blog.pearce.org.nz/2011/11/firefoxs-html-full-screen-api-enabled.html

m-c changesets:
http://hg.mozilla.org/mozilla-central/rev/64e10f2dfc60 (Bug 684625)
http://hg.mozilla.org/mozilla-central/rev/e7e0950aec83 (Bug 685402)
http://hg.mozilla.org/mozilla-central/rev/de9859b56d6e (Bug 699885)
http://hg.mozilla.org/mozilla-central/rev/f7355dc166cd (Bug 700151)

Comment 1

[Jens Hatlak (:InvisibleSmiley)]

Note that this will have l10n impact due to the domFullScreenWarning.label entity addition.

Comment 2

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch on top of bug 610509 patch v1

Not requesting review until bug 610509 has been finalized.

I don't know why, but unlike with FF, the "ESC" warning message doesn't appear when triggering DOM full-screen (see comment 0 for an example). Maybe a layering issue. :-/

Karsten, you may want to move some of my fixes to your patch, namely:
* renaming of function parameters (event, forceHide)
* moving of opening curly bracket to new line (toggle, cleanup) 
* removing of curly brackets for single-command if/else (aData, aShow)

Comment 3

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch on top of bug 610509 patch v1a

This is just a s/min-// update (porting bug 701618).

Comment 4

[Jens Hatlak (:InvisibleSmiley)]

Note: We should not change the omFullScreenWarning.label string (entity value) in order to ease localizers' job (i.e. they can re-use the string from FF) in case we port this to Aurora.

Comment 5

[Philip Chee]

Do we need:
https://hg.mozilla.org/mozilla-central/rev/816d3aca13c4
From:
Bug 704011 - mozfullscreenchange event handler assumes target is an element not a document

Comment 6

[Stefan [:stefanh]]

Comment on attachment 574288 [details] [diff] [review]
patch on top of bug 610509 patch v1a

diff --git a/suite/themes/classic/navigator/navigator.css b/suite/themes/classic/navigator/navigator.css

This file is not used on Mac, so you'll need to add stuff (from pinstripe) to suite/themes/classic/mac/navigator.css.

Comment 7

[Jens Hatlak (:InvisibleSmiley)]

Attachment: patch on top of bug 610509 patch v1b

(In reply to Philip Chee from comment #5)
> Do we need:
> https://hg.mozilla.org/mozilla-central/rev/816d3aca13c4
> From:
> Bug 704011 - mozfullscreenchange event handler assumes target is an element
> not a document

Yes. Citing from that bug:
"Bug 703079 changed the mozfullscreenchange event target is always the owning document, rather than the element upon entering full-screen and document on exiting.

The mozfullscreenchange handler in browser.js needs to be updated to reflect that, though the existing code still works as is."

(In reply to Stefan [:stefanh] from comment #6)
> b/suite/themes/classic/navigator/navigator.css
> 
> This file is not used on Mac, so you'll need to add stuff (from pinstripe)
> to suite/themes/classic/mac/navigator.css.

So true. Thanks for keeping a sharp eye, I don't easily spot these without a way to test it.

Comment 8

[[e-mail]]

Nice, the pref actually works with the existing "full" screen code too.

Comment 9

[Jens Hatlak (:InvisibleSmiley)]

Nothing happening anywhere (esp. on the base bug), so since in fact I'm not actively working on this, I can just as well unassign myself for the time being.