Bug 1329051 - Remove Old Driver Workaround. r=jgilbert
authorChih-Yi Leu <cleu@mozilla.com>
Thu, 12 Jan 2017 19:42:00 +0100
changeset 329220 bf6a6ad3296eb520a318fec018ad3e11f861e189
parent 329219 bee517272e14ea1817b7b3f1ecfaa98b022ee242
child 329221 78d54111d9be3065681e0b21395a12e3c4485401
push id85665
push usercbook@mozilla.com
push dateFri, 13 Jan 2017 07:21:43 +0000
treeherdermozilla-inbound@bf6a6ad3296e [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjgilbert
bugs1329051
milestone53.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
Bug 1329051 - Remove Old Driver Workaround. r=jgilbert
dom/canvas/WebGLShader.cpp
--- 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.