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

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

Issue 55063002: Prefer opening PDF downloads in the browser. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Destroy PluginService once we are done with our plugin tests. 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
59 // Increment one of the above counts. 76 // Increment one of the above counts.
60 void RecordDownloadCount(ChromeDownloadCountTypes type); 77 void RecordDownloadCount(ChromeDownloadCountTypes type);
61 78
62 // Record initiation of a download from a specific source. 79 // Record initiation of a download from a specific source.
63 void RecordDownloadSource(ChromeDownloadSource source); 80 void RecordDownloadSource(ChromeDownloadSource source);
64 81
65 // Record that the user opened the confirmation dialog for a dangerous download. 82 // Record that the user opened the confirmation dialog for a dangerous download.
66 void RecordOpenedDangerousConfirmDialog( 83 void RecordOpenedDangerousConfirmDialog(
67 content::DownloadDangerType danger_type); 84 content::DownloadDangerType danger_type);
68 85
86 // Record how a download was opened.
87 void RecordDownloadOpenMethod(ChromeDownloadOpenMethod open_method);
88
69 #endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_STATS_H_ 89 #endif // CHROME_BROWSER_DOWNLOAD_DOWNLOAD_STATS_H_
OLDNEW
« no previous file with comments | « chrome/browser/download/download_shelf_context_menu.cc ('k') | chrome/browser/download/download_stats.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698