Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-07-24 15:50:21 +01:00
Code Issues Projects Releases Packages Wiki Activity

Improved the WebGL2 framebuffer render validation

Browse source 
It seems `MissingColorAttachment` should be returned only when
all of the color attachments are missing.
This commit is contained in:
Mátyás Mustoha 2020-03-05 12:01:22 +01:00
parent 2ed5694194
commit 5cb6eb2744
3 changed files with 1 additions and 23 deletions
Download patch file Download diff file Expand all files Collapse all files

2
components/script/dom/webglframebuffer.rs
View file

@ -468,7 +468,7 @@ impl WebGLFramebuffer {
return CompleteForRendering::Complete; return CompleteForRendering::Complete;
} }
if self.colors.iter().any(|att| att.borrow().is_none()) { if self.colors.iter().all(|att| att.borrow().is_none()) {
return CompleteForRendering::MissingColorAttachment; return CompleteForRendering::MissingColorAttachment;
} }

7
tests/wpt/webgl/meta/conformance2/reading/read-pixels-from-rgb8-into-pbo-bug.html.ini
View file

@ -1,7 +0,0 @@
[read-pixels-from-rgb8-into-pbo-bug.html]
[WebGL test #2: getError expected: NO_ERROR. Was INVALID_OPERATION : Tests should complete without gl errors]
expected: FAIL
[WebGL test #1: Expected in pixel 0: [255,0,0,255\], got: 0,0,0,0]
expected: FAIL

15
tests/wpt/webgl/meta/conformance2/rendering/framebuffer-texture-changing-base-level.html.ini
View file

@ -8,24 +8,9 @@
[WebGL test #8: getError expected: NO_ERROR. Was INVALID_ENUM : Setup framebuffer with texture should succeed.] [WebGL test #8: getError expected: NO_ERROR. Was INVALID_ENUM : Setup framebuffer with texture should succeed.]
expected: FAIL expected: FAIL
[WebGL test #3: getError expected: NO_ERROR. Was INVALID_OPERATION : Clearing the texture level 0 to green should succeed.]
expected: FAIL
[WebGL test #9: getError expected: NO_ERROR. Was INVALID_OPERATION : Clearing the texture level 2 to green should succeed.]
expected: FAIL
[WebGL test #6: getError expected: NO_ERROR. Was INVALID_OPERATION : Clearing the texture level 1 to green should succeed.]
expected: FAIL
[WebGL test #14: should be green\nat (0, 0) expected: 0,255,0,255 was 0,0,0,0]
expected: FAIL
[WebGL test #13: getError expected: NO_ERROR. Was INVALID_ENUM : Drawing the texture to default framebuffer with base level 0 should succeed.] [WebGL test #13: getError expected: NO_ERROR. Was INVALID_ENUM : Drawing the texture to default framebuffer with base level 0 should succeed.]
expected: FAIL expected: FAIL
[WebGL test #2: getError expected: NO_ERROR. Was INVALID_ENUM : Setup framebuffer with texture should succeed.] [WebGL test #2: getError expected: NO_ERROR. Was INVALID_ENUM : Setup framebuffer with texture should succeed.]
expected: FAIL expected: FAIL
[WebGL test #12: getError expected: NO_ERROR. Was INVALID_OPERATION : Clearing the texture level 3 to green should succeed.]
expected: FAIL