New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Test for signed int attribs conversion #2488
Conversation
The test verifies that a WebGL 2 implementation converts singed integer vertex attributes using zero-presering rule as it's required to the GLES 3 spec.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this test and sorry for taking so long to review it. It looks OK; tested locally. I'll update #2486 in response. Could you please make a few small changes?
@@ -1,4 +1,5 @@ | |||
gl-vertex-attrib.html | |||
gl-vertex-attrib-i-render.html | |||
gl-vertex-attrib-normalized-int.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add --min-version 2.0.1
@@ -0,0 +1,98 @@ | |||
<!-- | |||
/* | |||
** Copyright (c) 2015 The Khronos Group Inc. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please update to 2017
<!DOCTYPE html> | ||
<html> | ||
<head> | ||
<title>WebGL 2 Normalized Vertex Attrbiutes Conformance Test</title> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: Attributes
}()); | ||
</script> | ||
<script> | ||
description('Verify that conversion of normalized singed int attributes to floats uses zero-preserving rule.'); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo: signed
<html> | ||
<head> | ||
<title>WebGL 2 Normalized Vertex Attrbiutes Conformance Test</title> | ||
<meta charset="utf-8"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you also add:
<link rel="stylesheet" href="../../resources/js-test-style.css"/>
Review fixes.
Fixed:) |
LGTM |
The test verifies that a WebGL 2 implementation converts singed integer
vertex attributes using zero-presering rule as it's required to the
GLES 3 spec.
See also #2486.
/cc @kenrussell @zhenyao