Open Bug 1289637 Opened 3 years ago Updated 3 years ago

800 instances of "We can't support non-premultiplied alpha for video!: '(aSurfaceFlags & SFE_PREFER_NO_PREMULTIPLY_ALPHA) == 0'" emitted from layout/base/nsLayoutUtils.cpp during linux64 debug testing

Categories

(Core :: Graphics, defect, P4, minor)

defect

Tracking

()

People

(Reporter: erahm, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [gfx-noted])

> 777 WARNING: We can't support non-premultiplied alpha for video!: '(aSurfaceFlags & SFE_PREFER_NO_PREMULTIPLY_ALPHA) == 0', file layout/base/nsLayoutUtils.cpp, line 7494

This warning [1] shows up in the following test suites:

>    380 - desktop-test-linux64/debug-mochitest-webgl-e10s gl
>    373 - desktop-test-linux64/debug-mochitest-webgl gl
>     12 - desktop-test-linux64/debug-mochitest-gpu gpu
>     12 - desktop-test-linux64/debug-mochitest-gpu-e10s gpu

It shows up in 22 tests. A few of the most prevalent:

>     48 -        dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgba-rgba-unsigned_byte.html
>     48 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgba-rgba-unsigned_byte.html
>     48 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgba-rgba-unsigned_short_5_5_5_1.html
>     48 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgb-rgb-unsigned_short_5_6_5.html
>     48 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgb-rgb-unsigned_byte.html
>     48 -        dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgba-rgba-unsigned_short_5_5_5_1.html
>     48 -        dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgba-rgba-unsigned_short_4_4_4_4.html
>     48 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgba-rgba-unsigned_short_4_4_4_4.html
>     48 -        dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgb-rgb-unsigned_short_5_6_5.html
>     48 -        dom/canvas/test/webgl-conf/generated/test_conformance__extensions__oes-texture-half-float-with-video.html

[1] https://hg.mozilla.org/mozilla-central/annotate/81e206ee8ba7/layout/base/nsLayoutUtils.cpp#l7494
:jgilbert it looks like the uptick in warnings started when landing the latest webgl conformance tests in bug 1277424. The warning is actually on the layout side, but it primarily occurs during webgl tests.
Blocks: webgl-104-ci
Flags: needinfo?(jgilbert)
The full list of tests:

>     48 -        dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgba-rgba-unsigned_byte.html
>     48 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgba-rgba-unsigned_byte.html
>     48 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgba-rgba-unsigned_short_5_5_5_1.html
>     48 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgb-rgb-unsigned_short_5_6_5.html
>     48 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgb-rgb-unsigned_byte.html
>     48 -        dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgba-rgba-unsigned_short_5_5_5_1.html
>     48 -        dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgba-rgba-unsigned_short_4_4_4_4.html
>     48 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgba-rgba-unsigned_short_4_4_4_4.html
>     48 -        dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgb-rgb-unsigned_short_5_6_5.html
>     48 -        dom/canvas/test/webgl-conf/generated/test_conformance__extensions__oes-texture-half-float-with-video.html
>     48 -        dom/canvas/test/webgl-conf/generated/test_conformance__extensions__oes-texture-float-with-video.html
>     48 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__extensions__oes-texture-float-with-video.html
>     48 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__extensions__oes-texture-half-float-with-video.html
>     48 -        dom/canvas/test/webgl-conf/generated/test_conformance__textures__video__tex-2d-rgb-rgb-unsigned_byte.html
>     18 -        dom/canvas/test/webgl-conf/generated/test_conformance__textures__misc__texture-upload-size.html
>     18 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__more__conformance__quickCheckAPI-S_V.html
>     18 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__textures__misc__texture-upload-size.html
>     12 -        dom/canvas/test/crossorigin/test_video_crossorigin.html
>     12 - [e10s] dom/canvas/test/crossorigin/test_video_crossorigin.html
>     11 -        dom/canvas/test/webgl-conf/generated/test_conformance__more__conformance__quickCheckAPI-S_V.html
>      8 -        dom/canvas/test/webgl-conf/generated/test_conformance__textures__misc__texture-npot-video.html
>      8 - [e10s] dom/canvas/test/webgl-conf/generated/test_conformance__textures__misc__texture-npot-video.html
Whiteboard: [gfx-noted]
These should all be spurious, since we're passing the tests.
Callers of this should be vetted and the warning removed. The docs should make it clear that this is a preference from the caller, not a requirement.
Severity: normal → minor
Component: Canvas: WebGL → Graphics
Flags: needinfo?(jgilbert)
Priority: -- → P4
You need to log in before you can comment on or make changes to this bug.