Fix open dialog not remembering last opened folder on drive

chrome/browser/ui/browser.cc

 // 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 "chrome/browser/ui/browser.h"
 
 #if defined(OS_WIN)
 #include <windows.h>
 #include <shellapi.h>
 #endif  // OS_WIN
@@ skipping 167 matching lines @@
 #include "content/public/common/renderer_preferences.h"
 #include "grit/chromium_strings.h"
 #include "grit/generated_resources.h"
 #include "grit/locale_settings.h"
 #include "grit/theme_resources.h"
 #include "net/base/net_util.h"
 #include "net/base/registry_controlled_domains/registry_controlled_domain.h"
 #include "net/cookies/cookie_monster.h"
 #include "net/url_request/url_request_context.h"
 #include "ui/base/animation/animation.h"
+#include "ui/base/dialogs/selected_file_info.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/gfx/point.h"
 #include "webkit/glue/web_intent_data.h"
 #include "webkit/glue/web_intent_service_data.h"
 #include "webkit/glue/webkit_glue.h"
 #include "webkit/glue/window_open_disposition.h"
 #include "webkit/plugins/webplugininfo.h"
 
 #if defined(OS_WIN)
 #include "base/win/metro.h"
@@ skipping 1549 matching lines @@
     // Only show the zoom bubble for zoom changes in the active window.
     if (can_show_bubble && window_->IsActive())
       window_->ShowZoomBubble(zoom_percent);
   }
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 // Browser, SelectFileDialog::Listener implementation:
 
 void Browser::FileSelected(const FilePath& path, int index, void* params) {
-  profile_->set_last_selected_directory(path.DirName());
+  FileSelectedWithExtraInfo(ui::SelectedFileInfo(path, path), index, params);
+}
+
+void Browser::FileSelectedWithExtraInfo(
+    const ui::SelectedFileInfo& file_info,
+    int index,
+    void* params) {
+  profile_->set_last_selected_directory(file_info.file_path.DirName());
+
+  FilePath path = file_info.real_path;

[achuithb, 2012/07/24 20:13:05]

nit: const FilePath& would save a copy.

[tbarzic, 2012/07/25 02:02:54]

Done.

   GURL file_url = net::FilePathToFileURL(path);
 
 #if defined(OS_CHROMEOS)
   gdata::util::ModifyGDataFileResourceUrl(profile_, path, &file_url);
 #endif
 
   if (file_url.is_empty())
     return;
 
   OpenURL(OpenURLParams(
@@ skipping 508 matching lines @@
   if (contents && !allow_js_access) {
     contents->web_contents()->GetController().LoadURL(
         target_url,
         content::Referrer(),
         content::PAGE_TRANSITION_LINK,
         std::string());  // No extra headers.
   }
 
   return contents != NULL;
 }