New issue
Advanced search Search tips

Issue 657539 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Oct 2016
Components:
EstimatedDays: ----
NextAction: ----
OS: Android
Pri: 2
Type: Bug



Sign in to add a comment

Android.DownloadManager.InitialCount.Total reports externally deleted content

Project Member Reported by dah...@chromium.org, Oct 19 2016

Issue description

Android.DownloadManager.InitialCount.Total reports externally deleted content. As a result, the value contained in that field is 18% higher than summing up the content in the other fields. In order to remove this discrepancy we should eliminate externally deleted files from the total count.
 
Should this target M55?

Comment 2 by dah...@chromium.org, Oct 19 2016

It would be nice to have, but not a requirement. M56 works as well.
Project Member

Comment 4 by bugdroid1@chromium.org, Oct 25 2016

Labels: merge-merged-2883
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/80d9048098e1e4d974f7590d5a7ac1a513a7dd0b

commit 80d9048098e1e4d974f7590d5a7ac1a513a7dd0b
Author: Theresa Wellington <twellington@google.com>
Date: Tue Oct 25 14:32:04 2016

Fix a download histograms

Record opening downloads in CCT and fix histogram for download counts.

BUG=657542,  657539 
TBR=dfalcantara@chromium.org

Review URL: https://codereview.chromium.org/2453443002 .

Review-Url: https://codereview.chromium.org/2443623002
Cr-Original-Commit-Position: refs/heads/master@{#427167}
Cr-Commit-Position: refs/branch-heads/2883@{#279}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[modify] https://crrev.com/80d9048098e1e4d974f7590d5a7ac1a513a7dd0b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java
[modify] https://crrev.com/80d9048098e1e4d974f7590d5a7ac1a513a7dd0b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryItemWrapper.java

Status: Fixed (was: Assigned)
Project Member

Comment 6 by bugdroid1@chromium.org, Oct 27 2016

Labels: merge-merged-2840
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/80d9048098e1e4d974f7590d5a7ac1a513a7dd0b

commit 80d9048098e1e4d974f7590d5a7ac1a513a7dd0b
Author: Theresa Wellington <twellington@google.com>
Date: Tue Oct 25 14:32:04 2016

Fix a download histograms

Record opening downloads in CCT and fix histogram for download counts.

BUG=657542,  657539 
TBR=dfalcantara@chromium.org

Review URL: https://codereview.chromium.org/2453443002 .

Review-Url: https://codereview.chromium.org/2443623002
Cr-Original-Commit-Position: refs/heads/master@{#427167}
Cr-Commit-Position: refs/branch-heads/2883@{#279}
Cr-Branched-From: 614d31daee2f61b0180df403a8ad43f20b9f6dd7-refs/heads/master@{#423768}

[modify] https://crrev.com/80d9048098e1e4d974f7590d5a7ac1a513a7dd0b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java
[modify] https://crrev.com/80d9048098e1e4d974f7590d5a7ac1a513a7dd0b/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryItemWrapper.java

Comment 7 by dimu@google.com, Nov 4 2016

[Automated comment] removing mislabelled merge-merged-2840

Comment 8 by dimu@google.com, Nov 4 2016

Labels: -merge-merged-2840
[Automated comment] removing mislabelled merge-merged-2840

Sign in to add a comment