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

Unified Diff: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py

Issue 2633753003: [Android] Stop using universal_newlines in the layout tests server constructor. (RELAND) (Closed)
Patch Set: accept unicode and str objects Created 3 years, 11 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
Index: third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py
diff --git a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py
index 7f0bed839f1a5f094545421e68169e39ff91b7a6..52d406462b552a3308a96346dbf3c64bfd9a3086 100644
--- a/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py
+++ b/third_party/WebKit/Tools/Scripts/webkitpy/layout_tests/port/base.py
@@ -1465,12 +1465,12 @@ class Port(object):
# We require stdout and stderr to be bytestrings, not character strings.
if stdout:
- assert isinstance(stdout, str)
+ assert isinstance(stdout, basestring)
stdout_lines = stdout.decode('utf8', 'replace').splitlines()
else:
stdout_lines = [u'<empty>']
if stderr:
- assert isinstance(stderr, str)
+ assert isinstance(stderr, basestring)
stderr_lines = stderr.decode('utf8', 'replace').splitlines()
else:
stderr_lines = [u'<empty>']

Powered by Google App Engine
This is Rietveld 408576698