test harness tests complain about testRunner not being declared inside a browser
authorrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 29 Apr 2016 19:39:44 +0000 (19:39 +0000)
committerrniwa@webkit.org <rniwa@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 29 Apr 2016 19:39:44 +0000 (19:39 +0000)
https://bugs.webkit.org/show_bug.cgi?id=157056

Reviewed by Alex Christensen.

Don't run the code meant to re-format results for DRT/WTR when testRunner is not defined (i.e. inside a browser).
Also fixed various styling issues with this JavaScript file and removed useless comments.

* resources/testharnessreport.js:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@200255 268f45cc-cd09-0410-ab3c-d52691b4dbfc

LayoutTests/ChangeLog
LayoutTests/resources/testharnessreport.js

index 9e63a90..0f951a8 100644 (file)
@@ -1,3 +1,15 @@
+2016-04-29  Ryosuke Niwa  <rniwa@webkit.org>
+
+        test harness tests complain about testRunner not being declared inside a browser
+        https://bugs.webkit.org/show_bug.cgi?id=157056
+
+        Reviewed by Alex Christensen.
+
+        Don't run the code meant to re-format results for DRT/WTR when testRunner is not defined (i.e. inside a browser).
+        Also fixed various styling issues with this JavaScript file and removed useless comments.
+
+        * resources/testharnessreport.js:
+
 2016-04-29  Commit Queue  <commit-queue@webkit.org>
 
         Unreviewed, rolling out r200232.
index 8f0d754..be7f34f 100644 (file)
@@ -25,66 +25,50 @@ if (self.testRunner) {
 \r
 // Function used to convert the test status code into\r
 // the corresponding string\r
-function convertResult(resultStatus){\r
-       if(resultStatus == 0)\r
-               return("PASS");\r
-       else if(resultStatus == 1)\r
-               return("FAIL");\r
-       else if(resultStatus == 2)\r
-               return("TIMEOUT");\r
-       else\r
-               return("NOTRUN");\r
+function convertResult(resultStatus)\r
+{\r
+    if(resultStatus == 0)\r
+        return("PASS");\r
+    else if(resultStatus == 1)\r
+        return("FAIL");\r
+    else if(resultStatus == 2)\r
+        return("TIMEOUT");\r
+    else\r
+        return("NOTRUN");\r
 }\r
 \r
-/* Disable the default output of testharness.js.  The default output formats\r
-*  test results into an HTML table.  When that table is dumped as text, no\r
-*  spacing between cells is preserved, and it is therefore not readable. By\r
-*  setting output to false, the HTML table will not be created\r
-*/\r
-setup({"output": false, "explicit_timeout": !!testRunner});\r
+if (self.testRunner) {\r
+    /* Disable the default output of testharness.js.  The default output formats\r
+    *  test results into an HTML table.  When that table is dumped as text, no\r
+    *  spacing between cells is preserved, and it is therefore not readable. By\r
+    *  setting output to false, the HTML table will not be created\r
+    */\r
+    setup({"output": false, "explicit_timeout": true});\r
 \r
-/*  Using a callback function, test results will be added to the page in a \r
-*   manner that allows dumpAsText to produce readable test results\r
-*/\r
-add_completion_callback(function (tests, harness_status){\r
-       \r
-       // Create element to hold results\r
-       var results = document.createElement("pre");\r
-       \r
-       // Declare result string\r
-       var resultStr = "\n";\r
-       \r
-       // Check harness_status.  If it is not 0, tests did not\r
-       // execute correctly, output the error code and message\r
-       if(harness_status.status != 0){\r
-               resultStr += "Harness Error (" + \r
-                                        convertResult(harness_status.status) +\r
-                                        "), message = " +\r
-                                        harness_status.message + "\n\n";\r
-       }\r
-       // Iterate through tests array and build string that contains\r
-       // results for all tests\r
-       for(var i = 0; i < tests.length; i++) {\r
-               var message = (tests[i].message != null) ? tests[i].message : "";\r
-               if (tests[i].status == 1 && !tests[i].dumpStack) {\r
-                       // Remove stack for failed tests for proper string comparison without file paths.\r
-                       // For a test to dump the stack set its dumpStack attribute to true.\r
-                       var stackIndex = message.indexOf("(stack:");\r
-                       if (stackIndex > 0) {\r
-                               message = message.substr(0, stackIndex);\r
-                       }\r
-               }\r
-               resultStr += convertResult(tests[i].status) + " " + \r
-                                       ( (tests[i].name!=null) ? tests[i].name : "" ) + " " +\r
-                                       message +\r
-                                       "\n";\r
-        }                      \r
+    /*  Using a callback function, test results will be added to the page in a \r
+    *   manner that allows dumpAsText to produce readable test results\r
+    */\r
+    add_completion_callback(function (tests, harness_status) {\r
+        var results = document.createElement("pre");\r
+        var resultStr = "\n";\r
 \r
-       results.innerText = resultStr;\r
+        if(harness_status.status != 0)\r
+            resultStr += "Harness Error (" + convertResult(harness_status.status) + "), message = " + harness_status.message + "\n\n";\r
 \r
-       // Add results element to document\r
-       document.body.appendChild(results);\r
+        for (var i = 0; i < tests.length; i++) {\r
+            var message = (tests[i].message != null) ? tests[i].message : "";\r
+            if (tests[i].status == 1 && !tests[i].dumpStack) {\r
+                // Remove stack for failed tests for proper string comparison without file paths.\r
+                // For a test to dump the stack set its dumpStack attribute to true.\r
+                var stackIndex = message.indexOf("(stack:");\r
+                if (stackIndex > 0)\r
+                    message = message.substr(0, stackIndex);\r
+            }\r
+            resultStr += convertResult(tests[i].status) + " " +  (tests[i].name != null ? tests[i].name || "") + " " + message + "\n";\r
+        }\r
 \r
-       if (self.testRunner)\r
-               testRunner.notifyDone();\r
-});\r
+        results.innerText = resultStr;\r
+        document.body.appendChild(results);\r
+        testRunner.notifyDone();\r
+    });\r
+}\r