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

Issue 658312 link

Starred by 1 user

Issue metadata

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



Sign in to add a comment

Add new telemetry to better understand what files are being downloaded, shared and opened

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

Issue description

In the downloads home telemetry we see that most of the files that are downloaded, opened and shared are falling into the "Other" category. We want to be able to better understand what specifically is in that "Other" bucket that is getting so much activity. We need to add three new histograms:

The first is related to the Android.DownloadManager,InitialCount and should track what "other" files are shown in the downloads home (currently 36% of downloaded files are "Other").

The second is related to the Android.DownloadManager.OpenFailed and should track what "other" files are failing to open (currently about 98% of all failures are other).

The third is related to the Android.DownloadManager.Share.FileTypes and should show what "Other" files are being shared (currently 58% of shared files are "Other").

There is a long tail of thousands of file formats that could be applicable, the list below represents the ones that are most likely to be impactful. Additionally, we should have an "other" category that catches the long tail to see if we need to break things out further.

.CSV	Comma separated, variable length file (Open in Excel)
.DOC	Microsoft Word for Windows/Word97
.DOCX	Microsoft Word for Windows/Word97
.EXE	PC Application
.PDF	Acrobat -Portable document format
.PPT	PowerPoint
.PPTX	PowerPoint
.PSD	Adobe PhotoShop
.RTF	Rich Text Format
.TXT	ASCII text
.XLS	Excel spreadsheet
.XLSX   Excel spreadsheet
.ZIP	PC Zip Compressed Archive
.apk	android app package

 
 
Cc: dah...@chromium.org
For the initial count, I'm seeing ~28% in "other" for 7DA on beta. I divided the total # of other downloads by the sum of all downloads.

For the share histogram -- the total count in the new histogram may be larger than the count in Android.DownloadManager.Share.FileTypes "other" bucket because items with an "other" type that are shared with pages, videos, audio, etc. get counted in the "all" bucket of the existing histogram.
Labels: M-57
CL is in-flight: https://codereview.chromium.org/2563503002/
Status: Fixed (was: Assigned)
Jon, the new metrics are all under Android.DownloadManager.OtherExtensions....:
 - Android.DownloadManager.OtherExtensions.InitialCount -- this logs a histogram entry for the file extension type when downloads are populating; it's a bit different than the other initial count histograms but it's logged at the same time.
 - Android.DownloadManager.OtherExtensions.OpenFailed
 - Android.DownloadManager.OtherExtensions.OpenSucceeded
 - Android.DownloadManager.OtherExtensions.Share
Labels: -M-57 M-56 Merge-Request-56
Status: Started (was: Fixed)
Requesting merge to M56 so that we can get metrics 6 weeks sooner. This change only adds new metrics; it does not change any behavior.

Comment 6 by dimu@chromium.org, Dec 12 2016

Labels: -Merge-Request-56 Merge-Approved-56 Hotlist-Merge-Approved
Your change meets the bar and is auto-approved for M56 (branch: 2924)
Project Member

Comment 7 by bugdroid1@chromium.org, Dec 12 2016

Labels: -merge-approved-56 merge-merged-2924
The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/605af5f1e105f9645968a7d8fb7b8809560a6f38

commit 605af5f1e105f9645968a7d8fb7b8809560a6f38
Author: Theresa Wellington <twellington@google.com>
Date: Mon Dec 12 20:03:53 2016

[Downloads UI] Add histograms for file extensions

Add histograms that break down the file extension type
for "other" downloads.

BUG= 658312 

Review-Url: https://codereview.chromium.org/2563503002
Cr-Commit-Position: refs/heads/master@{#437127}
TBR=dfalcantara@chromium.org,holte@chromium.org

Review-Url: https://codereview.chromium.org/2568263002 .
Cr-Commit-Position: refs/branch-heads/2924@{#467}
Cr-Branched-From: 3a87aecc31cd1ffe751dd72c04e5a96a1fc8108a-refs/heads/master@{#433059}

[modify] https://crrev.com/605af5f1e105f9645968a7d8fb7b8809560a6f38/chrome/android/java/src/org/chromium/chrome/browser/download/DownloadUtils.java
[modify] https://crrev.com/605af5f1e105f9645968a7d8fb7b8809560a6f38/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadFilter.java
[modify] https://crrev.com/605af5f1e105f9645968a7d8fb7b8809560a6f38/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryAdapter.java
[modify] https://crrev.com/605af5f1e105f9645968a7d8fb7b8809560a6f38/chrome/android/java/src/org/chromium/chrome/browser/download/ui/DownloadHistoryItemWrapper.java
[modify] https://crrev.com/605af5f1e105f9645968a7d8fb7b8809560a6f38/tools/metrics/histograms/histograms.xml

Status: Fixed (was: Started)

Sign in to add a comment