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

Side by Side Diff: chrome/browser/chromeos/file_system_provider/operations/operation.h

Issue 335753004: [fsp] Cleanup handling errors for operation requests. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebased. Created 6 years, 6 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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_CHROMEOS_FILE_SYSTEM_PROVIDER_OPERATIONS_OPERATION_H_ 5 #ifndef CHROME_BROWSER_CHROMEOS_FILE_SYSTEM_PROVIDER_OPERATIONS_OPERATION_H_
6 #define CHROME_BROWSER_CHROMEOS_FILE_SYSTEM_PROVIDER_OPERATIONS_OPERATION_H_ 6 #define CHROME_BROWSER_CHROMEOS_FILE_SYSTEM_PROVIDER_OPERATIONS_OPERATION_H_
7 7
8 #include <string> 8 #include <string>
9 9
10 #include "base/files/file.h" 10 #include "base/files/file.h"
(...skipping 23 matching lines...) Expand all
34 34
35 Operation(extensions::EventRouter* event_router, 35 Operation(extensions::EventRouter* event_router,
36 const ProvidedFileSystemInfo& file_system_info); 36 const ProvidedFileSystemInfo& file_system_info);
37 virtual ~Operation(); 37 virtual ~Operation();
38 38
39 // RequestManager::HandlerInterface overrides. 39 // RequestManager::HandlerInterface overrides.
40 virtual bool Execute(int request_id) OVERRIDE = 0; 40 virtual bool Execute(int request_id) OVERRIDE = 0;
41 virtual void OnSuccess(int request_id, 41 virtual void OnSuccess(int request_id,
42 scoped_ptr<RequestValue> result, 42 scoped_ptr<RequestValue> result,
43 bool has_more) OVERRIDE = 0; 43 bool has_more) OVERRIDE = 0;
44 virtual void OnError(int request_id, base::File::Error error) OVERRIDE = 0; 44 virtual void OnError(int request_id,
45 scoped_ptr<RequestValue> result,
46 base::File::Error error) OVERRIDE = 0;
45 47
46 // Sets custom dispatchign event implementation for tests. 48 // Sets custom dispatchign event implementation for tests.
47 void SetDispatchEventImplForTesting( 49 void SetDispatchEventImplForTesting(
48 const DispatchEventImplCallback& callback); 50 const DispatchEventImplCallback& callback);
49 51
50 protected: 52 protected:
51 // Sends an event to the providing extension. Automatically adds the file 53 // Sends an event to the providing extension. Automatically adds the file
52 // system id and the request id fields. Returns false, if the providing 54 // system id and the request id fields. Returns false, if the providing
53 // extension does not handle the |event_name| event. 55 // extension does not handle the |event_name| event.
54 bool SendEvent(int request_id, 56 bool SendEvent(int request_id,
55 const std::string& event_name, 57 const std::string& event_name,
56 scoped_ptr<base::DictionaryValue> options); 58 scoped_ptr<base::DictionaryValue> options);
57 59
58 ProvidedFileSystemInfo file_system_info_; 60 ProvidedFileSystemInfo file_system_info_;
59 61
60 private: 62 private:
61 DispatchEventImplCallback dispatch_event_impl_; 63 DispatchEventImplCallback dispatch_event_impl_;
62 DISALLOW_COPY_AND_ASSIGN(Operation); 64 DISALLOW_COPY_AND_ASSIGN(Operation);
63 }; 65 };
64 66
65 } // namespace operations 67 } // namespace operations
66 } // namespace file_system_provider 68 } // namespace file_system_provider
67 } // namespace chromeos 69 } // namespace chromeos
68 70
69 #endif // CHROME_BROWSER_CHROMEOS_FILE_SYSTEM_PROVIDER_OPERATIONS_OPERATION_H_ 71 #endif // CHROME_BROWSER_CHROMEOS_FILE_SYSTEM_PROVIDER_OPERATIONS_OPERATION_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698