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

Issue 721713 link

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



Sign in to add a comment

child_process_launcher_helper_posix.cc incorrectly logging ERROR line during Android tests

Project Member Reported by kjellander@chromium.org, May 12 2017 
All Android tests executing are seeing this ERROR line in the logs, no matter if the test is successful or failing. It's constantly confusing developers trying to debug failures.

Please change the warning level or preferably make it only appear when it makes sense.

Examples:
Successful build: https://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/42421
Failing build: https://build.chromium.org/p/chromium.linux/builders/Android%20Tests%20%28dbg%29/builds/42424

Assigning jcivelli@chromium.org according to git blame.
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 7 2017 

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

commit 8e5349aee2dbfd6aadb9c2453d10628be8448125
Author: Jay Civelli <jcivelli@google.com>
Date: Fri Jul 07 17:39:32 2017

Changing a LOG from error to warning in ChildProcessLauncher.

Changing a LOG from error to warning in ChildProcessLauncher as it is a
case that can happen on Android and is not an error.

Bug:  721713 
Change-Id: Ie310f843143108b66965273e7b8b6d6f1c52f9db
Reviewed-on: https://chromium-review.googlesource.com/563499
Reviewed-by: Avi Drissman <avi@chromium.org>
Commit-Queue: Jay Civelli <jcivelli@chromium.org>
Cr-Commit-Position: refs/heads/master@{#484968}
[modify] https://crrev.com/8e5349aee2dbfd6aadb9c2453d10628be8448125/content/browser/child_process_launcher_helper_posix.cc

Comment 2 by jcivelli@chromium.org, Jul 7 2017

Status: Fixed (was: Assigned)
►

Sign in to add a comment