Closed Bug 1686804 Opened 3 years ago Closed 22 days ago

[sw-wr] Windows 10 talos xperf test doesn't appear to composite any frames

Categories

(Core :: Graphics: WebRender, defect, P3)

x86_64
Windows 10
defect

Tracking

()

RESOLVED INACTIVE

People

(Reporter: aosmond, Unassigned)

References

(Blocks 1 open bug)

Details

Investigation into why bug 1684170 got backed out lead me to conclude that we don't properly render frames for the Windows 10 talos xperf test with Software WebRender. Bug 1684170 allowed the test to actually use SW-WR at all -- before it would have been running with Basic because it was denied to lacking D3D11 ANGLE support. This initially appear to be because the test configuration appears to be running with a WARP device ID; however disabling the D3D11 compositor with SW-WR did not resolve the issue.

After bug 1679215 landed, the test began passing with SW-WR, presumably because we create a tiny scratch buffer to allow compositing to continue, allowing the higher level events to fire and the test to move forward:

https://searchfox.org/mozilla-central/rev/2a24205479519e70c0574929f45730d285141584/gfx/webrender_bindings/RenderCompositorSWGL.cpp#146

The reason we failed to get a mapped buffer is because we hit this error path:

https://searchfox.org/mozilla-central/rev/2a24205479519e70c0574929f45730d285141584/widget/windows/InProcessWinCompositorWidget.cpp#134

where the size of the surface is 0x0. Presumably this represents a cairo surface with an error code.

See Also: → 1679215
Depends on: 1684170
See Also: 1684170

Note that we see the same thing on the Windows 7 SW-WR xperf test, however it goes away after a half dozen attempts to create the DrawTarget.

It hits this path to create an opaque surface:

https://searchfox.org/mozilla-central/rev/2a24205479519e70c0574929f45730d285141584/widget/windows/InProcessWinCompositorWidget.cpp#126

It does not appear to fail creating the cairo surface as I initially suspected, that succeeds. We have a client size of 1024x772.

Status: NEW → RESOLVED
Closed: 22 days ago
Resolution: --- → INACTIVE
You need to log in before you can comment on or make changes to this bug.