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

Side by Side Diff: content/public/browser/download_destination_observer.h

Issue 12224027: Revert "GTTF: Add missing virtual destructors." (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: rebase Created 7 years, 10 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 CONTENT_PUBLIC_BROWSER_DOWNLOAD_DESTINATION_OBSERVER_H_ 5 #ifndef CONTENT_PUBLIC_BROWSER_DOWNLOAD_DESTINATION_OBSERVER_H_
6 #define CONTENT_PUBLIC_BROWSER_DOWNLOAD_DESTINATION_OBSERVER_H_ 6 #define CONTENT_PUBLIC_BROWSER_DOWNLOAD_DESTINATION_OBSERVER_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "content/public/browser/download_interrupt_reasons.h" 10 #include "content/public/browser/download_interrupt_reasons.h"
11 11
12 namespace content { 12 namespace content {
13 13
14 // Class that receives asynchronous events from a DownloadDestination about 14 // Class that receives asynchronous events from a DownloadDestination about
15 // downloading progress and completion. These should report status when the 15 // downloading progress and completion. These should report status when the
16 // data arrives at its final location; i.e. DestinationUpdate should be 16 // data arrives at its final location; i.e. DestinationUpdate should be
17 // called after the destination is finished with whatever operation it 17 // called after the destination is finished with whatever operation it
18 // is doing on the data described by |bytes_so_far| and DestinationCompleted 18 // is doing on the data described by |bytes_so_far| and DestinationCompleted
19 // should only be called once that is true for all data. 19 // should only be called once that is true for all data.
20 // 20 //
21 // All methods are invoked on the UI thread. 21 // All methods are invoked on the UI thread.
22 // 22 //
23 // Note that this interface does not deal with cross-thread lifetime issues. 23 // Note that this interface does not deal with cross-thread lifetime issues.
24 class DownloadDestinationObserver { 24 class DownloadDestinationObserver {
25 public: 25 public:
26 virtual ~DownloadDestinationObserver() { }
27
28 virtual void DestinationUpdate(int64 bytes_so_far, 26 virtual void DestinationUpdate(int64 bytes_so_far,
29 int64 bytes_per_sec, 27 int64 bytes_per_sec,
30 const std::string& hash_state) = 0; 28 const std::string& hash_state) = 0;
31 29
32 virtual void DestinationError(DownloadInterruptReason reason) = 0; 30 virtual void DestinationError(DownloadInterruptReason reason) = 0;
33 31
34 virtual void DestinationCompleted(const std::string& final_hash) = 0; 32 virtual void DestinationCompleted(const std::string& final_hash) = 0;
35 }; 33 };
36 34
37 } // namespace content 35 } // namespace content
38 36
39 #endif // CONTENT_PUBLIC_BROWSER_DOWNLOAD_DESTINATION_OBSERVER_H_ 37 #endif // CONTENT_PUBLIC_BROWSER_DOWNLOAD_DESTINATION_OBSERVER_H_
OLDNEW
« no previous file with comments | « content/browser/download/download_item_impl.h ('k') | content/public/utility/content_utility_client.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698