Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(56)

Side by Side Diff: trunk/src/chrome/browser/download/download_stats.h

Issue 61623006: Revert 233460 "Prefer opening PDF downloads in the browser." (Closed) Base URL: svn://svn.chromium.org/chrome/
Patch Set: Created 7 years, 1 month ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_STATS_H_ 5 #ifndef CHROME_BROWSER_DOWNLOAD_DOWNLOAD_STATS_H_
6 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_STATS_H_ 6 #define CHROME_BROWSER_DOWNLOAD_DOWNLOAD_STATS_H_
7 7
8 #include "content/public/browser/download_danger_type.h" 8 #include "content/public/browser/download_danger_type.h"
9 9
10 // Record the total number of items and the number of in-progress items showing 10 // Record the total number of items and the number of in-progress items showing
(...skipping 38 matching lines...) Expand 10 before | Expand all | Expand 10 after
49 49
50 // The download was initiated by the PDF plugin.. 50 // The download was initiated by the PDF plugin..
51 DOWNLOAD_INITIATED_BY_PDF_SAVE, 51 DOWNLOAD_INITIATED_BY_PDF_SAVE,
52 52
53 // The download was initiated by chrome.downloads.download(). 53 // The download was initiated by chrome.downloads.download().
54 DOWNLOAD_INITIATED_BY_EXTENSION, 54 DOWNLOAD_INITIATED_BY_EXTENSION,
55 55
56 CHROME_DOWNLOAD_SOURCE_LAST_ENTRY, 56 CHROME_DOWNLOAD_SOURCE_LAST_ENTRY,
57 }; 57 };
58 58
59 // How a download was opened. Note that a download could be opened multiple
60 // times.
61 enum ChromeDownloadOpenMethod {
62 // The download was opened using the platform handler. There was no special
63 // handling for this download.
64 DOWNLOAD_OPEN_METHOD_DEFAULT_PLATFORM = 0,
65
66 // The download was opened using the browser bypassing the system handler.
67 DOWNLOAD_OPEN_METHOD_DEFAULT_BROWSER,
68
69 // The user chose to open the download using the system handler even though
70 // the preferred method was to open the download using the browser.
71 DOWNLOAD_OPEN_METHOD_USER_PLATFORM,
72
73 DOWNLOAD_OPEN_METHOD_LAST_ENTRY
74 };
75
76 // Increment one of the above counts. 59 // Increment one of the above counts.
77 void RecordDownloadCount(ChromeDownloadCountTypes type); 60 void RecordDownloadCount(ChromeDownloadCountTypes type);
78 61
79 // Record initiation of a download from a specific source. 62 // Record initiation of a download from a specific source.
80 void RecordDownloadSource(ChromeDownloadSource source); 63 void RecordDownloadSource(ChromeDownloadSource source);
81 64
82 // Record that the user opened the confirmation dialog for a dangerous download. 65 // Record that the user opened the confirmation dialog for a dangerous download.
83 void RecordOpenedDangerousConfirmDialog( 66 void RecordOpenedDangerousConfirmDialog(
84 content::DownloadDangerType danger_type); 67 content::DownloadDangerType danger_type);
85 68
86 // Record how a download was opened.
87 void RecordDownloadOpenMethod(ChromeDownloadOpenMethod open_method);
88
89 #endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_STATS_H_ 69 #endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_STATS_H_
OLDNEW
« no previous file with comments | « trunk/src/chrome/browser/download/download_shelf_context_menu.cc ('k') | trunk/src/chrome/browser/download/download_stats.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698