[CodeHealth] Clean up some TODOs for a bug that's fixed.
Bug: 859477 Change-Id: I75c28c85cfd50cd3edb376b019012ea907fabbe0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2867335 Reviewed-by: Dominic Battré <battre@chromium.org> Reviewed-by: Ken Rockot <rockot@google.com> Reviewed-by: Weilun Shi <sweilun@chromium.org> Commit-Queue: David Bertoni <dbertoni@chromium.org> Cr-Commit-Position: refs/heads/master@{#881279}
This commit is contained in:

committed by
Chromium LUCI CQ

parent
6b5ea7cbe5
commit
f3b663f7f6
@ -172,9 +172,6 @@ void UnsentLogStore::MarkStagedLogAsSent() {
|
||||
|
||||
void UnsentLogStore::TrimAndPersistUnsentLogs() {
|
||||
ListPrefUpdate update(local_state_, log_data_pref_name_);
|
||||
// TODO(crbug.com/859477): Verify that the preference has been properly
|
||||
// registered.
|
||||
CHECK(update.Get());
|
||||
TrimLogs();
|
||||
WriteLogsToPrefList(update.Get());
|
||||
}
|
||||
|
@ -145,11 +145,6 @@ void WriteValue(base::Pickle* m, const base::Value* value, int recursion) {
|
||||
}
|
||||
break;
|
||||
}
|
||||
|
||||
// TODO(crbug.com/859477): Remove after root cause is found.
|
||||
default:
|
||||
CHECK(false);
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user