Get rid of FromWStringHack in select_file_dialog.cc

chrome/browser/ui/views/select_file_dialog.cc

 // Copyright (c) 2011 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/shell_dialogs.h"
 
 #include "app/l10n_util.h"
 #include "base/callback.h"
 #include "base/file_path.h"
 #include "base/json/json_reader.h"
@@ skipping 242 matching lines @@
   if (!json.empty()) {
     scoped_ptr<Value> value(base::JSONReader::Read(json, false));
     if (!value.get() || !value->IsType(Value::TYPE_DICTIONARY)) {
       // Bad json value returned.
       NOTREACHED();
     } else {
       const DictionaryValue* dict = static_cast<DictionaryValue*>(value.get());
       if (delegate->type_ == SELECT_OPEN_FILE ||
           delegate->type_ == SELECT_SAVEAS_FILE ||
           delegate->type_ == SELECT_FOLDER) {
-        std::string path_string;
+        FilePath path;
         if (dict->HasKey(kKeyNamePath) &&
-            dict->GetString(kKeyNamePath, &path_string)) {
-          FilePath path = FilePath::FromWStringHack(UTF8ToWide(path_string));

[tfarina, 2011/01/12 01:30:21]

Evan, thinking on it again, can't I just do:
FilePath path(UTF8ToWide(path_string));

Since this is Windows only?

-
+            dict->GetFilePath(kKeyNamePath, &path)) {
           listener_->FileSelected(path, kSaveCompletePageIndex,
                                   delegate->params_);
           notification_fired = true;
         }
       } else if (delegate->type_ == SELECT_OPEN_MULTI_FILE) {
         ListValue* paths_value = NULL;
         if (dict->HasKey(kKeyNamePath) &&
             dict->GetList(kKeyNamePath, &paths_value) &&
             paths_value) {
           std::vector<FilePath> paths;
           paths.reserve(paths_value->GetSize());
           for (size_t i = 0; i < paths_value->GetSize(); ++i) {
-            std::string path_string;
-            if (paths_value->GetString(i, &path_string) &&
-                !path_string.empty()) {
-              paths.push_back(FilePath::FromWStringHack(
-                  UTF8ToWide(path_string)));
+            FilePath path;
+            if (paths_value->GetFilePath(i, &path) && !path.empty()) {
+              paths.push_back(path);
             }
           }
 
           listener_->MultiFilesSelected(paths, delegate->params_);
           notification_fired = true;
         }
       } else {
         NOTREACHED();
       }
     }
@@ skipping 183 matching lines @@
     // TODO(xiyuan): Change this when the infrastructure is improved.
     HtmlDialogUIDelegate** delegate = HtmlDialogUI::GetPropertyAccessor().
         GetProperty(dom_ui_->tab_contents()->property_bag());
     HtmlDialogView* containing_view = static_cast<HtmlDialogView*>(*delegate);
     DCHECK(containing_view);
 
     containing_view->GetWindow()->UpdateWindowTitle();
     containing_view->GetWindow()->GetNonClientView()->SchedulePaint();
   }
 }