1255659 - Add tag 'fullscreen' to all tests need Fullscreen API or Fullscreen Mode

Status: RESOLVED FIXED

(Testing :: General, defect)

Comment 1

[Xidorn Quan [:xidorn] UTC+11]

Attachment: MozReview Request: Bug 1255659 part 1 - Remove test_contextmenu.html which is no longer used.

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

Comment 2

[Xidorn Quan [:xidorn] UTC+11]

Attachment: MozReview Request: Bug 1255659 part 2 - Add 'fullscreen' tag to tests which ever put window into fullscreen.

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

Comment 3

[Xidorn Quan [:xidorn] UTC+11]

The first part was rewritten in bug 1243838, but the old test file was not removed properly.

I guess the second part probably doesn't really need review... but not sure... so request a review anyway.

Comment 4

[Olli Pettay [:smaug][[e-mail]]]

Comment on attachment 8729313 [details]
MozReview Request: Bug 1255659 part 2 - Add 'fullscreen' tag to tests which ever put window into fullscreen.

https://reviewboard.mozilla.org/r/39359/#review36129

rs+

Comment 5

[Jared Wein [:jaws] (please needinfo? me)]

Comment on attachment 8729312 [details]
MozReview Request: Bug 1255659 part 1 - Remove test_contextmenu.html which is no longer used.

https://reviewboard.mozilla.org/r/39357/#review36175

Comment 6

[Pulsebot]

https://hg.mozilla.org/integration/mozilla-inbound/rev/c3b28b2e4d0c
https://hg.mozilla.org/integration/mozilla-inbound/rev/c8d3ebfc4d36

Comment 7

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/mozilla-central/rev/c3b28b2e4d0c
https://hg.mozilla.org/mozilla-central/rev/c8d3ebfc4d36