Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(589)

Issues Search
    My Issues | Starred     Open | Closed | All

Unified Diff: third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp

Issue 2698863002: Implement WebGL2's uniform*() with sub source. (Closed)
Patch Set: Created 3 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.h ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
diff --git a/third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp b/third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
index e4a0bf40951a7ffc29e0c81ce372d7ef43edff3a..f1f9aac7eb804f0bcaf79aaabaec937ea92e6ad2 100644
--- a/third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
+++ b/third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.cpp
@@ -5639,7 +5639,7 @@ void WebGLRenderingContextBase::uniform1fv(const WebGLUniformLocation* location,
const FlexibleFloat32ArrayView& v) {
if (isContextLost() ||
!validateUniformParameters<WTF::Float32Array>("uniform1fv", location, v,
- 1))
+ 1, 0, v.length()))
return;
contextGL()->Uniform1fv(location->location(), v.length(),
@@ -5649,7 +5649,8 @@ void WebGLRenderingContextBase::uniform1fv(const WebGLUniformLocation* location,
void WebGLRenderingContextBase::uniform1fv(const WebGLUniformLocation* location,
Vector<GLfloat>& v) {
if (isContextLost() ||
- !validateUniformParameters("uniform1fv", location, v.data(), v.size(), 1))
+ !validateUniformParameters("uniform1fv", location, v.data(), v.size(), 1,
+ 0, v.size()))
return;
contextGL()->Uniform1fv(location->location(), v.size(), v.data());
@@ -5672,7 +5673,8 @@ void WebGLRenderingContextBase::uniform1i(const WebGLUniformLocation* location,
void WebGLRenderingContextBase::uniform1iv(const WebGLUniformLocation* location,
const FlexibleInt32ArrayView& v) {
if (isContextLost() ||
- !validateUniformParameters<WTF::Int32Array>("uniform1iv", location, v, 1))
+ !validateUniformParameters<WTF::Int32Array>("uniform1iv", location, v, 1,
+ 0, v.length()))
return;
contextGL()->Uniform1iv(location->location(), v.length(),
@@ -5682,7 +5684,8 @@ void WebGLRenderingContextBase::uniform1iv(const WebGLUniformLocation* location,
void WebGLRenderingContextBase::uniform1iv(const WebGLUniformLocation* location,
Vector<GLint>& v) {
if (isContextLost() ||
- !validateUniformParameters("uniform1iv", location, v.data(), v.size(), 1))
+ !validateUniformParameters("uniform1iv", location, v.data(), v.size(), 1,
+ 0, v.size()))
return;
contextGL()->Uniform1iv(location->location(), v.size(), v.data());
@@ -5707,7 +5710,7 @@ void WebGLRenderingContextBase::uniform2fv(const WebGLUniformLocation* location,
const FlexibleFloat32ArrayView& v) {
if (isContextLost() ||
!validateUniformParameters<WTF::Float32Array>("uniform2fv", location, v,
- 2))
+ 2, 0, v.length()))
return;
contextGL()->Uniform2fv(location->location(), v.length() >> 1,
@@ -5717,7 +5720,8 @@ void WebGLRenderingContextBase::uniform2fv(const WebGLUniformLocation* location,
void WebGLRenderingContextBase::uniform2fv(const WebGLUniformLocation* location,
Vector<GLfloat>& v) {
if (isContextLost() ||
- !validateUniformParameters("uniform2fv", location, v.data(), v.size(), 2))
+ !validateUniformParameters("uniform2fv", location, v.data(), v.size(), 2,
+ 0, v.size()))
return;
contextGL()->Uniform2fv(location->location(), v.size() >> 1, v.data());
@@ -5741,7 +5745,8 @@ void WebGLRenderingContextBase::uniform2i(const WebGLUniformLocation* location,
void WebGLRenderingContextBase::uniform2iv(const WebGLUniformLocation* location,
const FlexibleInt32ArrayView& v) {
if (isContextLost() ||
- !validateUniformParameters<WTF::Int32Array>("uniform2iv", location, v, 2))
+ !validateUniformParameters<WTF::Int32Array>("uniform2iv", location, v, 2,
+ 0, v.length()))
return;
contextGL()->Uniform2iv(location->location(), v.length() >> 1,
@@ -5751,7 +5756,8 @@ void WebGLRenderingContextBase::uniform2iv(const WebGLUniformLocation* location,
void WebGLRenderingContextBase::uniform2iv(const WebGLUniformLocation* location,
Vector<GLint>& v) {
if (isContextLost() ||
- !validateUniformParameters("uniform2iv", location, v.data(), v.size(), 2))
+ !validateUniformParameters("uniform2iv", location, v.data(), v.size(), 2,
+ 0, v.size()))
return;
contextGL()->Uniform2iv(location->location(), v.size() >> 1, v.data());
@@ -5777,7 +5783,7 @@ void WebGLRenderingContextBase::uniform3fv(const WebGLUniformLocation* location,
const FlexibleFloat32ArrayView& v) {
if (isContextLost() ||
!validateUniformParameters<WTF::Float32Array>("uniform3fv", location, v,
- 3))
+ 3, 0, v.length()))
return;
contextGL()->Uniform3fv(location->location(), v.length() / 3,
@@ -5787,7 +5793,8 @@ void WebGLRenderingContextBase::uniform3fv(const WebGLUniformLocation* location,
void WebGLRenderingContextBase::uniform3fv(const WebGLUniformLocation* location,
Vector<GLfloat>& v) {
if (isContextLost() ||
- !validateUniformParameters("uniform3fv", location, v.data(), v.size(), 3))
+ !validateUniformParameters("uniform3fv", location, v.data(), v.size(), 3,
+ 0, v.size()))
return;
contextGL()->Uniform3fv(location->location(), v.size() / 3, v.data());
@@ -5812,7 +5819,8 @@ void WebGLRenderingContextBase::uniform3i(const WebGLUniformLocation* location,
void WebGLRenderingContextBase::uniform3iv(const WebGLUniformLocation* location,
const FlexibleInt32ArrayView& v) {
if (isContextLost() ||
- !validateUniformParameters<WTF::Int32Array>("uniform3iv", location, v, 3))
+ !validateUniformParameters<WTF::Int32Array>("uniform3iv", location, v, 3,
+ 0, v.length()))
return;
contextGL()->Uniform3iv(location->location(), v.length() / 3,
@@ -5822,7 +5830,8 @@ void WebGLRenderingContextBase::uniform3iv(const WebGLUniformLocation* location,
void WebGLRenderingContextBase::uniform3iv(const WebGLUniformLocation* location,
Vector<GLint>& v) {
if (isContextLost() ||
- !validateUniformParameters("uniform3iv", location, v.data(), v.size(), 3))
+ !validateUniformParameters("uniform3iv", location, v.data(), v.size(), 3,
+ 0, v.size()))
return;
contextGL()->Uniform3iv(location->location(), v.size() / 3, v.data());
@@ -5849,7 +5858,7 @@ void WebGLRenderingContextBase::uniform4fv(const WebGLUniformLocation* location,
const FlexibleFloat32ArrayView& v) {
if (isContextLost() ||
!validateUniformParameters<WTF::Float32Array>("uniform4fv", location, v,
- 4))
+ 4, 0, v.length()))
return;
contextGL()->Uniform4fv(location->location(), v.length() >> 2,
@@ -5859,7 +5868,8 @@ void WebGLRenderingContextBase::uniform4fv(const WebGLUniformLocation* location,
void WebGLRenderingContextBase::uniform4fv(const WebGLUniformLocation* location,
Vector<GLfloat>& v) {
if (isContextLost() ||
- !validateUniformParameters("uniform4fv", location, v.data(), v.size(), 4))
+ !validateUniformParameters("uniform4fv", location, v.data(), v.size(), 4,
+ 0, v.size()))
return;
contextGL()->Uniform4fv(location->location(), v.size() >> 2, v.data());
@@ -5885,7 +5895,8 @@ void WebGLRenderingContextBase::uniform4i(const WebGLUniformLocation* location,
void WebGLRenderingContextBase::uniform4iv(const WebGLUniformLocation* location,
const FlexibleInt32ArrayView& v) {
if (isContextLost() ||
- !validateUniformParameters<WTF::Int32Array>("uniform4iv", location, v, 4))
+ !validateUniformParameters<WTF::Int32Array>("uniform4iv", location, v, 4,
+ 0, v.length()))
return;
contextGL()->Uniform4iv(location->location(), v.length() >> 2,
@@ -5895,7 +5906,8 @@ void WebGLRenderingContextBase::uniform4iv(const WebGLUniformLocation* location,
void WebGLRenderingContextBase::uniform4iv(const WebGLUniformLocation* location,
Vector<GLint>& v) {
if (isContextLost() ||
- !validateUniformParameters("uniform4iv", location, v.data(), v.size(), 4))
+ !validateUniformParameters("uniform4iv", location, v.data(), v.size(), 4,
+ 0, v.size()))
return;
contextGL()->Uniform4iv(location->location(), v.size() >> 2, v.data());
@@ -5907,7 +5919,7 @@ void WebGLRenderingContextBase::uniformMatrix2fv(
DOMFloat32Array* v) {
if (isContextLost() ||
!validateUniformMatrixParameters("uniformMatrix2fv", location, transpose,
- v, 4))
+ v, 4, 0, v->length()))
return;
contextGL()->UniformMatrix2fv(location->location(), v->length() >> 2,
transpose, v->data());
@@ -5919,7 +5931,7 @@ void WebGLRenderingContextBase::uniformMatrix2fv(
Vector<GLfloat>& v) {
if (isContextLost() ||
!validateUniformMatrixParameters("uniformMatrix2fv", location, transpose,
- v.data(), v.size(), 4))
+ v.data(), v.size(), 4, 0, v.size()))
return;
contextGL()->UniformMatrix2fv(location->location(), v.size() >> 2, transpose,
v.data());
@@ -5931,7 +5943,7 @@ void WebGLRenderingContextBase::uniformMatrix3fv(
DOMFloat32Array* v) {
if (isContextLost() ||
!validateUniformMatrixParameters("uniformMatrix3fv", location, transpose,
- v, 9))
+ v, 9, 0, v->length()))
return;
contextGL()->UniformMatrix3fv(location->location(), v->length() / 9,
transpose, v->data());
@@ -5943,7 +5955,7 @@ void WebGLRenderingContextBase::uniformMatrix3fv(
Vector<GLfloat>& v) {
if (isContextLost() ||
!validateUniformMatrixParameters("uniformMatrix3fv", location, transpose,
- v.data(), v.size(), 9))
+ v.data(), v.size(), 9, 0, v.size()))
return;
contextGL()->UniformMatrix3fv(location->location(), v.size() / 9, transpose,
v.data());
@@ -5955,7 +5967,7 @@ void WebGLRenderingContextBase::uniformMatrix4fv(
DOMFloat32Array* v) {
if (isContextLost() ||
!validateUniformMatrixParameters("uniformMatrix4fv", location, transpose,
- v, 16))
+ v, 16, 0, v->length()))
return;
contextGL()->UniformMatrix4fv(location->location(), v->length() >> 4,
transpose, v->data());
@@ -5967,7 +5979,7 @@ void WebGLRenderingContextBase::uniformMatrix4fv(
Vector<GLfloat>& v) {
if (isContextLost() ||
!validateUniformMatrixParameters("uniformMatrix4fv", location, transpose,
- v.data(), v.size(), 16))
+ v.data(), v.size(), 16, 0, v.size()))
return;
contextGL()->UniformMatrix4fv(location->location(), v.size() >> 4, transpose,
v.data());
@@ -7199,37 +7211,13 @@ bool WebGLRenderingContextBase::validateCapability(const char* functionName,
bool WebGLRenderingContextBase::validateUniformParameters(
const char* functionName,
const WebGLUniformLocation* location,
- DOMFloat32Array* v,
- GLsizei requiredMinSize) {
- if (!v) {
- synthesizeGLError(GL_INVALID_VALUE, functionName, "no array");
- return false;
- }
- return validateUniformMatrixParameters(
- functionName, location, false, v->data(), v->length(), requiredMinSize);
-}
-
-bool WebGLRenderingContextBase::validateUniformParameters(
- const char* functionName,
- const WebGLUniformLocation* location,
- DOMInt32Array* v,
- GLsizei requiredMinSize) {
- if (!v) {
- synthesizeGLError(GL_INVALID_VALUE, functionName, "no array");
- return false;
- }
- return validateUniformMatrixParameters(
- functionName, location, false, v->data(), v->length(), requiredMinSize);
-}
-
-bool WebGLRenderingContextBase::validateUniformParameters(
- const char* functionName,
- const WebGLUniformLocation* location,
void* v,
GLsizei size,
- GLsizei requiredMinSize) {
+ GLsizei requiredMinSize,
+ GLuint srcOffset,
+ GLuint srcLength) {
return validateUniformMatrixParameters(functionName, location, false, v, size,
- requiredMinSize);
+ requiredMinSize, srcOffset, srcLength);
}
bool WebGLRenderingContextBase::validateUniformMatrixParameters(
@@ -7237,14 +7225,16 @@ bool WebGLRenderingContextBase::validateUniformMatrixParameters(
const WebGLUniformLocation* location,
GLboolean transpose,
DOMFloat32Array* v,
- GLsizei requiredMinSize) {
+ GLsizei requiredMinSize,
+ GLuint srcOffset,
+ GLuint srcLength) {
if (!v) {
synthesizeGLError(GL_INVALID_VALUE, functionName, "no array");
return false;
}
return validateUniformMatrixParameters(functionName, location, transpose,
v->data(), v->length(),
- requiredMinSize);
+ requiredMinSize, srcOffset, srcLength);
}
bool WebGLRenderingContextBase::validateUniformMatrixParameters(
@@ -7253,7 +7243,10 @@ bool WebGLRenderingContextBase::validateUniformMatrixParameters(
GLboolean transpose,
void* v,
GLsizei size,
- GLsizei requiredMinSize) {
+ GLsizei requiredMinSize,
+ GLuint srcOffset,
+ GLuint srcLength) {
+ DCHECK(size >= 0 && requiredMinSize > 0);
if (!location)
return false;
if (location->program() != m_currentProgram) {
@@ -7269,7 +7262,20 @@ bool WebGLRenderingContextBase::validateUniformMatrixParameters(
synthesizeGLError(GL_INVALID_VALUE, functionName, "transpose not FALSE");
return false;
}
- if (size < requiredMinSize || (size % requiredMinSize)) {
+ if (srcOffset >= static_cast<GLuint>(size)) {
+ synthesizeGLError(GL_INVALID_VALUE, functionName, "invalid srcOffset");
+ return false;
+ }
+ GLsizei actualSize = size - srcOffset;
+ if (srcLength > 0) {
+ if (srcLength > static_cast<GLuint>(actualSize)) {
+ synthesizeGLError(GL_INVALID_VALUE, functionName,
+ "invalid srcOffset + srcLength");
+ return false;
+ }
+ actualSize = srcLength;
+ }
+ if (actualSize < requiredMinSize || (actualSize % requiredMinSize)) {
synthesizeGLError(GL_INVALID_VALUE, functionName, "invalid size");
return false;
}
« no previous file with comments | « third_party/WebKit/Source/modules/webgl/WebGLRenderingContextBase.h ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698