Seth Fowler [:seth] [:s2h] Assignee
	Description • 9 years ago

We shouldn't assume all frames are visible when the top-level content document is a XUL document. This is true for mochitests, which means that mochitests are not running in an environment that is comparable to a real web page.

	Seth Fowler [:seth] [:s2h] Assignee
	Comment 1 • 9 years ago

Here's the patch.

	Timothy Nikkel (:tnikkel)
	Comment 2 • 9 years ago

Background: I originally included XUL docs to make it easier to turn on image visible with less chance of regressions. There wasn't anything specifically blocking turning it on for xul docs, just little upside in enabling it there.

	Seth Fowler [:seth] [:s2h] Assignee
	Comment 3 • 9 years ago

https://treeherder.mozilla.org/#/jobs?repo=try&revision=8903985055d7

	Seth Fowler [:seth] [:s2h] Assignee
	Comment 4 • 9 years ago

(In reply to Timothy Nikkel (:tn) from comment #2)
> Background: I originally included XUL docs to make it easier to turn on
> image visible with less chance of regressions. There wasn't anything
> specifically blocking turning it on for xul docs, just little upside in
> enabling it there.

Thanks for the review! Based on the try results, it looks like everything went ok.

	Timothy Nikkel (:tnikkel)
	Comment 5 • 9 years ago

This seems like something that would be good to get landed ASAP, as then any changes it causes can be easily tracked and noticed sooner.

	Seth Fowler [:seth] [:s2h] Assignee
	Comment 6 • 8 years ago

https://hg.mozilla.org/integration/mozilla-inbound/rev/5209d6b9fe6e0e39f1b4c46b0642aae3ece65973
Bug 1223747 - Don't assume all frames are visible in XUL documents. r=tn

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

https://hg.mozilla.org/mozilla-central/rev/5209d6b9fe6e