Commit 5f20d581 authored by Anqing Zhao's avatar Anqing Zhao Committed by Commit Bot

Add upload_status into CrashReportInfo proto

Consider that the crash collection in local device doesn't promise the
reports are uploaded in a specific duration, we have to add a new field
into proto to let DM server know the upload status. The corresponding
detailed crash reports cannot be found in crash servers if their status
are not 'uploaded'.

Bug: 1040078
Change-Id: I36ba510bc7ddffa840539eee096d320e7bd5aec2
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2020766
Commit-Queue: Anqing Zhao <anqing@google.com>
Reviewed-by: default avatarSergey Poromov <poromov@chromium.org>
Cr-Commit-Position: refs/heads/master@{#737644}
parent f38df9a1
...@@ -1032,6 +1032,15 @@ message GraphicsStatus { ...@@ -1032,6 +1032,15 @@ message GraphicsStatus {
// Status of a crash report. // Status of a crash report.
message CrashReportInfo { message CrashReportInfo {
// The status options should align with crash_reporter::ReportUploadState.
enum CrashReportUploadStatus {
UPLOAD_STATUS_UNKNOWN = 0;
UPLOAD_STATUS_NOT_UPLOADED = 1;
UPLOAD_STATUS_PENDING = 2;
UPLOAD_STATUS_PENDING_USER_REQUESTED = 3;
UPLOAD_STATUS_UPLOADED = 4;
}
// ID as provided by chrome://crashes. // ID as provided by chrome://crashes.
optional string remote_id = 1; optional string remote_id = 1;
...@@ -1041,6 +1050,9 @@ message CrashReportInfo { ...@@ -1041,6 +1050,9 @@ message CrashReportInfo {
// Human readable string that identifies what caused the crash. // Human readable string that identifies what caused the crash.
optional string cause = 3; optional string cause = 3;
// The upload status of crash report.
optional CrashReportUploadStatus upload_status = 4;
} }
// Report device level status. // Report device level status.
......
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