Title: [124071] trunk/Tools
Revision
124071
Author
dpra...@chromium.org
Date
2012-07-30 11:55:10 -0700 (Mon, 30 Jul 2012)

Log Message

After r123895, new-run-webkit-tests can fail with TypeError when a test crashes
https://bugs.webkit.org/show_bug.cgi?id=92664

Reviewed by Michael Saboff.

Updated to handle crash_logs being returned as a tuple of (stderr, crash_log).

* Scripts/webkitpy/layout_tests/port/mac.py:
(MacPort.look_for_new_crash_logs):

Modified Paths

Diff

Modified: trunk/Tools/ChangeLog (124070 => 124071)


--- trunk/Tools/ChangeLog	2012-07-30 18:54:59 UTC (rev 124070)
+++ trunk/Tools/ChangeLog	2012-07-30 18:55:10 UTC (rev 124071)
@@ -1,3 +1,15 @@
+2012-07-30  Dirk Pranke  <dpra...@chromium.org>
+
+        After r123895, new-run-webkit-tests can fail with TypeError when a test crashes
+        https://bugs.webkit.org/show_bug.cgi?id=92664
+
+        Reviewed by Michael Saboff.
+
+        Updated to handle crash_logs being returned as a tuple of (stderr, crash_log).
+
+        * Scripts/webkitpy/layout_tests/port/mac.py:
+        (MacPort.look_for_new_crash_logs):
+
 2012-07-30  Stephen White  <senorbla...@chromium.org>
 
         Added a SkiaGraphics definition to the watchlist file; cc'ed myself

Modified: trunk/Tools/Scripts/webkitpy/layout_tests/port/mac.py (124070 => 124071)


--- trunk/Tools/Scripts/webkitpy/layout_tests/port/mac.py	2012-07-30 18:54:59 UTC (rev 124070)
+++ trunk/Tools/Scripts/webkitpy/layout_tests/port/mac.py	2012-07-30 18:55:10 UTC (rev 124071)
@@ -235,7 +235,7 @@
         for (test_name, process_name, pid) in crashed_processes:
             # Passing None for output.  This is a second pass after the test finished so
             # if the output had any loggine we would have already collected it.
-            crash_log = self._get_crash_log(process_name, pid, None, None, start_time, wait_for_log=False)
+            crash_log = self._get_crash_log(process_name, pid, None, None, start_time, wait_for_log=False)[1]
             if not crash_log:
                 continue
             crash_logs[test_name] = crash_log
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-changes

Reply via email to