1739930 - SanityTest.jsm triggers to create 2 compositor sessions

Status: RESOLVED FIXED

(Core :: General, enhancement)

Description

[Sotaro Ikeda [:sotaro]]

During debugging, I noticed that 2 compositor sessions were created for SanityTest.jsm.

Comment 1

[Sotaro Ikeda [:sotaro]]

SanityTest.jsm called one Services.ww.openWindow().

Comment 2

[Sotaro Ikeda [:sotaro]]

Parent widget was created by nsWindowWatcher::OpenWindow(). And then child widget was created since window was pop up.

The child widget does actual rendering of pop up. And CompositorSession of parent widget was created by nsWindow::OnPaint().

Comment 3

[Sotaro Ikeda [:sotaro]]

From nsDocumentViewer::ShouldAttachToTopLevel(), content rendering of popup is always done by child widget.

Comment 4

[Sotaro Ikeda [:sotaro]]

Attachment: Bug 1739930 - Disable CompositorSession in popup parent widget

Comment 5

[Sotaro Ikeda [:sotaro]]

Test result
https://treeherder.mozilla.org/jobs?repo=try&revision=8d9df396f96c78a8d6339a2c29ac7fc4e28b7dee

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/62db00bc54c9
Disable CompositorSession in popup parent widget r=gfx-reviewers,bradwerth

Comment 7

[Cristina Cozmuta (:CrissCozmuta)]

https://hg.mozilla.org/mozilla-central/rev/62db00bc54c9