wpt diffing: Escape commas in test names

Escape commas in test names by surrounding the test name in quotes.

Bug: 1079015
Change-Id: Iefa3ab8d4bee1955f118f0c6cd0adcf96f356688
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2787314
Reviewed-by: Luke Z <lpz@chromium.org>
Commit-Queue: Rakib Hasan <rmhasan@google.com>
Cr-Commit-Position: refs/heads/master@{#867299}

Author: Rakib M. Hasan

third_party/blink/tools/diff_wpt_results.py

@@ -43,7 +43,11 @@ def create_csv(actual_results_map, baseline_results_map, csv_output):
     file_output = 'Test name, Test Result, Baseline Result, Result Comparison\n'
 
     for test in sorted(super_set):
-        line = [test]
+        if ',' in test:
+            line = ['"%s"' % test]
+        else:
+            line = [test]
+
         for result_mp in [actual_results_map, baseline_results_map]:
             line.append(result_mp.get(test, {'actual': 'MISSING'})
                                  .get('actual').split()[-1])

third_party/blink/tools/diff_wpt_results_unittest.py

@@ -41,6 +41,15 @@ def test_compress_json(self):
 
 
 class CreateCsvTest(unittest.TestCase):
+    def test_name_with_comma_escaped_in_csv(self):
+        actual_mp = {'test, name.html': {'actual': 'PASS'}}
+        with io.BytesIO() as csv_out:
+            create_csv(actual_mp, actual_mp, csv_out)
+            csv_out.seek(0)
+            content = csv_out.read()
+            self.assertEquals(content, CSV_HEADING + '\n' +
+                              '"test, name.html",PASS,PASS,SAME RESULTS\n')
+
     def test_create_csv_with_same_result(self):
         actual_mp = {'test.html': {'actual': 'PASS'}}
         with io.BytesIO() as csv_out: