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

Issue 606081 link

Starred by 2 users

Issue metadata

Status: Fixed
Owner:
Last visit 16 days ago
Closed: May 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug



Sign in to add a comment

ARC: Add more logging to ArcBridgeService

Project Member Reported by lhchavez@chromium.org, Apr 22 2016

Issue description

A few feedback reports have been submitted where there is no clear reason why ArcBridgeService did not start ARC, or that the startup process failed. We should be able to find this information.
 
Status: Started (was: Assigned)
Project Member

Comment 2 by bugdroid1@chromium.org, May 4 2016

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

commit e143a0260007413b48e82690fdf455efbe836789
Author: lhchavez <lhchavez@chromium.org>
Date: Wed May 04 03:15:59 2016

arc_bridge: Add more logging, plus cleanup

In order to better troubleshoot startup issues, add more VLOG
entries and promote a couple of critical ones from VLOG(1) to
VLOG(0). Also a couple of small changes:

* Clean up some confusing logging statements.
* Add a small delay to reconnects so that the container can
  finish cleaning up in case it was doing so.

BUG= 606081 
TEST=components_unittests

Review-Url: https://codereview.chromium.org/1925183004
Cr-Commit-Position: refs/heads/master@{#391434}

[modify] https://crrev.com/e143a0260007413b48e82690fdf455efbe836789/chrome/browser/chromeos/arc/arc_auth_service.cc
[modify] https://crrev.com/e143a0260007413b48e82690fdf455efbe836789/chrome/browser/chromeos/arc/arc_policy_bridge.cc
[modify] https://crrev.com/e143a0260007413b48e82690fdf455efbe836789/chrome/browser/chromeos/arc/arc_settings_service.cc
[modify] https://crrev.com/e143a0260007413b48e82690fdf455efbe836789/components/arc/arc_bridge_bootstrap.cc
[modify] https://crrev.com/e143a0260007413b48e82690fdf455efbe836789/components/arc/arc_bridge_bootstrap.h
[modify] https://crrev.com/e143a0260007413b48e82690fdf455efbe836789/components/arc/arc_bridge_service.cc
[modify] https://crrev.com/e143a0260007413b48e82690fdf455efbe836789/components/arc/arc_bridge_service_impl.cc
[modify] https://crrev.com/e143a0260007413b48e82690fdf455efbe836789/components/arc/arc_bridge_service_impl.h
[modify] https://crrev.com/e143a0260007413b48e82690fdf455efbe836789/components/arc/arc_bridge_service_unittest.cc
[modify] https://crrev.com/e143a0260007413b48e82690fdf455efbe836789/components/arc/net/arc_net_host_impl.cc

Project Member

Comment 3 by bugdroid1@chromium.org, May 4 2016

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/platform2/+/0aeaf2f8b762b3c211e004c146e60ddd3059c288

commit 0aeaf2f8b762b3c211e004c146e60ddd3059c288
Author: Luis Hector Chavez <lhchavez@google.com>
Date: Fri Apr 29 22:19:10 2016

login: Make ARC logs more verbose

If building for an ARC target, it's useful to get more verbose logs so
we can collect better information in feedback reports, especially about
the lifecycle of ARC. This change adds *arc/* to vmodule.

BUG= chromium:606081 
TEST=More logs seen in /home/user/<hash>/log/chrome

Change-Id: I22e6c23ca32006cc24686f0a867ad6ab5433e0a0
Reviewed-on: https://chromium-review.googlesource.com/341502
Commit-Ready: Luis Hector Chavez <lhchavez@google.com>
Tested-by: Luis Hector Chavez <lhchavez@google.com>
Reviewed-by: Dan Erat <derat@chromium.org>

[modify] https://crrev.com/0aeaf2f8b762b3c211e004c146e60ddd3059c288/login_manager/chrome_setup.cc

Status: Fixed (was: Started)

Sign in to add a comment