[Android] Do not immediately delete skipped crash dump uploads.

[Android] Do not immediately delete skipped crash dump uploads.

This will give users a chance to decide to manually upload any skipped
crashes, which can be useful when reporting bugs.

This CL pairs nicely with [ https://codereview.chromium.org/2280313002/ ],
which adds some heuristics to clean up old/obsolete crash reports.

BUG=641628
TEST=CrashFileManagerTest.*, MinidumpUploadCallableTest.*
R=gayane@chromium.org

Committed: https://crrev.com/17ad87db2d875747460f05b46a1f1032b57c77b9
Cr-Commit-Position: refs/heads/master@{#415103}

[Ilya Sherman, 2016-08-27 06:21:03]

The CQ bit was checked by isherman@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2016-08-27 06:21:30]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-27 07:19:37]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-08-27 07:19:38]

Dry run: This issue passed the CQ dry run.

[gayane -on leave until 09-2017, 2016-08-29 17:23:44]

lgtm

[Ilya Sherman, 2016-08-29 19:01:04]

isherman@chromium.org changed reviewers:
+ mariakhomenko@chromium.org

[Ilya Sherman, 2016-08-29 19:01:05]

+Maria for OWNERS approval

[gayane -on leave until 09-2017, 2016-08-29 19:12:15]

As owners for this CL and your other CL are not the same maybe just briefly
mention in the description that you will take care of deleting accumulated
reports with some heuristics.

[Ilya Sherman, 2016-08-29 19:15:44]

Description was changed from

==========
[Android] Do not immediately delete skipped crash dump uploads.

This will give users a chance to decide to manually upload any skipped
crashes, which can be useful when reporting bugs.

BUG=641628
TEST=CrashFileManagerTest.*, MinidumpUploadCallableTest.*
R=gayane@chromium.org
==========

to

==========
[Android] Do not immediately delete skipped crash dump uploads.

This will give users a chance to decide to manually upload any skipped
crashes, which can be useful when reporting bugs.

This CL pairs nicely with [ https://codereview.chromium.org/2280313002/ ],
which adds some heuristics to clean up old/obsolete crash reports.

BUG=641628
TEST=CrashFileManagerTest.*, MinidumpUploadCallableTest.*
R=gayane@chromium.org
==========

[Ilya Sherman, 2016-08-29 19:16:20]

On 2016/08/29 19:12:15, gayane wrote:
> As owners for this CL and your other CL are not the same maybe just briefly
> mention in the description that you will take care of deleting accumulated
> reports with some heuristics.

Thanks, done =)  And, cc'ed Maria on the other CL --
https://codereview.chromium.org/2280313002/

[Maria, 2016-08-29 20:50:50]

lgtm

[Ilya Sherman, 2016-08-29 20:55:39]

The CQ bit was checked by isherman@chromium.org

[commit-bot: I haz the power, 2016-08-29 21:19:24]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-30 04:37:48]

Description was changed from

==========
[Android] Do not immediately delete skipped crash dump uploads.

This will give users a chance to decide to manually upload any skipped
crashes, which can be useful when reporting bugs.

This CL pairs nicely with [ https://codereview.chromium.org/2280313002/ ],
which adds some heuristics to clean up old/obsolete crash reports.

BUG=641628
TEST=CrashFileManagerTest.*, MinidumpUploadCallableTest.*
R=gayane@chromium.org
==========

to

==========
[Android] Do not immediately delete skipped crash dump uploads.

This will give users a chance to decide to manually upload any skipped
crashes, which can be useful when reporting bugs.

This CL pairs nicely with [ https://codereview.chromium.org/2280313002/ ],
which adds some heuristics to clean up old/obsolete crash reports.

BUG=641628
TEST=CrashFileManagerTest.*, MinidumpUploadCallableTest.*
R=gayane@chromium.org
==========

[commit-bot: I haz the power, 2016-08-30 04:37:49]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-30 04:39:18]

Description was changed from

==========
[Android] Do not immediately delete skipped crash dump uploads.

This will give users a chance to decide to manually upload any skipped
crashes, which can be useful when reporting bugs.

This CL pairs nicely with [ https://codereview.chromium.org/2280313002/ ],
which adds some heuristics to clean up old/obsolete crash reports.

BUG=641628
TEST=CrashFileManagerTest.*, MinidumpUploadCallableTest.*
R=gayane@chromium.org
==========

to

==========
[Android] Do not immediately delete skipped crash dump uploads.

This will give users a chance to decide to manually upload any skipped
crashes, which can be useful when reporting bugs.

This CL pairs nicely with [ https://codereview.chromium.org/2280313002/ ],
which adds some heuristics to clean up old/obsolete crash reports.

BUG=641628
TEST=CrashFileManagerTest.*, MinidumpUploadCallableTest.*
R=gayane@chromium.org

Committed: https://crrev.com/17ad87db2d875747460f05b46a1f1032b57c77b9
Cr-Commit-Position: refs/heads/master@{#415103}
==========

[commit-bot: I haz the power, 2016-08-30 04:39:19]

Patchset 1 (id:??) landed as
https://crrev.com/17ad87db2d875747460f05b46a1f1032b57c77b9
Cr-Commit-Position: refs/heads/master@{#415103}