Xidorn Quan [:xidorn] UTC+11 Assignee
	Description • 9 years ago

Per bug 1199519, this selector is no longer part of the spec. If we determined that it is not necessary to keep this selector, we should completely remove it.

It would be hard to maintain this selector both correctly and efficiently once we implement bug 1195213, which removes the hierarchy restriction.

	Kohei Yoshino [:kohei]
	Comment 1 • 9 years ago

Posted the site compatibility doc: https://www.fxsitecompat.com/en-US/docs/2015/moz-full-screen-ancestor-will-be-removed/

	Xidorn Quan [:xidorn] UTC+11 Assignee
	Comment 2 • 8 years ago

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

	Olli Pettay [:smaug][bugs@pettay.fi]
	Comment 3 • 8 years ago

Comment on attachment 8766276 [details]
Bug 1199529 - Remove :-moz-full-screen-ancestor pseudo-class as well as its state bit.

https://reviewboard.mozilla.org/r/61258/#review58136

Ah, we remove the state in this bug. So no need to add the assertion in the other bug.

	Pulsebot
	Comment 4 • 8 years ago

Pushed by xquan@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/e00ba8448c08
Remove :-moz-full-screen-ancestor pseudo-class as well as its state bit. r=smaug

	Pulsebot
	Comment 5 • 8 years ago

Pushed by xquan@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/515e35547dbe
followup - Remove unused function.

	Carsten Book [:Tomcat]
	Comment 6 • 8 years ago
bugherder

https://hg.mozilla.org/mozilla-central/rev/e00ba8448c08
https://hg.mozilla.org/mozilla-central/rev/515e35547dbe

	Kohei Yoshino [:kohei]
	Comment 7 • 8 years ago

Updated the site compat doc for Firefox 50: https://www.fxsitecompat.com/en-CA/docs/2016/moz-full-screen-ancestor-has-been-removed/

	Jean-Yves Perrier [:teoli]
	Comment 8 • 8 years ago

Updated:
https://developer.mozilla.org/en-US/docs/Web/CSS/:-moz-full-screen-ancestor
and
https://developer.mozilla.org/en-US/Firefox/Releases/50#CSS