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

Side by Side Diff: chrome/browser/chromeos/file_system_provider/provided_file_system.cc

Issue 239993002: [fsp] Create a RequestManager per a ProvidedFileSystem instance. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fixed and rebased. Created 6 years, 8 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 #include "chrome/browser/chromeos/file_system_provider/provided_file_system.h" 5 #include "chrome/browser/chromeos/file_system_provider/provided_file_system.h"
6 6
7 #include "base/files/file.h" 7 #include "base/files/file.h"
8 #include "base/values.h" 8 #include "base/values.h"
9 #include "chrome/browser/chromeos/file_system_provider/request_manager.h" 9 #include "chrome/browser/chromeos/file_system_provider/request_manager.h"
10 #include "chrome/common/extensions/api/file_system_provider.h" 10 #include "chrome/common/extensions/api/file_system_provider.h"
(...skipping 19 matching lines...) Expand all
30 const fileapi::AsyncFileUtil::StatusCallback& callback, 30 const fileapi::AsyncFileUtil::StatusCallback& callback,
31 scoped_ptr<base::DictionaryValue> /* result */, 31 scoped_ptr<base::DictionaryValue> /* result */,
32 bool /* has_next */) { 32 bool /* has_next */) {
33 callback.Run(base::File::FILE_OK); 33 callback.Run(base::File::FILE_OK);
34 } 34 }
35 35
36 } // namespace 36 } // namespace
37 37
38 ProvidedFileSystem::ProvidedFileSystem( 38 ProvidedFileSystem::ProvidedFileSystem(
39 extensions::EventRouter* event_router, 39 extensions::EventRouter* event_router,
40 RequestManager* request_manager,
41 const ProvidedFileSystemInfo& file_system_info) 40 const ProvidedFileSystemInfo& file_system_info)
42 : event_router_(event_router), 41 : event_router_(event_router), file_system_info_(file_system_info) {}
43 request_manager_(request_manager),
44 file_system_info_(file_system_info) {}
45 42
46 ProvidedFileSystem::~ProvidedFileSystem() {} 43 ProvidedFileSystem::~ProvidedFileSystem() {}
47 44
48 bool ProvidedFileSystem::RequestUnmount( 45 bool ProvidedFileSystem::RequestUnmount(
49 const fileapi::AsyncFileUtil::StatusCallback& callback) { 46 const fileapi::AsyncFileUtil::StatusCallback& callback) {
50 int request_id = request_manager_->CreateRequest( 47 int request_id = request_manager_.CreateRequest(
51 file_system_info_.extension_id(), 48 base::Bind(&OnRequestUnmountSuccess, callback), callback);
52 file_system_info_.file_system_id(),
53 base::Bind(&OnRequestUnmountSuccess, callback),
54 callback);
55 49
56 if (!request_id) 50 if (!request_id)
57 return false; 51 return false;
58 52
59 scoped_ptr<base::ListValue> values( 53 scoped_ptr<base::ListValue> values(
60 CreateRequestValues(file_system_info_.file_system_id(), request_id)); 54 CreateRequestValues(file_system_info_.file_system_id(), request_id));
61 55
62 event_router_->DispatchEventToExtension( 56 event_router_->DispatchEventToExtension(
63 file_system_info_.extension_id(), 57 file_system_info_.extension_id(),
64 make_scoped_ptr(new extensions::Event( 58 make_scoped_ptr(new extensions::Event(
65 extensions::api::file_system_provider::OnUnmountRequested::kEventName, 59 extensions::api::file_system_provider::OnUnmountRequested::kEventName,
66 values.Pass()))); 60 values.Pass())));
67 61
68 return true; 62 return true;
69 } 63 }
70 64
71 const ProvidedFileSystemInfo& ProvidedFileSystem::GetFileSystemInfo() const { 65 const ProvidedFileSystemInfo& ProvidedFileSystem::GetFileSystemInfo() const {
72 return file_system_info_; 66 return file_system_info_;
73 } 67 }
74 68
69 RequestManager* ProvidedFileSystem::GetRequestManager() {
70 return &request_manager_;
71 }
72
75 } // namespace file_system_provider 73 } // namespace file_system_provider
76 } // namespace chromeos 74 } // namespace chromeos
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698