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

Issue 692830 link

Starred by 1 user

Issue metadata

Status: Verified
Owner:
Last visit 16 days ago
Closed: Feb 2017
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Bug



Sign in to add a comment

Log ARC data removal reasons

Project Member Reported by lhchavez@chromium.org, Feb 16 2017

Issue description

There have been cases where the ARC user data has been requested to be
removed without having a paper trail about the reason why it happened.
This change should provide the reason in the logs so feedback reports at
least can tell what happened.
 
Labels: Merge-Request-57
This landed in R58 in https://chromium.googlesource.com/chromium/src/+/2512542164d696c21c6a62a0f103e1af9c37ca45

Requesting to merge to R57
Labels: -Merge-Request-57 Merge-Approved-57
Approving merge to M57 Chrome OS.
Project Member

Comment 3 by bugdroid1@chromium.org, Feb 16 2017

Labels: -merge-approved-57 merge-merged-2987
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/5fef0127c7d48ef621e3ded6568eabb473ad9d54

commit 5fef0127c7d48ef621e3ded6568eabb473ad9d54
Author: Luis Hector Chavez <lhchavez@google.com>
Date: Thu Feb 16 21:57:03 2017

[Merge-57] arc: Logging ARC data removal reasons

There have been cases where the ARC user data has been requested to be
removed without having a paper trail about the reason why it happened.
This change should provide the reason in the logs so feedback reports at
least can tell what happened.

BUG= 692830 
TEST=cq

Review-Url: https://codereview.chromium.org/2678273005
Cr-Commit-Position: refs/heads/master@{#449047}
(cherry picked from commit 2512542164d696c21c6a62a0f103e1af9c37ca45)

R=yusukes@chromium.org

Review-Url: https://codereview.chromium.org/2705493002 .
Cr-Commit-Position: refs/branch-heads/2987@{#562}
Cr-Branched-From: ad51088c0e8776e8dcd963dbe752c4035ba6dab6-refs/heads/master@{#444943}

[modify] https://crrev.com/5fef0127c7d48ef621e3ded6568eabb473ad9d54/chrome/browser/chromeos/arc/arc_optin_uma.cc
[modify] https://crrev.com/5fef0127c7d48ef621e3ded6568eabb473ad9d54/chrome/browser/chromeos/arc/arc_optin_uma.h
[modify] https://crrev.com/5fef0127c7d48ef621e3ded6568eabb473ad9d54/chrome/browser/chromeos/arc/arc_session_manager.cc
[modify] https://crrev.com/5fef0127c7d48ef621e3ded6568eabb473ad9d54/chrome/browser/chromeos/arc/arc_session_manager.h
[modify] https://crrev.com/5fef0127c7d48ef621e3ded6568eabb473ad9d54/chrome/browser/chromeos/arc/enterprise/arc_enterprise_reporting_service.cc

Status: Fixed (was: Untriaged)
Status: Verified (was: Fixed)

Sign in to add a comment