Improve usefulness of webglcontextcreationerror statusMessage

third_party/WebKit/LayoutTests/fast/canvas/webgl/webgl-error-response.html

 <!-- This is a test for crbug.com/412440, tests meaningful information in
 WebGL context creation error message. -->
 <script src="../../../resources/js-test.js"></script>
 <canvas id="webgl-canvas" width="200" height="200" class="nomargin"></canvas>
 <script>
 function main()
 {
   if (!window.testRunner) {
     testFailed("Requires window.testRunner");
   } else {
     initTest();
   }
 }
 
 function initTest() {
   var canvas = document.getElementById("webgl-canvas");
   testRunner.forceNextWebGLContextCreationToFail();
   canvas.addEventListener("webglcontextcreationerror", onContextCreationError, false);
   var gl = canvas.getContext("webgl");
   testRunner.dumpAsText();
 }
 
 function onContextCreationError(e) {
   var splitStatus = e.statusMessage.split(",");
-  var stringLocator = 1;
-  var status = true;
-  // Start iterating from 1 as , the initial message has no data.
+  var error =  "";
   for (var i = 1; i < splitStatus.length; i++) {
     var gpuInfoStr = splitStatus[i].split(" = ");
-    if (gpuInfoStr[stringLocator] == "")
-      status = false;
+    var key = gpuInfoStr[0].trim();
+    var value = gpuInfoStr[1];
+    if(key == "DEVICE" && (value == "0x0000" || value == ""))

[Ken Russell (switch to Gerrit), 2015/10/15 21:30:28]

Space after "if".

[sivag, 2015/10/16 18:29:20]

Done.

+      error = error +"DeviceID is null.";

[Ken Russell (switch to Gerrit), 2015/10/15 21:30:28]

Add space after "+".

[sivag, 2015/10/16 18:29:19]

Done.

+    if(key == "VENDOR" && (value == "0x0000" || value == ""))

[Ken Russell (switch to Gerrit), 2015/10/15 21:30:28]

Space after "if".

[sivag, 2015/10/16 18:29:19]

Done.

+      error = error + "VendorID is null.";
+    if((key != "GL_VENDOR" ||

[Ken Russell (switch to Gerrit), 2015/10/15 21:30:28]

Space after "if".

[sivag, 2015/10/16 18:29:20]

Done.

+      key != "GL_RENDERER" ||
+      key != "GL_VERSION" ||
+      key != "Sandboxed" ||
+      key != "Optimus" ||
+      key != "AMD switchable" ||
+      key != "Reset notification strategy" ||
+      key != "GPU process crash count" ||
+      key != "ErrorMessage") && (value == ""))

[Ken Russell (switch to Gerrit), 2015/10/15 21:30:28]

Please use an open-brace after the parenthesis here. The multi-line if-statement
is very hard to read and understand that there's a single-line statement after
it.

[sivag, 2015/10/16 18:29:19]

Done.

+        error = error + "Gpu data is incorrect.";

[Zhenyao Mo, 2015/10/15 22:30:37]

This condition is true if key == "DEVICE" or "VENDOR" and value is "".

I don't think that's intentional, is it?

[sivag, 2015/10/16 18:29:19]

Done.

[sivag, 2015/10/16 18:29:20]

i corrected it, it was supposed to check if one of the key other than vendor and
device has a blank value.

   }
-  if(status)
+  if(error.length)

[Ken Russell (switch to Gerrit), 2015/10/15 21:30:28]

Space after "if".

[sivag, 2015/10/16 18:29:19]

Done.

+    testFailed("webglcontextcreationerror test failed" + error);
+  else
     testPassed("Status message displayed for webglcontextcreationerror");
-  else
-    testFailed("Context creation error test failed");
+
 }
 main();
 </script>