Changeset 74163 in webkit

Timestamp:

Dec 15, 2010 4:58:08 PM (13 years ago)

Author:

commit-queue@webkit.org

Message:

2010-12-15 Sheriff Bot <​webkit.review.bot@gmail.com>

Unreviewed, rolling out r74136.
​http://trac.webkit.org/changeset/74136
​https://bugs.webkit.org/show_bug.cgi?id=51135

r74136 breaks chromium canary bots because some tests are not
rebaselined correctly to resolve EOL differences (Requested by
jianli on #webkit).

• Scripts/webkitpy/layout_tests/port/base.py:
• Scripts/webkitpy/layout_tests/port/test.py:
• Scripts/webkitpy/layout_tests/test_types/text_diff.py:

Location:

trunk/WebKitTools

Files:

• ChangeLog (modified) (1 diff)
• Scripts/webkitpy/layout_tests/port/base.py (modified) (2 diffs)
• Scripts/webkitpy/layout_tests/port/test.py (modified) (4 diffs)
• Scripts/webkitpy/layout_tests/test_types/text_diff.py (modified) (1 diff)

trunk/WebKitTools/ChangeLog

@@ +1 @@
+2010-12-15  Sheriff Bot  <webkit.review.bot@gmail.com>
+
+        Unreviewed, rolling out r74136.
+        http://trac.webkit.org/changeset/74136
+        https://bugs.webkit.org/show_bug.cgi?id=51135
+
+        r74136 breaks chromium canary bots because some tests are not
+        rebaselined correctly to resolve EOL differences (Requested by
+        jianli on #webkit).
+
+        * Scripts/webkitpy/layout_tests/port/base.py:
+        * Scripts/webkitpy/layout_tests/port/test.py:
+        * Scripts/webkitpy/layout_tests/test_types/text_diff.py:
+
 2010-12-15  Adam Roben  <aroben@apple.com>
 

trunk/WebKitTools/Scripts/webkitpy/layout_tests/port/base.py

@@ -323 +323 @@
 
     def expected_text(self, test):
-        """Returns the text output we expect the test to produce.
-        End-of-line characters are normalized to '\n'."""
+        """Returns the text output we expect the test to produce."""
         # FIXME: DRT output is actually utf-8, but since we don't decode the
         # output from DRT (instead treating it as a binary string), we read the
@@ -332 +331 @@
             return ''
         text = self._filesystem.read_binary_file(path)
-        return text.replace("\r\n", "\n")
+        return text.strip("\r\n").replace("\r\n", "\n") + "\n"
 
     def filename_to_uri(self, filename):

trunk/WebKitTools/Scripts/webkitpy/layout_tests/port/test.py

@@ -91 +91 @@
         base.Port.__init__(self, **kwargs)
         tests = TestList(self)
+        tests.add('passes/image.html')
+        tests.add('passes/text.html')
         tests.add('failures/expected/checksum.html',
                   actual_checksum='checksum_fail-checksum')
@@ -113 +115 @@
         tests.add('failures/expected/missing_text.html',
                   expected_text=None)
-        tests.add('failures/expected/newlines_leading.html',
-                  expected_text="\nfoo\n",
-                  actual_text="foo\n")
-        tests.add('failures/expected/newlines_trailing.html',
-                  expected_text="foo\n\n",
-                  actual_text="foo\n")
-        tests.add('failures/expected/newlines_with_excess_CR.html',
-                  expected_text="foo\r\r\r\n",
-                  actual_text="foo\n")
         tests.add('failures/expected/text.html',
                   actual_text='text_fail-png')
@@ -132 +125 @@
         tests.add('passes/image.html')
         tests.add('passes/platform_image.html')
-        # Text output files contain "\r\n" on Windows.  This may be
-        # helpfully filtered to "\r\r\n" by our Python/Cygwin tooling.
-        tests.add('passes/text.html',
-                  expected_text='\nfoo\n\n',
-                  actual_text='\nfoo\r\n\r\r\n')
+        tests.add('passes/text.html')
         tests.add('websocket/tests/passes/text.html')
         self._tests = tests
@@ -282 +271 @@
 WONTFIX : failures/expected/missing_image.html = MISSING PASS
 WONTFIX : failures/expected/missing_text.html = MISSING PASS
-WONTFIX : failures/expected/newlines_leading.html = TEXT
-WONTFIX : failures/expected/newlines_trailing.html = TEXT
-WONTFIX : failures/expected/newlines_with_excess_CR.html = TEXT
 WONTFIX : failures/expected/text.html = TEXT
 WONTFIX : failures/expected/timeout.html = TIMEOUT

trunk/WebKitTools/Scripts/webkitpy/layout_tests/test_types/text_diff.py

@@ -50 +50 @@
 
     def _get_normalized_output_text(self, output):
-        """Returns the normalized text output, i.e. the output in which
-        the end-of-line characters are normalized to "\n"."""
-        # Running tests on Windows produces "\r\n".  The "\n" part is helpfully
-        # changed to "\r\n" by our system (Python/Cygwin), resulting in
-        # "\r\r\n", when, in fact, we wanted to compare the text output with
-        # the normalized text expectation files.
-        return output.replace("\r\r\n", "\r\n").replace("\r\n", "\n")
+        # Some tests produce "\r\n" explicitly.  Our system (Python/Cygwin)
+        # helpfully changes the "\n" to "\r\n", resulting in "\r\r\n".
+        norm = output.replace("\r\r\n", "\r\n").strip("\r\n").replace(
+             "\r\n", "\n")
+        return norm + "\n"
 
     def compare_output(self, port, filename, test_args, actual_test_output,