author | Chih-Yi Leu <cleu@mozilla.com> |
Thu, 12 Jan 2017 19:42:00 +0100 | |
changeset 329220 | bf6a6ad3296eb520a318fec018ad3e11f861e189 |
parent 329219 | bee517272e14ea1817b7b3f1ecfaa98b022ee242 |
child 329221 | 78d54111d9be3065681e0b21395a12e3c4485401 |
push id | 85665 |
push user | cbook@mozilla.com |
push date | Fri, 13 Jan 2017 07:21:43 +0000 |
treeherder | mozilla-inbound@bf6a6ad3296e [default view] [failures only] |
perfherder | [talos] [build metrics] [platform microbench] (compared to previous push) |
reviewers | jgilbert |
bugs | 1329051 |
milestone | 53.0a1 |
first release with | nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
|
last release without | nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
|
--- a/dom/canvas/WebGLShader.cpp +++ b/dom/canvas/WebGLShader.cpp @@ -168,26 +168,16 @@ WebGLShader::ShaderSource(const nsAStrin if (!ValidateGLSLPreprocString(mContext, funcName, sourceWithoutComments)) return; // We checked that the source stripped of comments is in the // 7-bit ASCII range, so we can skip the NS_IsAscii() check. const NS_LossyConvertUTF16toASCII cleanSource(sourceWithoutComments); - if (mContext->gl->WorkAroundDriverBugs()) { - const size_t maxSourceLength = 0x3ffff; - if (cleanSource.Length() > maxSourceLength) { - mContext->ErrorInvalidValue("shaderSource: Source has more than %d" - " characters. (Driver workaround)", - maxSourceLength); - return; - } - } - if (PR_GetEnv("MOZ_WEBGL_DUMP_SHADERS")) { printf_stderr("////////////////////////////////////////\n"); printf_stderr("// MOZ_WEBGL_DUMP_SHADERS:\n"); // Wow - Roll Your Own Foreach-Lines because printf_stderr has a hard-coded // internal size, so long strings are truncated. const size_t maxChunkSize = 1024-1; // -1 for null-term.