Fix uninintialized ResourceBundle in ShowUserDataDirDialog.

chrome/browser/user_data_dir_extractor_win.cc

Index: chrome/browser/user_data_dir_extractor_win.cc
diff --git a/chrome/browser/user_data_dir_extractor_win.cc b/chrome/browser/user_data_dir_extractor_win.cc
index 5c4b94147f6e7f861874da86b248d27a0a7aa433..fbf94404feb6b8a83c28dfd15a892a972d1d72c9 100644
--- a/chrome/browser/user_data_dir_extractor_win.cc
+++ b/chrome/browser/user_data_dir_extractor_win.cc
@@ -14,6 +14,7 @@
 #include "chrome/common/chrome_paths.h"
 #include "chrome/common/chrome_switches.h"
 #include "content/public/common/main_function_params.h"
+#include "ui/base/resource/resource_bundle.h"
 
 namespace chrome {
 
@@ -46,6 +47,11 @@ base::FilePath GetUserDataDir(const content::MainFunctionParams& parameters) {
     // TODO(beng):
     NOTIMPLEMENTED();
 #else
+    // Make sure ResourceBundle is initialized. The user data dialog needs to
+    // access string resources. See http://crbug.com/230432
+    if (!ResourceBundle::HasSharedInstance())
+      ResourceBundle::InitSharedInstanceWithLocale("en-US", NULL);

[Nico, 2013/04/11 18:29:29]

Is "en-US" correct? I suppose prefs::kApplicationLocale isn't ready yet, but do
you want to use command_line.GetSwitchValueASCII(switches::kLang)?

[hshi1, 2013/04/11 20:23:19]

I've added a check for command line locale switch. Please review the Patch Set
#2, thanks.

On 2013/04/11 18:29:29, Nico wrote:

+
     base::FilePath new_user_data_dir =
         chrome::ShowUserDataDirDialog(user_data_dir);