Commit 9d25dd0a authored by Yann Dago's avatar Yann Dago Committed by Chromium LUCI CQ

Retire Downgrade.TakeSnapshot.MoveExistingSnapshot.*

Bug: 1149461
Change-Id: I15ab9cc3c2db147e2dd9222840a5fc8752dc50fc
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2562229
Commit-Queue: Yann Dago <ydago@chromium.org>
Reviewed-by: default avatarMark Pearson <mpearson@chromium.org>
Reviewed-by: default avatarGreg Thompson <grt@chromium.org>
Reviewed-by: default avatarMichael Lippautz <mlippautz@chromium.org>
Cr-Commit-Position: refs/heads/master@{#833108}
parent ddb9bd5e
...@@ -160,10 +160,10 @@ void SnapshotManager::TakeSnapshot(const base::Version& version) { ...@@ -160,10 +160,10 @@ void SnapshotManager::TakeSnapshot(const base::Version& version) {
auto move_target_dir = user_data_dir_.Append(kSnapshotsDir) auto move_target_dir = user_data_dir_.Append(kSnapshotsDir)
.AddExtension(kDowngradeDeleteSuffix); .AddExtension(kDowngradeDeleteSuffix);
base::CreateDirectory(move_target_dir); base::CreateDirectory(move_target_dir);
MoveFolderForLaterDeletion( // This succeeds more than 80% of the time.
snapshot_dir, move_target_dir.AppendASCII(version.GetString()), MoveFolderForLaterDeletion(snapshot_dir,
"Downgrade.TakeSnapshot.MoveExistingSnapshot.Result", move_target_dir.AppendASCII(version.GetString()),
"Downgrade.TakeSnapshot.MoveExistingSnapshot.FailureCount"); nullptr, nullptr);
} }
size_t success_count = 0; size_t success_count = 0;
......
...@@ -4375,8 +4375,24 @@ reviews. Googlers can read more about this at go/gwsq-gerrit. ...@@ -4375,8 +4375,24 @@ reviews. Googlers can read more about this at go/gwsq-gerrit.
</summary> </summary>
</histogram> </histogram>
<histogram name="Downgrade.TakeSnapshot.MoveExistingSnapshot.FailureCount"
units="count" expires_after="2020-12-31">
<obsolete>
Removed in M89.
</obsolete>
<owner>grt@chromium.org</owner>
<owner>ydago@chromium.org</owner>
<summary>
The number of items within an existing snapshot directory that could not be
moved aside before replacing it with a newer snapshot.
</summary>
</histogram>
<histogram name="Downgrade.TakeSnapshot.MoveExistingSnapshot.Result" <histogram name="Downgrade.TakeSnapshot.MoveExistingSnapshot.Result"
enum="BooleanSuccess" expires_after="2020-12-31"> enum="BooleanSuccess" expires_after="2020-12-31">
<obsolete>
Removed in M89.
</obsolete>
<owner>grt@chromium.org</owner> <owner>grt@chromium.org</owner>
<owner>ydago@chromium.org</owner> <owner>ydago@chromium.org</owner>
<summary> <summary>
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment