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

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

Issue 2901563003: PDF: Add UMA to track embedded PDF triggered drive-by downloads. (Closed)
Patch Set: Created 3 years, 7 months 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
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_CHROME_DOWNLOAD_MANAGER_DELEGATE_H_ 5 #ifndef CHROME_BROWSER_DOWNLOAD_CHROME_DOWNLOAD_MANAGER_DELEGATE_H_
6 #define CHROME_BROWSER_DOWNLOAD_CHROME_DOWNLOAD_MANAGER_DELEGATE_H_ 6 #define CHROME_BROWSER_DOWNLOAD_CHROME_DOWNLOAD_MANAGER_DELEGATE_H_
7 7
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <memory> 10 #include <memory>
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 DownloadConfirmationReason reason, 114 DownloadConfirmationReason reason,
115 const ConfirmationCallback& callback) override; 115 const ConfirmationCallback& callback) override;
116 void DetermineLocalPath(content::DownloadItem* download, 116 void DetermineLocalPath(content::DownloadItem* download,
117 const base::FilePath& virtual_path, 117 const base::FilePath& virtual_path,
118 const LocalPathCallback& callback) override; 118 const LocalPathCallback& callback) override;
119 void CheckDownloadUrl(content::DownloadItem* download, 119 void CheckDownloadUrl(content::DownloadItem* download,
120 const base::FilePath& suggested_virtual_path, 120 const base::FilePath& suggested_virtual_path,
121 const CheckDownloadUrlCallback& callback) override; 121 const CheckDownloadUrlCallback& callback) override;
122 void GetFileMimeType(const base::FilePath& path, 122 void GetFileMimeType(const base::FilePath& path,
123 const GetFileMimeTypeCallback& callback) override; 123 const GetFileMimeTypeCallback& callback) override;
124 void RecordDriveByPDFDownload() const override;
124 125
125 // So that test classes that inherit from this for override purposes 126 // So that test classes that inherit from this for override purposes
126 // can call back into the DownloadManager. 127 // can call back into the DownloadManager.
127 content::DownloadManager* download_manager_; 128 content::DownloadManager* download_manager_;
128 129
129 private: 130 private:
130 friend class base::RefCountedThreadSafe<ChromeDownloadManagerDelegate>; 131 friend class base::RefCountedThreadSafe<ChromeDownloadManagerDelegate>;
131 FRIEND_TEST_ALL_PREFIXES(ChromeDownloadManagerDelegateTest, 132 FRIEND_TEST_ALL_PREFIXES(ChromeDownloadManagerDelegateTest,
132 RequestConfirmation_Android); 133 RequestConfirmation_Android);
133 134
(...skipping 42 matching lines...) Expand 10 before | Expand all | Expand 10 after
176 #endif 177 #endif
177 178
178 content::NotificationRegistrar registrar_; 179 content::NotificationRegistrar registrar_;
179 180
180 base::WeakPtrFactory<ChromeDownloadManagerDelegate> weak_ptr_factory_; 181 base::WeakPtrFactory<ChromeDownloadManagerDelegate> weak_ptr_factory_;
181 182
182 DISALLOW_COPY_AND_ASSIGN(ChromeDownloadManagerDelegate); 183 DISALLOW_COPY_AND_ASSIGN(ChromeDownloadManagerDelegate);
183 }; 184 };
184 185
185 #endif // CHROME_BROWSER_DOWNLOAD_CHROME_DOWNLOAD_MANAGER_DELEGATE_H_ 186 #endif // CHROME_BROWSER_DOWNLOAD_CHROME_DOWNLOAD_MANAGER_DELEGATE_H_
OLDNEW
« no previous file with comments | « no previous file | chrome/browser/download/chrome_download_manager_delegate.cc » ('j') | chrome/common/pdf_uma.h » ('J')

Powered by Google App Engine
This is Rietveld 408576698