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

Issue 869734 link

Starred by 3 users
Status: Fixed
Owner:
█ rjwright@chromium.org
Last visit > 30 days ago
Closed: Aug 2
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Bug



Sign in to add a comment

Crostini apps aren't cleaned up properly when cros device is reformatted

Project Member Reported by rjwright@chromium.org, Aug 1 
If a user installs Crostini & Crostini apps and then reformats their device, residual app data stays in their preferences and causes problems with re-installing the apps on the new Cros install.
Project Member

Comment 1 by bugdroid1@chromium.org, Aug 2 

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

commit 01baf65d5cf76e95acbe58f90f10e65bcb9fc289
Author: Renee Wright <rjwright@chromium.org>
Date: Thu Aug 02 00:06:37 2018

Remove Crostini apps before inserting them to clean up any residual prefs

Bug:  869734 
Change-Id: Ie863c125d49c351b79f5b072bc20535a75dd8d6a
Reviewed-on: https://chromium-review.googlesource.com/1157963
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Renée Wright <rjwright@chromium.org>
Cr-Commit-Position: refs/heads/master@{#580002}
[modify] https://crrev.com/01baf65d5cf76e95acbe58f90f10e65bcb9fc289/chrome/browser/ui/app_list/crostini/crostini_app_model_builder.cc

Comment 2 by rjwright@chromium.org, Aug 2

Status: Fixed (was: Started)

Comment 3 by rjwright@chromium.org, Aug 3

Labels: Merge-Request-69
Project Member

Comment 4 by sheriffbot@chromium.org, Aug 3

Labels: -Merge-Request-69 Merge-Review-69 Hotlist-Merge-Review
This bug requires manual review: M69 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), kariahda@(iOS), cindyb@(ChromeOS), govind@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot

Comment 5 by gov...@chromium.org, Aug 3 

Pls apply appropriate OSs label. Thank you.

Comment 6 by gov...@chromium.org, Aug 6 

Pls apply appropriate OSs label.

Comment 7 by rjwright@chromium.org, Aug 7

Labels: OS-Chrome

Comment 8 by cindyb@chromium.org, Aug 7

Labels: -Merge-Review-69 Merge-Approved-69 M-69
M69 merge approved.
Project Member

Comment 9 by bugdroid1@chromium.org, Aug 8

Labels: -merge-approved-69 merge-merged-3497
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/43b8fb7f38ba4a7b7a88e7def64aca36527e4285

commit 43b8fb7f38ba4a7b7a88e7def64aca36527e4285
Author: Renee Wright <rjwright@chromium.org>
Date: Wed Aug 08 04:57:22 2018

Remove Crostini apps before inserting them to clean up any residual prefs

Bug:  869734 
Change-Id: Ie863c125d49c351b79f5b072bc20535a75dd8d6a
Reviewed-on: https://chromium-review.googlesource.com/1157963
Reviewed-by: Xiyuan Xia <xiyuan@chromium.org>
Commit-Queue: Renée Wright <rjwright@chromium.org>
Cr-Original-Commit-Position: refs/heads/master@{#580002}(cherry picked from commit 01baf65d5cf76e95acbe58f90f10e65bcb9fc289)
Reviewed-on: https://chromium-review.googlesource.com/1166362
Reviewed-by: Renée Wright <rjwright@chromium.org>
Cr-Commit-Position: refs/branch-heads/3497@{#487}
Cr-Branched-From: 271eaf50594eb818c9295dc78d364aea18c82ea8-refs/heads/master@{#576753}
[modify] https://crrev.com/43b8fb7f38ba4a7b7a88e7def64aca36527e4285/chrome/browser/ui/app_list/crostini/crostini_app_model_builder.cc
►

Sign in to add a comment