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

Unified Diff: chrome/browser/ui/intents/native_file_picker_service.cc

Issue 11071005: Add native file picker impl using SelectFileDialog. (Closed) Base URL: http://git.chromium.org/chromium/src.git@filePicker
Patch Set: Don't use "DeleteService" as a method name since Windows munges it at compile time. Created 8 years, 2 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/ui/intents/native_file_picker_service.cc
diff --git a/chrome/browser/ui/intents/native_file_picker_service.cc b/chrome/browser/ui/intents/native_file_picker_service.cc
new file mode 100644
index 0000000000000000000000000000000000000000..aa66057974a72f642284b81792eb0eeffc87d348
--- /dev/null
+++ b/chrome/browser/ui/intents/native_file_picker_service.cc
@@ -0,0 +1,144 @@
+// Copyright (c) 2012 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 <vector>
+
+#include "base/file_path.h"
+#include "base/logging.h"
+#include "base/string16.h"
+#include "base/utf_string_conversions.h"
+#include "chrome/browser/intents/intent_service_host.h"
+#include "chrome/browser/intents/native_services.h"
+#include "chrome/browser/intents/web_intents_util.h"
+#include "chrome/browser/platform_util.h"
+#include "content/public/browser/web_contents.h"
+#include "content/public/browser/web_intents_dispatcher.h"
+#include "googleurl/src/gurl.h"
+#include "grit/generated_resources.h"
+#include "net/base/mime_util.h"
+#include "ui/base/dialogs/select_file_dialog.h"
+#include "ui/base/dialogs/selected_file_info.h"
+#include "ui/base/l10n/l10n_util.h"
+#include "webkit/glue/web_intent_data.h"
+#include "webkit/glue/web_intent_service_data.h"
+
+namespace web_intents {
+namespace {
+
+void AddTypeInfo(
+ const std::string& mime_type,
+ ui::SelectFileDialog::FileTypeInfo* info) {
+
+ info->include_all_files = true;
+ info->extensions.resize(1);
+ net::GetExtensionsForMimeType(mime_type, &info->extensions.back());
+
+ // Provide a "helpful" description when possible.
+ int description_id = 0;
+ if (mime_type == "image/*")
+ description_id = IDS_IMAGE_FILES;
+ else if (mime_type == "audio/*")
+ description_id = IDS_AUDIO_FILES;
+ else if (mime_type == "video/*")
+ description_id = IDS_VIDEO_FILES;
+
+ if (description_id) {
+ info->extension_description_overrides.push_back(
+ l10n_util::GetStringUTF16(description_id));
+ }
+}
+
+// FilePicker service allowing a native file picker to handle
+// pick + */* intents.
+class NativeFilePickerService
+ : public IntentServiceHost, public ui::SelectFileDialog::Listener {
+ public:
+ explicit NativeFilePickerService(content::WebContents* web_contents);
+ virtual ~NativeFilePickerService();
+ virtual void HandleIntent(content::WebIntentsDispatcher* dispatcher) OVERRIDE;
+
+ // SelectFileDialog::Listener
+ virtual void FileSelected(
+ const FilePath& path, int index, void* params) OVERRIDE;
+ virtual void FileSelectionCanceled(void* params) OVERRIDE;
+
+ private:
+ // Weak pointer to the web contents on which the selector will be displayed.
+ content::WebContents* web_contents_;
+
+ // Weak pointer to the dispatcher for the current intent. Only
+ // set at the time the intent request is delivered to HandleIntent.
+ content::WebIntentsDispatcher* dispatcher_;
+
+ scoped_refptr<ui::SelectFileDialog> dialog_;
+
+ DISALLOW_COPY_AND_ASSIGN(NativeFilePickerService);
+};
+
+} // namespace
+
+NativeFilePickerService::NativeFilePickerService(
+ content::WebContents* web_contents)
+ : web_contents_(web_contents), dispatcher_(NULL) {
+}
+
+NativeFilePickerService::~NativeFilePickerService() {}
+
+void NativeFilePickerService::HandleIntent(
+ content::WebIntentsDispatcher* dispatcher) {
+ DCHECK(dispatcher);
+ dispatcher_ = dispatcher;
+
+ const webkit_glue::WebIntentData& intent = dispatcher_->GetIntent();
+
+ std::string ascii_type = UTF16ToASCII(intent.type);
+ DCHECK(!(net::GetIANAMediaType(ascii_type).empty()));
+
+ dialog_ = ui::SelectFileDialog::Create(this, NULL);
+
+ ui::SelectFileDialog::FileTypeInfo type_info;
+ AddTypeInfo(ascii_type, &type_info);
+
+ const FilePath default_path(FILE_PATH_LITERAL("."));
+ const FilePath::StringType default_extension = FILE_PATH_LITERAL("");
+ const string16 title = FilePickerFactory::GetServiceTitle();
+
+ dialog_->SelectFile(
+ ui::SelectFileDialog::SELECT_OPEN_FILE,
+ title,
+ default_path,
+ &type_info,
+ 1, // index of which file description to show
+ default_extension,
+ platform_util::GetTopLevel(web_contents_->GetNativeView()),
+ NULL);
+}
+
+void NativeFilePickerService::FileSelected(
+ const FilePath& path, int index, void* params) {
+ DCHECK(dispatcher_);
+ // TODO(smckay): once we've worked out the RPCs, we'll return the FilePath.
+ string16 url = path.LossyDisplayName();
+ dispatcher_->SendReplyMessage(webkit_glue::WEB_INTENT_REPLY_SUCCESS, url);
+}
+
+void NativeFilePickerService::FileSelectionCanceled(void* params) {
+ DCHECK(dispatcher_);
+ dispatcher_->SendReplyMessage(
+ webkit_glue::WEB_INTENT_REPLY_FAILURE, string16());
+}
+
+// static
+IntentServiceHost* FilePickerFactory::CreateServiceInstance(
+ const webkit_glue::WebIntentData& intent,
+ content::WebContents* web_contents) {
+ return new NativeFilePickerService(web_contents);
+}
+
+// Returns the action-specific string for |action|.
+string16 FilePickerFactory::GetServiceTitle() {
+ return l10n_util::GetStringUTF16(IDS_WEB_INTENTS_FILE_PICKER_SERVICE_TITLE);
+}
+
+} // web_intents namespace
« no previous file with comments | « chrome/browser/intents/native_services_unittest.cc ('k') | chrome/browser/ui/intents/web_intent_picker_controller.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698