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

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

Issue 3071005: Download code cleanup patch of death: (Closed)
Patch Set: inline the dtor Created 10 years, 4 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) 2009-2010 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2009-2010 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_DRAG_DOWNLOAD_FILE_H_ 5 #ifndef CHROME_BROWSER_DOWNLOAD_DRAG_DOWNLOAD_FILE_H_
6 #define CHROME_BROWSER_DOWNLOAD_DRAG_DOWNLOAD_FILE_H_ 6 #define CHROME_BROWSER_DOWNLOAD_DRAG_DOWNLOAD_FILE_H_
7 #pragma once 7 #pragma once
8 8
9 #include "app/download_file_interface.h" 9 #include "app/download_file_interface.h"
10 #include "base/file_path.h" 10 #include "base/file_path.h"
(...skipping 34 matching lines...) Expand 10 before | Expand all | Expand 10 after
45 // Called on UI thread (MacOSX). 45 // Called on UI thread (MacOSX).
46 virtual bool Start(DownloadFileObserver* observer); 46 virtual bool Start(DownloadFileObserver* observer);
47 virtual void Stop(); 47 virtual void Stop();
48 #if defined(OS_WIN) 48 #if defined(OS_WIN)
49 virtual IStream* GetStream() { return NULL; } 49 virtual IStream* GetStream() { return NULL; }
50 #endif 50 #endif
51 51
52 // DownloadManager::Observer methods. 52 // DownloadManager::Observer methods.
53 // Called on UI thread. 53 // Called on UI thread.
54 virtual void ModelChanged(); 54 virtual void ModelChanged();
55 virtual void SetDownloads(std::vector<DownloadItem*>& downloads);
56 55
57 // DownloadItem::Observer methods. 56 // DownloadItem::Observer methods.
58 // Called on UI thread. 57 // Called on UI thread.
59 virtual void OnDownloadUpdated(DownloadItem* download); 58 virtual void OnDownloadUpdated(DownloadItem* download);
60 virtual void OnDownloadFileCompleted(DownloadItem* download); 59 virtual void OnDownloadFileCompleted(DownloadItem* download);
61 virtual void OnDownloadOpened(DownloadItem* download) { } 60 virtual void OnDownloadOpened(DownloadItem* download) { }
62 61
63 private: 62 private:
64 // Called on drag-and-drop thread (Windows). 63 // Called on drag-and-drop thread (Windows).
65 // Called on UI thread (Windows). 64 // Called on UI thread (Windows).
(...skipping 39 matching lines...) Expand 10 before | Expand all | Expand 10 after
105 #endif 104 #endif
106 105
107 // Access on UI thread. 106 // Access on UI thread.
108 DownloadManager* download_manager_; 107 DownloadManager* download_manager_;
109 bool download_item_observer_added_; 108 bool download_item_observer_added_;
110 109
111 DISALLOW_COPY_AND_ASSIGN(DragDownloadFile); 110 DISALLOW_COPY_AND_ASSIGN(DragDownloadFile);
112 }; 111 };
113 112
114 #endif // CHROME_BROWSER_DOWNLOAD_DRAG_DOWNLOAD_FILE_H_ 113 #endif // CHROME_BROWSER_DOWNLOAD_DRAG_DOWNLOAD_FILE_H_
OLDNEW
« no previous file with comments | « chrome/browser/download/download_manager.cc ('k') | chrome/browser/download/drag_download_file.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698