1736479 - Make gfx SanityTest.jsm work again

Status: RESOLVED FIXED

(Core :: Graphics, task)

Description

[Sotaro Ikeda [:sotaro]]

SanityTest.jsm does nothing with WebRender. SanityTest.jsm could be removed.
https://searchfox.org/mozilla-central/rev/483cfde5a54f6c1cd94c6295564993aeb4f10980/toolkit/components/gfx/SanityTest.jsm#203

Comment 1

[Sotaro Ikeda [:sotaro]]

Attachment: Bug 1736479 - Remove SanityTest.jsm

Comment 2

[Sotaro Ikeda [:sotaro]]

:jrmuizel, commented if we could re-enable the original intention of SanityTest. The SanityTest was originally used for checking if hardware video decoder works. It was added by Bug 1156135.
The problem is that taking screenshot with DirectComposition could take very long time(Bug 1615590, Bug 1602643)

Comment 3

[Sotaro Ikeda [:sotaro]]

If we want to re-enable SanityTest.jsm test, we need to disable native compositor(DirectComposition) on the testing window, since taking snapshot of native compositor is very slow on Windows.

CompositorOptions could be used to notice to disable native compositor(DirectComposition).

But we do not have a good way to notice it from SanityTest.jsm to gecko via nsWindowWatcher::OpenWindow(). Feature might be used for it, but chrome flag is already fully used

• https://searchfox.org/mozilla-central/rev/7241c1f9e94377b70b49cf702264269779c5c322/toolkit/components/browser/nsIWebBrowserChrome.idl#28

Comment 4

[Sotaro Ikeda [:sotaro]]

Attachment: Bug 1736479 - Make gfx SanityTest.jsm work again

Comment 5

[Sotaro Ikeda [:sotaro]]

test result
https://treeherder.mozilla.org/jobs?repo=try&revision=b5b84087f23d23d0214dcf1744fcd934e64287b9

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/d5b01209cf78
Make gfx SanityTest.jsm work again r=gfx-reviewers,bradwerth

Comment 7

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/d5b01209cf78