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

Issue 659938 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Last visit > 30 days ago
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 3
Type: Bug



Sign in to add a comment

Move minidump renaming from MinidumpUploadCallable to MinidumpUploadService

Project Member Reported by gsennton@chromium.org, Oct 27 2016

Issue description

In general all of the renaming of minidumps happens in the MinidumpUploadService except for the case where a minidump is uploaded - CrashFileManager.markUploadSuccess - and the cases where a minidump is skipped - CrashFileManager.markUploadSkipped, these are called in MinidumpUploadCallable but could just as well be called in MinidumpUploadService since the upload callable has return values for each of these cases.
 
Cc: isherman@chromium.org mariakho...@chromium.org
Blocking: -652719
This doesn't actually block the minidump-upload componentization - we only need to componentize CrashFileManager and MinidumpUploadCallable.

(and it's not certain we even want to fix this).
Status: WontFix (was: Assigned)
Cleaning up some old bugs - I don't think we see this as important enough to fix given that the fix itself will be a bit tricky (in terms of test-changes).

Sign in to add a comment