User Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:47.0) Gecko/20100101 Firefox/47.0 Build ID: 20160604131506 Steps to reproduce: See simple repro example here: https://s3.amazonaws.com/storage2.interlude.fm/dev_temp/tomer/firefox_fullscreen_bug/ff_fullscreen.html Actual results: Using the prefixed Fullscreen API causes a warning to be issued: "Prefixed Fullscreen API is deprecated." However, there's no unprefixed API available. Expected results: Either unprefixed API should be made available, or no warning should be displayed.

The API is disabled by default, See Bug 1268749 If you want to use the API, set full-screen-api.unprefix.enabled to true in about:config.

(In reply to Alice0775 White from comment #1) > The API is disabled by default, See Bug 1268749 > > If you want to use the API, set full-screen-api.unprefix.enabled to true in > about:config. Obviously site authors can't force users to flip the pref. We should suppress the warning if the pref is false.

Looks like the WebIDL [Deprecated] annotation cannot be conditional :(

Review commit: https://reviewboard.mozilla.org/r/61526/diff/#index_header See other reviews: https://reviewboard.mozilla.org/r/61526/

I intentionally left string changes to make uplift easier.

Comment on attachment 8766743 [details] Bug 1280545 - Remove [Deprecated="PrefixedFullscreenAPI"] annotations until we are ready to unprefix fullscreen API. https://reviewboard.mozilla.org/r/61526/#review58368 Hmmm... okay...

Pushed by VYV03354@nifty.ne.jp: https://hg.mozilla.org/integration/autoland/rev/bedd6091246e Remove [Deprecated="PrefixedFullscreenAPI"] annotations until we are ready to unprefix fullscreen API. r=xidorn

Comment on attachment 8766743 [details] Bug 1280545 - Remove [Deprecated="PrefixedFullscreenAPI"] annotations until we are ready to unprefix fullscreen API. This one touches WebIDL and needs a DOM peer to review. (Though this isn't really a change to any API...)

sorry had to back this out since this was running into the hook that prevent webidl changes without dom peer reviews. Seems mozreview/autoland did not noticed this and i filed bug 1283798 for this

Backout by cbook@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/0f7988afce2c Backed out changeset bedd6091246e run into hg hook warning for webidl and backout for making autoland to m-c merge possible

Comment on attachment 8766743 [details] Bug 1280545 - Remove [Deprecated="PrefixedFullscreenAPI"] annotations until we are ready to unprefix fullscreen API. https://reviewboard.mozilla.org/r/61526/#review58632

Pushed by VYV03354@nifty.ne.jp: https://hg.mozilla.org/integration/autoland/rev/3ac069d1e022 Remove [Deprecated="PrefixedFullscreenAPI"] annotations until we are ready to unprefix fullscreen API. r=smaug,xidorn

Comment on attachment 8766743 [details] Bug 1280545 - Remove [Deprecated="PrefixedFullscreenAPI"] annotations until we are ready to unprefix fullscreen API. Approval Request Comment [Feature/regressing bug #]: 1268749 [User impact if declined]: Some advanced users and web authors will be confused by console messages. [Describe test coverage new/current, TreeHerder]: Manually tested [Risks and why]: Very low, only removes an auto-generated warning message. [String/UUID change made/needed]: none. strings are left intact.

Comment on attachment 8766743 [details] Bug 1280545 - Remove [Deprecated="PrefixedFullscreenAPI"] annotations until we are ready to unprefix fullscreen API. This patch removes an auto-generated warning message. Take it in 48 beta 6 and aurora.

has problems applying to beta: grafting 352707:3534349c6d78 "Bug 1280545 - Remove [Deprecated="PrefixedFullscreenAPI"] annotations until we are ready to unprefix fullscreen API. r=smaug r=xidorn, a=gchang" merging dom/webidl/Document.webidl warning: conflicts while merging dom/webidl/Document.webidl! (edit, then use 'hg resolve --mark') abort: unresolved conflicts, can't continue (use 'hg resolve' and 'hg graft --continue')