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

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

Issue 625113002: replace OVERRIDE and FINAL with override and final in chrome/browser/[a-i]* (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix newly added OVERRIDEs Created 6 years, 2 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 "base/compiler_specific.h" 8 #include "base/compiler_specific.h"
9 #include "base/containers/hash_tables.h" 9 #include "base/containers/hash_tables.h"
10 #include "base/memory/scoped_ptr.h" 10 #include "base/memory/scoped_ptr.h"
(...skipping 36 matching lines...) Expand 10 before | Expand all | Expand 10 after
47 // disable SafeBrowsing checks for |item|. 47 // disable SafeBrowsing checks for |item|.
48 static void DisableSafeBrowsing(content::DownloadItem* item); 48 static void DisableSafeBrowsing(content::DownloadItem* item);
49 49
50 void SetDownloadManager(content::DownloadManager* dm); 50 void SetDownloadManager(content::DownloadManager* dm);
51 51
52 // Callbacks passed to GetNextId() will not be called until the returned 52 // Callbacks passed to GetNextId() will not be called until the returned
53 // callback is called. 53 // callback is called.
54 content::DownloadIdCallback GetDownloadIdReceiverCallback(); 54 content::DownloadIdCallback GetDownloadIdReceiverCallback();
55 55
56 // content::DownloadManagerDelegate 56 // content::DownloadManagerDelegate
57 virtual void Shutdown() OVERRIDE; 57 virtual void Shutdown() override;
58 virtual void GetNextId(const content::DownloadIdCallback& callback) OVERRIDE; 58 virtual void GetNextId(const content::DownloadIdCallback& callback) override;
59 virtual bool DetermineDownloadTarget( 59 virtual bool DetermineDownloadTarget(
60 content::DownloadItem* item, 60 content::DownloadItem* item,
61 const content::DownloadTargetCallback& callback) OVERRIDE; 61 const content::DownloadTargetCallback& callback) override;
62 virtual bool ShouldOpenFileBasedOnExtension( 62 virtual bool ShouldOpenFileBasedOnExtension(
63 const base::FilePath& path) OVERRIDE; 63 const base::FilePath& path) override;
64 virtual bool ShouldCompleteDownload( 64 virtual bool ShouldCompleteDownload(
65 content::DownloadItem* item, 65 content::DownloadItem* item,
66 const base::Closure& complete_callback) OVERRIDE; 66 const base::Closure& complete_callback) override;
67 virtual bool ShouldOpenDownload( 67 virtual bool ShouldOpenDownload(
68 content::DownloadItem* item, 68 content::DownloadItem* item,
69 const content::DownloadOpenDelayedCallback& callback) OVERRIDE; 69 const content::DownloadOpenDelayedCallback& callback) override;
70 virtual bool GenerateFileHash() OVERRIDE; 70 virtual bool GenerateFileHash() override;
71 virtual void GetSaveDir(content::BrowserContext* browser_context, 71 virtual void GetSaveDir(content::BrowserContext* browser_context,
72 base::FilePath* website_save_dir, 72 base::FilePath* website_save_dir,
73 base::FilePath* download_save_dir, 73 base::FilePath* download_save_dir,
74 bool* skip_dir_check) OVERRIDE; 74 bool* skip_dir_check) override;
75 virtual void ChooseSavePath( 75 virtual void ChooseSavePath(
76 content::WebContents* web_contents, 76 content::WebContents* web_contents,
77 const base::FilePath& suggested_path, 77 const base::FilePath& suggested_path,
78 const base::FilePath::StringType& default_extension, 78 const base::FilePath::StringType& default_extension,
79 bool can_save_as_complete, 79 bool can_save_as_complete,
80 const content::SavePackagePathPickedCallback& callback) OVERRIDE; 80 const content::SavePackagePathPickedCallback& callback) override;
81 virtual void OpenDownload(content::DownloadItem* download) OVERRIDE; 81 virtual void OpenDownload(content::DownloadItem* download) override;
82 virtual void ShowDownloadInShell(content::DownloadItem* download) OVERRIDE; 82 virtual void ShowDownloadInShell(content::DownloadItem* download) override;
83 virtual void CheckForFileExistence( 83 virtual void CheckForFileExistence(
84 content::DownloadItem* download, 84 content::DownloadItem* download,
85 const content::CheckForFileExistenceCallback& callback) OVERRIDE; 85 const content::CheckForFileExistenceCallback& callback) override;
86 virtual std::string ApplicationClientIdForFileScanning() const OVERRIDE; 86 virtual std::string ApplicationClientIdForFileScanning() const override;
87 87
88 // Opens a download using the platform handler. DownloadItem::OpenDownload, 88 // Opens a download using the platform handler. DownloadItem::OpenDownload,
89 // which ends up being handled by OpenDownload(), will open a download in the 89 // which ends up being handled by OpenDownload(), will open a download in the
90 // browser if doing so is preferred. 90 // browser if doing so is preferred.
91 void OpenDownloadUsingPlatformHandler(content::DownloadItem* download); 91 void OpenDownloadUsingPlatformHandler(content::DownloadItem* download);
92 92
93 DownloadPrefs* download_prefs() { return download_prefs_.get(); } 93 DownloadPrefs* download_prefs() { return download_prefs_.get(); }
94 94
95 protected: 95 protected:
96 // So that test classes that inherit from this for override purposes 96 // So that test classes that inherit from this for override purposes
97 // can call back into the DownloadManager. 97 // can call back into the DownloadManager.
98 content::DownloadManager* download_manager_; 98 content::DownloadManager* download_manager_;
99 99
100 virtual safe_browsing::DownloadProtectionService* 100 virtual safe_browsing::DownloadProtectionService*
101 GetDownloadProtectionService(); 101 GetDownloadProtectionService();
102 102
103 // DownloadTargetDeterminerDelegate. Protected for testing. 103 // DownloadTargetDeterminerDelegate. Protected for testing.
104 virtual void NotifyExtensions( 104 virtual void NotifyExtensions(
105 content::DownloadItem* download, 105 content::DownloadItem* download,
106 const base::FilePath& suggested_virtual_path, 106 const base::FilePath& suggested_virtual_path,
107 const NotifyExtensionsCallback& callback) OVERRIDE; 107 const NotifyExtensionsCallback& callback) override;
108 virtual void ReserveVirtualPath( 108 virtual void ReserveVirtualPath(
109 content::DownloadItem* download, 109 content::DownloadItem* download,
110 const base::FilePath& virtual_path, 110 const base::FilePath& virtual_path,
111 bool create_directory, 111 bool create_directory,
112 DownloadPathReservationTracker::FilenameConflictAction conflict_action, 112 DownloadPathReservationTracker::FilenameConflictAction conflict_action,
113 const ReservedPathCallback& callback) OVERRIDE; 113 const ReservedPathCallback& callback) override;
114 virtual void PromptUserForDownloadPath( 114 virtual void PromptUserForDownloadPath(
115 content::DownloadItem* download, 115 content::DownloadItem* download,
116 const base::FilePath& suggested_virtual_path, 116 const base::FilePath& suggested_virtual_path,
117 const FileSelectedCallback& callback) OVERRIDE; 117 const FileSelectedCallback& callback) override;
118 virtual void DetermineLocalPath( 118 virtual void DetermineLocalPath(
119 content::DownloadItem* download, 119 content::DownloadItem* download,
120 const base::FilePath& virtual_path, 120 const base::FilePath& virtual_path,
121 const LocalPathCallback& callback) OVERRIDE; 121 const LocalPathCallback& callback) override;
122 virtual void CheckDownloadUrl( 122 virtual void CheckDownloadUrl(
123 content::DownloadItem* download, 123 content::DownloadItem* download,
124 const base::FilePath& suggested_virtual_path, 124 const base::FilePath& suggested_virtual_path,
125 const CheckDownloadUrlCallback& callback) OVERRIDE; 125 const CheckDownloadUrlCallback& callback) override;
126 virtual void GetFileMimeType( 126 virtual void GetFileMimeType(
127 const base::FilePath& path, 127 const base::FilePath& path,
128 const GetFileMimeTypeCallback& callback) OVERRIDE; 128 const GetFileMimeTypeCallback& callback) override;
129 129
130 private: 130 private:
131 friend class base::RefCountedThreadSafe<ChromeDownloadManagerDelegate>; 131 friend class base::RefCountedThreadSafe<ChromeDownloadManagerDelegate>;
132 132
133 typedef std::vector<content::DownloadIdCallback> IdCallbackVector; 133 typedef std::vector<content::DownloadIdCallback> IdCallbackVector;
134 134
135 // content::NotificationObserver implementation. 135 // content::NotificationObserver implementation.
136 virtual void Observe(int type, 136 virtual void Observe(int type,
137 const content::NotificationSource& source, 137 const content::NotificationSource& source,
138 const content::NotificationDetails& details) OVERRIDE; 138 const content::NotificationDetails& details) override;
139 139
140 // Callback function after the DownloadProtectionService completes. 140 // Callback function after the DownloadProtectionService completes.
141 void CheckClientDownloadDone( 141 void CheckClientDownloadDone(
142 uint32 download_id, 142 uint32 download_id,
143 safe_browsing::DownloadProtectionService::DownloadCheckResult result); 143 safe_browsing::DownloadProtectionService::DownloadCheckResult result);
144 144
145 // Internal gateways for ShouldCompleteDownload(). 145 // Internal gateways for ShouldCompleteDownload().
146 bool IsDownloadReadyForCompletion( 146 bool IsDownloadReadyForCompletion(
147 content::DownloadItem* item, 147 content::DownloadItem* item,
148 const base::Closure& internal_complete_callback); 148 const base::Closure& internal_complete_callback);
(...skipping 26 matching lines...) Expand all
175 #endif 175 #endif
176 176
177 content::NotificationRegistrar registrar_; 177 content::NotificationRegistrar registrar_;
178 178
179 base::WeakPtrFactory<ChromeDownloadManagerDelegate> weak_ptr_factory_; 179 base::WeakPtrFactory<ChromeDownloadManagerDelegate> weak_ptr_factory_;
180 180
181 DISALLOW_COPY_AND_ASSIGN(ChromeDownloadManagerDelegate); 181 DISALLOW_COPY_AND_ASSIGN(ChromeDownloadManagerDelegate);
182 }; 182 };
183 183
184 #endif // CHROME_BROWSER_DOWNLOAD_CHROME_DOWNLOAD_MANAGER_DELEGATE_H_ 184 #endif // CHROME_BROWSER_DOWNLOAD_CHROME_DOWNLOAD_MANAGER_DELEGATE_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698