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

Side by Side Diff: content/shell/browser/shell_download_manager_delegate.h

Issue 671663002: Standardize usage of virtual/override/final in content/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: 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 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 CONTENT_SHELL_BROWSER_SHELL_DOWNLOAD_MANAGER_DELEGATE_H_ 5 #ifndef CONTENT_SHELL_BROWSER_SHELL_DOWNLOAD_MANAGER_DELEGATE_H_
6 #define CONTENT_SHELL_BROWSER_SHELL_DOWNLOAD_MANAGER_DELEGATE_H_ 6 #define CONTENT_SHELL_BROWSER_SHELL_DOWNLOAD_MANAGER_DELEGATE_H_
7 7
8 #include "base/callback_forward.h" 8 #include "base/callback_forward.h"
9 #include "base/compiler_specific.h" 9 #include "base/compiler_specific.h"
10 #include "base/memory/weak_ptr.h" 10 #include "base/memory/weak_ptr.h"
11 #include "content/public/browser/download_manager_delegate.h" 11 #include "content/public/browser/download_manager_delegate.h"
12 12
13 namespace content { 13 namespace content {
14 14
15 class DownloadManager; 15 class DownloadManager;
16 16
17 class ShellDownloadManagerDelegate : public DownloadManagerDelegate { 17 class ShellDownloadManagerDelegate : public DownloadManagerDelegate {
18 public: 18 public:
19 ShellDownloadManagerDelegate(); 19 ShellDownloadManagerDelegate();
20 virtual ~ShellDownloadManagerDelegate(); 20 ~ShellDownloadManagerDelegate() override;
21 21
22 void SetDownloadManager(DownloadManager* manager); 22 void SetDownloadManager(DownloadManager* manager);
23 23
24 virtual void Shutdown() override; 24 void Shutdown() override;
25 virtual bool DetermineDownloadTarget( 25 bool DetermineDownloadTarget(DownloadItem* download,
26 DownloadItem* download, 26 const DownloadTargetCallback& callback) override;
27 const DownloadTargetCallback& callback) override; 27 bool ShouldOpenDownload(DownloadItem* item,
28 virtual bool ShouldOpenDownload( 28 const DownloadOpenDelayedCallback& callback) override;
29 DownloadItem* item, 29 void GetNextId(const DownloadIdCallback& callback) override;
30 const DownloadOpenDelayedCallback& callback) override;
31 virtual void GetNextId(const DownloadIdCallback& callback) override;
32 30
33 // Inhibits prompting and sets the default download path. 31 // Inhibits prompting and sets the default download path.
34 void SetDownloadBehaviorForTesting( 32 void SetDownloadBehaviorForTesting(
35 const base::FilePath& default_download_path); 33 const base::FilePath& default_download_path);
36 34
37 private: 35 private:
38 friend class base::RefCountedThreadSafe<ShellDownloadManagerDelegate>; 36 friend class base::RefCountedThreadSafe<ShellDownloadManagerDelegate>;
39 37
40 typedef base::Callback<void(const base::FilePath&)> 38 typedef base::Callback<void(const base::FilePath&)>
41 FilenameDeterminedCallback; 39 FilenameDeterminedCallback;
(...skipping 15 matching lines...) Expand all
57 base::FilePath default_download_path_; 55 base::FilePath default_download_path_;
58 bool suppress_prompting_; 56 bool suppress_prompting_;
59 base::WeakPtrFactory<ShellDownloadManagerDelegate> weak_ptr_factory_; 57 base::WeakPtrFactory<ShellDownloadManagerDelegate> weak_ptr_factory_;
60 58
61 DISALLOW_COPY_AND_ASSIGN(ShellDownloadManagerDelegate); 59 DISALLOW_COPY_AND_ASSIGN(ShellDownloadManagerDelegate);
62 }; 60 };
63 61
64 } // namespace content 62 } // namespace content
65 63
66 #endif // CONTENT_SHELL_BROWSER_SHELL_DOWNLOAD_MANAGER_DELEGATE_H_ 64 #endif // CONTENT_SHELL_BROWSER_SHELL_DOWNLOAD_MANAGER_DELEGATE_H_
OLDNEW
« no previous file with comments | « content/shell/browser/shell_devtools_frontend.h ('k') | content/shell/browser/shell_javascript_dialog_manager.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698