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

Side by Side Diff: chrome/browser/extensions/api/downloads/downloads_api.h

Issue 11419224: Add missing (and remove superfluous) 'explicit' from constructors. Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase + remove non-straightforward changes Created 7 years, 11 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 | Annotate | Revision Log
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_EXTENSIONS_API_DOWNLOADS_DOWNLOADS_API_H_ 5 #ifndef CHROME_BROWSER_EXTENSIONS_API_DOWNLOADS_DOWNLOADS_API_H_
6 #define CHROME_BROWSER_EXTENSIONS_API_DOWNLOADS_DOWNLOADS_API_H_ 6 #define CHROME_BROWSER_EXTENSIONS_API_DOWNLOADS_DOWNLOADS_API_H_
7 7
8 #include <map> 8 #include <map>
9 #include <set> 9 #include <set>
10 #include <string> 10 #include <string>
(...skipping 197 matching lines...) Expand 10 before | Expand all | Expand 10 after
208 FilePath path_; 208 FilePath path_;
209 scoped_ptr<DownloadFileIconExtractor> icon_extractor_; 209 scoped_ptr<DownloadFileIconExtractor> icon_extractor_;
210 DISALLOW_COPY_AND_ASSIGN(DownloadsGetFileIconFunction); 210 DISALLOW_COPY_AND_ASSIGN(DownloadsGetFileIconFunction);
211 }; 211 };
212 212
213 // Observes a single DownloadManager and many DownloadItems and dispatches 213 // Observes a single DownloadManager and many DownloadItems and dispatches
214 // onCreated and onErased events. 214 // onCreated and onErased events.
215 class ExtensionDownloadsEventRouter 215 class ExtensionDownloadsEventRouter
216 : public AllDownloadItemNotifier::Observer { 216 : public AllDownloadItemNotifier::Observer {
217 public: 217 public:
218 explicit ExtensionDownloadsEventRouter( 218 ExtensionDownloadsEventRouter(
219 Profile* profile, content::DownloadManager* manager); 219 Profile* profile, content::DownloadManager* manager);
220 virtual ~ExtensionDownloadsEventRouter(); 220 virtual ~ExtensionDownloadsEventRouter();
221 221
222 // AllDownloadItemNotifier::Observer interface 222 // AllDownloadItemNotifier::Observer interface
223 virtual void OnDownloadCreated( 223 virtual void OnDownloadCreated(
224 content::DownloadManager* manager, 224 content::DownloadManager* manager,
225 content::DownloadItem* download_item) OVERRIDE; 225 content::DownloadItem* download_item) OVERRIDE;
226 virtual void OnDownloadUpdated( 226 virtual void OnDownloadUpdated(
227 content::DownloadManager* manager, 227 content::DownloadManager* manager,
228 content::DownloadItem* download_item) OVERRIDE; 228 content::DownloadItem* download_item) OVERRIDE;
(...skipping 10 matching lines...) Expand all
239 private: 239 private:
240 void DispatchEvent(const char* event_name, base::Value* json_arg); 240 void DispatchEvent(const char* event_name, base::Value* json_arg);
241 241
242 Profile* profile_; 242 Profile* profile_;
243 AllDownloadItemNotifier notifier_; 243 AllDownloadItemNotifier notifier_;
244 244
245 DISALLOW_COPY_AND_ASSIGN(ExtensionDownloadsEventRouter); 245 DISALLOW_COPY_AND_ASSIGN(ExtensionDownloadsEventRouter);
246 }; 246 };
247 247
248 #endif // CHROME_BROWSER_EXTENSIONS_API_DOWNLOADS_DOWNLOADS_API_H_ 248 #endif // CHROME_BROWSER_EXTENSIONS_API_DOWNLOADS_DOWNLOADS_API_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698