Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(539)

Unified Diff: Tools/Scripts/webkitpy/layout_tests/port/android.py

Issue 590983002: Revert of Try #3 to fix the android layout tests crash in run-webkit-tests. (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | Tools/Scripts/webkitpy/layout_tests/views/metered_stream.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Tools/Scripts/webkitpy/layout_tests/port/android.py
diff --git a/Tools/Scripts/webkitpy/layout_tests/port/android.py b/Tools/Scripts/webkitpy/layout_tests/port/android.py
index 8e6e71e452d3d46dccaaf96196ffd70a11812495..8a84b112a2b741e82e23e491d621c14682c86a49 100644
--- a/Tools/Scripts/webkitpy/layout_tests/port/android.py
+++ b/Tools/Scripts/webkitpy/layout_tests/port/android.py
@@ -257,7 +257,9 @@
result = self._executive.run_command(self.adb_command() + command, error_handler=error_handler, debug_logging=self._debug_logging)
# We limit the length to avoid outputting too verbose commands, such as "adb logcat".
- self._log_debug('Run adb result: ' + result[:80])
+ # Also make sure that the output is ascii-encoded to avoid confusing other parts of
+ # the system.
+ self._log_debug('Run adb result: ' + result[:80].encode('ascii', errors='replace'))
return result
def get_serial(self):
« no previous file with comments | « no previous file | Tools/Scripts/webkitpy/layout_tests/views/metered_stream.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698