3party/crashpad
0
0
Fork 0
mirror of https://github.com/chromium/crashpad.git synced 2025-03-19 01:43:46 +00:00
Code Issues Packages Projects Releases Wiki Activity

Don't throttle explicitly requested uploads

Browse Source 
R=mark@chromium.org
BUG=chromium:660955

Change-Id: Ia31846fe3487a52f4cad34859e23a7192ca4065e
Reviewed-on: https://chromium-review.googlesource.com/405533
Reviewed-by: Mark Mentovai <mark@chromium.org>
This commit is contained in:
Scott Graham 2016-10-31 14:16:12 -07:00
parent b978b03fa1
commit c2814e2519
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
handler/crash_report_upload_thread.cc
View File

@ -207,9 +207,12 @@ void CrashReportUploadThread::ProcessPendingReport(
// hour, and retire reports that would exceed this limit or for which the // hour, and retire reports that would exceed this limit or for which the
// upload fails on the first attempt. // upload fails on the first attempt.
// //
// If upload was requested explicitly (i.e. by user action), we do not
// throttle the upload.
//
// TODO(mark): Provide a proper rate-limiting strategy and allow for failed // TODO(mark): Provide a proper rate-limiting strategy and allow for failed
// upload attempts to be retried. // upload attempts to be retried.
if (rate_limit_) { if (!report.upload_explicitly_requested && rate_limit_) {
time_t last_upload_attempt_time; time_t last_upload_attempt_time;
if (settings->GetLastUploadAttemptTime(&last_upload_attempt_time)) { if (settings->GetLastUploadAttemptTime(&last_upload_attempt_time)) {
time_t now = time(nullptr); time_t now = time(nullptr);