Move SelectFileDialog implementation to ui/base/dialogs/

ui/base/dialogs/select_file_dialog.h

 // 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 CHROME_BROWSER_UI_SELECT_FILE_DIALOG_H_
-#define CHROME_BROWSER_UI_SELECT_FILE_DIALOG_H_
+#ifndef UI_BASE_DIALOGS_SELECT_FILE_DIALOG_H_
+#define UI_BASE_DIALOGS_SELECT_FILE_DIALOG_H_
 
 #include <string>
 #include <vector>
 
 #include "base/basictypes.h"
 #include "base/callback_forward.h"
 #include "base/file_path.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/string16.h"
 #include "ui/base/dialogs/base_shell_dialog.h"
+#include "ui/base/ui_export.h"
 #include "ui/gfx/native_widget_types.h"
 
 namespace ui {
+class SelectFileDialogFactory;
 class SelectFilePolicy;
-}
-
-namespace content {
-class WebContents;
-}
-
-namespace ui {
 struct SelectedFileInfo;
-}
-
-// This function is declared extern such that it is accessible for unit tests
-// in /chrome/browser/ui/views/select_file_dialog_win_unittest.cc
-extern std::wstring AppendExtensionIfNeeded(const std::wstring& filename,
-                                            const std::wstring& filter_selected,
-                                            const std::wstring& suggested_ext);
 
 // Shows a dialog box for selecting a file or a folder.
-class SelectFileDialog
+class UI_EXPORT SelectFileDialog
     : public base::RefCountedThreadSafe<SelectFileDialog>,
       public ui::BaseShellDialog {
  public:
   enum Type {
     SELECT_NONE,
     SELECT_FOLDER,
     SELECT_SAVEAS_FILE,
     SELECT_OPEN_FILE,
     SELECT_OPEN_MULTI_FILE
   };
 
   // An interface implemented by a Listener object wishing to know about the
   // the result of the Select File/Folder action. These callbacks must be
   // re-entrant.
-  class Listener {
+  class UI_EXPORT Listener {

[James Cook, 2012/07/11 22:34:14]

Optional suggestion: Perhaps this should be in a separate file and at ui:: scope
so it can be forward declared and includes of this file can be avoided?  I saw
an include elsewhere that seemed to exist solely to declare a pointer of
SelectFileDialog::Listener*.

[Elliot Glaysher, 2012/07/11 22:40:16]

That's a good idea. I've filed
https://code.google.com/p/chromium/issues/detail?id=136909 because this
interface is used all over the place and this patch is already getting pretty
big.

    public:
     // Notifies the Listener that a file/folder selection has been made. The
     // file/folder path is in |path|. |params| is contextual passed to
     // SelectFile. |index| specifies the index of the filter passed to the
     // the initial call to SelectFile.
     virtual void FileSelected(const FilePath& path,
                               int index, void* params) = 0;
 
     // Similar to FileSelected() but takes SelectedFileInfo instead of
     // FilePath. Used for passing extra information (ex. display name).
@@ skipping 19 matching lines @@
 
     // Notifies the Listener that the file/folder selection was aborted (via
     // the  user canceling or closing the selection dialog box, for example).
     // |params| is contextual passed to SelectFile.
     virtual void FileSelectionCanceled(void* params) {}
 
    protected:
     virtual ~Listener() {}
   };
 
+  // Sets the factory that creates SelectFileDialog objects, overriding default
+  // behaviour.
+  //
+  // This is optional and should only be used by components that have to live
+  // elsewhere in the tree due to layering violations. (For example, because of
+  // a dependency on chrome's extension system.)
+  static void SetFactory(ui::SelectFileDialogFactory* factory);
+
   // Creates a dialog box helper. This object is ref-counted, but the returned
   // object will have no reference (refcount is 0). |policy| is an optional
   // class that can prevent showing a dialog.
   static SelectFileDialog* Create(Listener* listener,
                                   ui::SelectFilePolicy* policy);
 
   // Holds information about allowed extensions on a file save dialog.
   // |extensions| is a list of allowed extensions. For example, it might be
   //   { { "htm", "html" }, { "txt" } }. Only pass more than one extension
   //   in the inner vector if the extensions are equivalent. Do NOT include
   //   leading periods.
   // |extension_description_overrides| overrides the system descriptions of the
   //   specified extensions. Entries correspond to |extensions|; if left blank
   //   the system descriptions will be used.
   // |include_all_files| specifies whether there will be a filter added for all
   //   files (i.e. *.*).
-  struct FileTypeInfo {
+  struct UI_EXPORT FileTypeInfo {
     FileTypeInfo();
     ~FileTypeInfo();
 
     std::vector<std::vector<FilePath::StringType> > extensions;
     std::vector<string16> extension_description_overrides;
     bool include_all_files;
   };
 
   // Selects a File.
   // Before doing anything this function checks if FileBrowsing is forbidden
@@ skipping 65 matching lines @@
   void CancelFileSelection(void* params);
 
   // Returns true if the dialog has multiple file type choices.
   virtual bool HasMultipleFileTypeChoicesImpl() = 0;
 
   scoped_ptr<ui::SelectFilePolicy> select_file_policy_;
 
   DISALLOW_COPY_AND_ASSIGN(SelectFileDialog);
 };
 
-#endif  // CHROME_BROWSER_UI_SELECT_FILE_DIALOG_H_
+}  // namespace ui
+
+#endif  // UI_BASE_DIALOGS_SELECT_FILE_DIALOG_H_