New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 791794 link

Starred by 1 user
Status: Fixed
Owner:
█ jcivelli@chromium.org
Last visit > 30 days ago
Closed: Dec 2017
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

Android test runner fails if the test outputs invalid UTF8 characters.

Project Member Reported by jcivelli@chromium.org, Dec 4 2017 
If a tests outputs any invalid UTF8 characters, it causes the Android test runner to fail as it passes that string to the python JSON library that require valid UTF8.
Project Member

Comment 1 by bugdroid1@chromium.org, Dec 5 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/ba78b329d7ab3d22386a9b7fa3be182bdbda73bf

commit ba78b329d7ab3d22386a9b7fa3be182bdbda73bf
Author: Jay Civelli <jcivelli@google.com>
Date: Tue Dec 05 01:14:40 2017

Make android test runner support invalid UTF8 output.

If a test produced non valid UTF8 characters, the test runner would fail
as it would provide invalid UTF8 to the JSON python library.
We now sanitize the output.

Bug:  791794 
Change-Id: I635d1dbb9fdf2601112bed2953cae0e2c7e9ae92
Reviewed-on: https://chromium-review.googlesource.com/807386
Reviewed-by: John Budorick <jbudorick@chromium.org>
Commit-Queue: Jay Civelli <jcivelli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#521565}
[modify] https://crrev.com/ba78b329d7ab3d22386a9b7fa3be182bdbda73bf/build/android/pylib/results/json_results.py

Comment 2 by jcivelli@chromium.org, Dec 5 2017

Status: Fixed (was: Untriaged)
►

Sign in to add a comment