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

Unified Diff: chrome/browser/chromeos/extensions/file_system_provider/file_system_provider_api.cc

Issue 50703013: fileSystemProvider: First cut at implementing fileSystemProvider API (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: address comments Created 7 years, 1 month 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/extensions/file_system_provider/file_system_provider_api.cc
diff --git a/chrome/browser/chromeos/extensions/file_system_provider/file_system_provider_api.cc b/chrome/browser/chromeos/extensions/file_system_provider/file_system_provider_api.cc
new file mode 100644
index 0000000000000000000000000000000000000000..0d9fc0dd5d06f3c22357c0fa5b50f9d9bbd1d1e1
--- /dev/null
+++ b/chrome/browser/chromeos/extensions/file_system_provider/file_system_provider_api.cc
@@ -0,0 +1,63 @@
+// Copyright 2013 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/extensions/file_system_provider/file_system_provider_api.h"
+
+#include "chrome/common/extensions/api/file_system_provider.h"
+
+namespace extensions {
+
+namespace {
+
+// Error names from
+// http://www.w3.org/TR/file-system-api/#errors-and-exceptions
+const char kSecurityErrorName[] = "SecurityError";
+
+// Error messages.
+const char kEmptyNameErrorMessage[] = "Empty display name is not allowed";
+
+// Creates a dictionary, which looks like a DOMError. The returned dictionary
+// will be converted to a real DOMError object in
+// file_system_provier_custom_bindings.js.
+base::DictionaryValue* CreateError(const std::string& name,
+ const std::string& message) {
+ base::DictionaryValue* error = new base::DictionaryValue();
+ error->SetString("name", name);
+ error->SetString("message", message);
+ return error;
+}
+
+} // namespace
+
+bool FileSystemProviderMountFunction::RunImpl() {
+ using extensions::api::file_system_provider::Mount::Params;
+ const scoped_ptr<Params> params(Params::Create(*args_));
+ EXTENSION_FUNCTION_VALIDATE(params);
+
+ // It's an error if the display name is empty.
+ if (params->display_name.empty()) {
+ const std::string file_system_id = "";
+
+ base::ListValue* result = new base::ListValue();
+ result->Append(new base::StringValue(file_system_id));
+ result->Append(CreateError(kSecurityErrorName,
+ kEmptyNameErrorMessage));
+ SetResult(result);
+ SendResponse(true);
+ return true;
+ }
+
+ // TODO(satorux): Implement the real logic.
+ const std::string file_system_id = params->display_name;
+
+ base::ListValue* result = new base::ListValue();
+ result->Append(new base::StringValue(file_system_id));
+ // Don't append an error on success.
+
+ SetResult(result);
+ SendResponse(true);
+ return true;
+}
+
+} // namespace extensions

Powered by Google App Engine
This is Rietveld 408576698