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

Issue 754001 link

Starred by 1 user
Status: Verified
Owner:
xiy...@chromium.org
Closed: Sep 2017
Cc:
r...@chromium.org
█ sduraisamy@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Bug



Sign in to add a comment

Put kiosk launch logs to system logs

Project Member Reported by xiy...@chromium.org, Aug 9 2017 
The kiosk launch code uses LOG(), which goes to chrome user log after kiosk app cryptohome is mounted. This makes it hard to figure out launching related problems, especially for problems happening only in the wild.

Propose to replace LOG() with SYSLOG() so that it goes to kernel log. There are not many of the logs generated during the app launch and kernel log is easy to extract from field.
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 15 2017 

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

commit 00c5893de609ae1fdc167799316fd43300491900
Author: Xiyuan Xia <xiyuan@chromium.org>
Date: Tue Aug 15 17:53:34 2017

kiosk: Use SYSLOG for launch code logging

The intention of the CL is to make it easier to debug kiosk
launching issues from the field, where chrome user log is
usually not available. So use SYSLOG for launch code so that
it would be part of kernel log and be collected from the field.

This CL also adds a few more logging to make debugging launch
hangs easier.

BUG= 754001 

Change-Id: I283398d84086ce5b56b5de7b06ecbe34669dd1b4
Reviewed-on: https://chromium-review.googlesource.com/611497
Reviewed-by: Jenny Zhang <jennyz@chromium.org>
Commit-Queue: Xiyuan Xia <xiyuan@chromium.org>
Cr-Commit-Position: refs/heads/master@{#494440}
[modify] https://crrev.com/00c5893de609ae1fdc167799316fd43300491900/chrome/browser/chromeos/app_mode/kiosk_profile_loader.cc
[modify] https://crrev.com/00c5893de609ae1fdc167799316fd43300491900/chrome/browser/chromeos/app_mode/startup_app_launcher.cc
[modify] https://crrev.com/00c5893de609ae1fdc167799316fd43300491900/chrome/browser/chromeos/login/app_launch_controller.cc

Comment 2 by xiy...@chromium.org, Sep 5 2017

Labels: -M-61 M-62
Status: Fixed (was: Assigned)
Fixed in M62.

Comment 3 by mlight@chromium.org, Oct 11 2017

Status: Verified (was: Fixed)
Verified in M62 Beta build 9901.44.0.
►

Sign in to add a comment