Upstreaming SelectFileDialog for Android

ui/base/dialogs/select_file_dialog_android.h

Index: ui/base/dialogs/select_file_dialog_android.h
diff --git a/ui/base/dialogs/select_file_dialog_android.h b/ui/base/dialogs/select_file_dialog_android.h
new file mode 100644
index 0000000000000000000000000000000000000000..bbc9b1292f3a463a39f2704ba5e7fb9f7bf1d888
--- /dev/null
+++ b/ui/base/dialogs/select_file_dialog_android.h
@@ -0,0 +1,64 @@
+// 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.
+
+#ifndef UI_BASE_DIALOGS_ANDROID_SELECT_FILE_DIALOG_ANDROID_H_
+#define UI_BASE_DIALOGS_ANDROID_SELECT_FILE_DIALOG_ANDROID_H_
+
+#include <jni.h>
+
+#include "base/android/scoped_java_ref.h"
+#include "base/file_path.h"
+#include "ui/base/dialogs/select_file_dialog.h"
+
+namespace ui {
+
+class SelectFileDialogImpl : public ui::SelectFileDialog {

[Ted C, 2012/09/17 18:40:16]

Hmm...the other platforms have the Impl definition in the .cc.  Do we do it
differently for JNI needs or something?

+ public:
+  static SelectFileDialogImpl* Create(Listener* listener,

[sky, 2012/09/17 20:53:44]

wrap listener to next line so that it aligns with indentation of policy (you
have it write on line 32).

[aurimas (slooooooooow), 2012/09/17 22:20:48]

Done.

+      ui::SelectFilePolicy* policy);
+
+  void OnFileSelected(JNIEnv* env, jobject javaObject, jstring filepath);

[sky, 2012/09/17 20:53:44]

java_object

[aurimas (slooooooooow), 2012/09/17 22:20:48]

Done.

+  void OnFileNotSelected(JNIEnv* env, jobject javaObject);

[sky, 2012/09/17 20:53:44]

java_object

[aurimas (slooooooooow), 2012/09/17 22:20:48]

Done.

+
+  // From SelectFileDialog
+  virtual bool IsRunning(gfx::NativeWindow) const OVERRIDE;
+  virtual void ListenerDestroyed() OVERRIDE;
+
+  // Called when it is time to display the file picker.
+  // params is expected to be a Vector<string16> with accept_types first and
+  // the capture value as the last element of the vector.
+  virtual void SelectFileImpl(
+      ui::SelectFileDialog::Type,

[sky, 2012/09/17 20:53:44]

remember, all parameters need names.

[aurimas (slooooooooow), 2012/09/17 22:20:48]

Done.

+      const string16& title,
+      const FilePath& default_path,
+      const ui::SelectFileDialog::FileTypeInfo* file_types,
+      int file_type_index,
+      const std::string& default_extension,
+      gfx::NativeWindow owning_window,
+      void* params) OVERRIDE;
+
+  static bool RegisterSelectFileDialog(JNIEnv*);

[sky, 2012/09/17 20:53:44]

parameter needs a name.

[aurimas (slooooooooow), 2012/09/17 22:20:48]

Done.

+
+ protected:
+  virtual ~SelectFileDialogImpl();
+
+ private:
+  SelectFileDialogImpl(Listener* listener,  ui::SelectFilePolicy* policy);
+
+  virtual bool HasMultipleFileTypeChoicesImpl() OVERRIDE;
+
+  base::android::ScopedJavaGlobalRef<jobject> java_object_;
+
+  bool is_running_;

[sky, 2012/09/17 20:53:44]

Add description.

[aurimas (slooooooooow), 2012/09/17 22:20:48]

Done.

+
+  DISALLOW_COPY_AND_ASSIGN(SelectFileDialogImpl);
+};
+
+SelectFileDialog* CreateAndroidSelectFileDialog(
+    SelectFileDialog::Listener* listener,
+    SelectFilePolicy* policy);
+
+}  // namespace ui
+
+#endif  // UI_BASE_DIALOGS_ANDROID_SELECT_FILE_DIALOG_ANDROID_H_