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

Unified Diff: chrome/browser/chromeos/file_system_provider/operations/configure.cc

Issue 1088883002: Add events for configuring and adding new providers to FSP API. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixed. Created 5 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 side-by-side diff with in-line comments
Download patch
Index: chrome/browser/chromeos/file_system_provider/operations/configure.cc
diff --git a/chrome/browser/chromeos/file_system_provider/operations/close_file.cc b/chrome/browser/chromeos/file_system_provider/operations/configure.cc
similarity index 50%
copy from chrome/browser/chromeos/file_system_provider/operations/close_file.cc
copy to chrome/browser/chromeos/file_system_provider/operations/configure.cc
index 321fd9c84a2f2e4fd6467399ac77c65052bc1ceb..c2b7d68467343d560370785aea161d93f49dc2dd 100644
--- a/chrome/browser/chromeos/file_system_provider/operations/close_file.cc
+++ b/chrome/browser/chromeos/file_system_provider/operations/configure.cc
@@ -1,52 +1,46 @@
-// Copyright 2014 The Chromium Authors. All rights reserved.
+// Copyright 2015 The Chromium Authors. All rights reserved.
// Use of this source code is governed by a BSD-style license that can be
// found in the LICENSE file.
-#include "chrome/browser/chromeos/file_system_provider/operations/close_file.h"
-
-#include <string>
+#include "chrome/browser/chromeos/file_system_provider/operations/configure.h"
+#include "base/values.h"
#include "chrome/common/extensions/api/file_system_provider.h"
-#include "chrome/common/extensions/api/file_system_provider_internal.h"
namespace chromeos {
namespace file_system_provider {
namespace operations {
-CloseFile::CloseFile(extensions::EventRouter* event_router,
+Configure::Configure(extensions::EventRouter* event_router,
const ProvidedFileSystemInfo& file_system_info,
- int open_request_id,
const storage::AsyncFileUtil::StatusCallback& callback)
- : Operation(event_router, file_system_info),
- open_request_id_(open_request_id),
- callback_(callback) {
+ : Operation(event_router, file_system_info), callback_(callback) {
}
-CloseFile::~CloseFile() {
+Configure::~Configure() {
}
-bool CloseFile::Execute(int request_id) {
- using extensions::api::file_system_provider::CloseFileRequestedOptions;
+bool Configure::Execute(int request_id) {
+ using extensions::api::file_system_provider::ConfigureRequestedOptions;
- CloseFileRequestedOptions options;
+ ConfigureRequestedOptions options;
options.file_system_id = file_system_info_.file_system_id();
options.request_id = request_id;
- options.open_request_id = open_request_id_;
return SendEvent(
request_id,
- extensions::api::file_system_provider::OnCloseFileRequested::kEventName,
- extensions::api::file_system_provider::OnCloseFileRequested::Create(
+ extensions::api::file_system_provider::OnConfigureRequested::kEventName,
+ extensions::api::file_system_provider::OnConfigureRequested::Create(
options));
}
-void CloseFile::OnSuccess(int /* request_id */,
- scoped_ptr<RequestValue> result,
- bool has_more) {
+void Configure::OnSuccess(int /* request_id */,
+ scoped_ptr<RequestValue> /* result */,
+ bool /* has_more */) {
callback_.Run(base::File::FILE_OK);
}
-void CloseFile::OnError(int /* request_id */,
+void Configure::OnError(int /* request_id */,
scoped_ptr<RequestValue> /* result */,
base::File::Error error) {
callback_.Run(error);

Powered by Google App Engine
This is Rietveld 408576698