Improve usefulness of webglcontextcreationerror statusMessage

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

Index: third_party/WebKit/LayoutTests/fast/canvas/webgl/webgl-error-response.html
diff --git a/third_party/WebKit/LayoutTests/fast/canvas/webgl/webgl-error-response.html b/third_party/WebKit/LayoutTests/fast/canvas/webgl/webgl-error-response.html
index e866a237d28b4586468b4b98cd73958ea3459593..3bee34ca8594119b056bfd1a1d0bb84daeea82a2 100644
--- a/third_party/WebKit/LayoutTests/fast/canvas/webgl/webgl-error-response.html
+++ b/third_party/WebKit/LayoutTests/fast/canvas/webgl/webgl-error-response.html
@@ -21,19 +21,25 @@ function initTest() {
 }
 
 function onContextCreationError(e) {
+  if (e.statusMessage.length == 0) {
+    testFailed("webglcontextcreationerror test failed");
+    return;
+  }
   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 == ""))
+      error = error + "DeviceID is null.";
+    if (key == "VENDOR" && (value == "0x0000" || value == ""))
+      error = error + "VendorID is null.";
   }
-  if(status)
-    testPassed("Status message displayed for webglcontextcreationerror");
+  if (error.length)
+    testFailed("webglcontextcreationerror test failed" + error);

[Zhenyao Mo, 2015/10/19 17:44:48]

nit: maybe an extra space after "failed"? Otherwise you will get something like
"failedDeviceID is null"

[sivag, 2015/10/20 10:14:41]

Done.

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