Fixed problems with initial locale change freezing Chrome OS on the device....

chrome/browser/chromeos/login/wizard_controller.cc

 // Copyright (c) 2010 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/chromeos/login/wizard_controller.h"
 
 #include <gdk/gdk.h>
 #include <signal.h>
 #include <sys/types.h>
 
 #include <string>
 #include <vector>
 
 #include "app/l10n_util.h"
 #include "base/command_line.h"
 #include "base/logging.h"
+#include "base/file_util.h"
 #include "chrome/browser/browser_process.h"
 #include "chrome/browser/chromeos/cros/cros_library.h"
 #include "chrome/browser/chromeos/cros/login_library.h"
 #include "chrome/browser/chromeos/cros/system_library.h"
 #include "chrome/browser/chromeos/customization_document.h"
 #include "chrome/browser/chromeos/input_method/input_method_util.h"
 #include "chrome/browser/chromeos/login/account_screen.h"
 #include "chrome/browser/chromeos/login/background_view.h"
 #include "chrome/browser/chromeos/login/eula_view.h"
 #include "chrome/browser/chromeos/login/existing_user_controller.h"
@@ skipping 17 matching lines @@
 #include "views/accelerator.h"
 #include "views/painter.h"
 #include "views/view.h"
 #include "views/widget/widget_gtk.h"
 
 namespace {
 
 // A boolean pref of the OOBE complete flag.
 const wchar_t kOobeComplete[] = L"OobeComplete";
 
+// Path to OEM partner startup customization manifest.
+const char kStartupCustomizationManifestPath[] =
+    "/mnt/partner_partition/etc/chromeos/startup_manifest.json";
+
+// URL where to fetch OEM services customization manifest from.
+// TODO(denisromanov): Change this to real URL when it becomes available.

[Nikita (slow), 2010/07/19 12:31:19]

Please file issue to track this and mention it here.

+const char kServicesCustomizationManifestUrl[] =
+    "file:///mnt/partner_partition/etc/chromeos/services_manifest.json";
+
+// Path to flag file indicating that OOBE was completed successfully.
+const char kOobeCompleteFlagFilePath[] =
+    "/home/chronos/.oobe_completed";
+
 // Default size of the OOBE screen.
 const int kWizardScreenWidth = 700;
 const int kWizardScreenHeight = 416;
 
 // RootView of the Widget WizardController creates. Contains the contents of the
 // WizardController.
 class ContentView : public views::View {
  public:
   ContentView(int window_x, int window_y, int screen_w, int screen_h)
       : window_x_(window_x),
@@ skipping 82 matching lines @@
   views::Accelerator accel_network_screen_;
   views::Accelerator accel_update_screen_;
   views::Accelerator accel_image_screen_;
   views::Accelerator accel_eula_screen_;
 
   DISALLOW_COPY_AND_ASSIGN(ContentView);
 };
 
 void DeleteWizardControllerAndLaunchBrowser(WizardController* controller) {
   delete controller;
+  // Apply OEM partner services customizations.
+  controller->ApplyPartnerServicesCustomizations();

[Nikita (slow), 2010/07/19 12:31:19]

It seems like a not the best place to apply these customizations. It will have
extensions and apps defined which take time to download so it should be done as
early as possible. Doing that right before the browser launch may slow down it.

At best it should be done in parallel.

   // Launch browser after controller is deleted and its windows are closed.
   chromeos::LoginUtils::Get()->EnableBrowserLaunch(true);
   ChromeThread::PostTask(
       ChromeThread::UI,
       FROM_HERE,
       NewRunnableFunction(&chromeos::LoginUtils::DoBrowserLaunch,
                           ProfileManager::GetDefaultProfile()));
 }
 
 }  // namespace
@@ skipping 170 matching lines @@
         new chromeos::ExistingUserController(users, screen_bounds);
     controller->OwnBackground(background_widget_, background_view_);
     controller->Init();
     background_widget_ = NULL;
     background_view_ = NULL;
     MessageLoop::current()->DeleteSoon(FROM_HERE, this);
     return;
   }
 
   SetCurrentScreen(GetLoginScreen());
+
+  LOG(INFO) << "WTF: " << (is_out_of_box_ ? "out" : "in");

[Nikita (slow), 2010/07/19 12:31:19]

Debug info - should be removed?
At least place a descriptive info message. :)

+
+  if (is_out_of_box_) {
+    // Initiate fetching partner services manifest here.
+    services_manifest_fetcher_.reset(new StringFetcher(

[Nikita (slow), 2010/07/19 12:31:19]

We're fetching manifest during OOBE
welcome-update-eula screens.

What if being at sign in screen user shutdowns device?
After restart it won't be OOBE mode part 1.
It will be OOBE part 2 (Register screen after sign in).
OOBE p2 is before .oobe_completed file is written.

We should make sure that services customization will be applied.

+        kServicesCustomizationManifestUrl));
+  }
 }
 
 void WizardController::ShowAccountScreen() {
   SetStatusAreaVisible(true);
   SetCurrentScreen(GetAccountScreen());
 }
 
 void WizardController::ShowUpdateScreen() {
   SetStatusAreaVisible(true);
   SetCurrentScreen(GetUpdateScreen());
@@ skipping 194 matching lines @@
       ShowLoginScreen();
     }
   }
 }
 
 void WizardController::MarkOobeCompleted() {
   PrefService* prefs = g_browser_process->local_state();
   prefs->SetBoolean(kOobeComplete, true);
   // Make sure that changes are reflected immediately.
   prefs->SavePersistentPrefs();
+  // Create flag file for boot-time init scripts.
+  FilePath oobe_complete_path(kOobeCompleteFlagFilePath);
+  FILE* oobe_flag_file = file_util::OpenFile(oobe_complete_path, "w+b");
+  DCHECK(oobe_flag_file != NULL) << kOobeCompleteFlagFilePath;
+  if (oobe_flag_file != NULL)
+    file_util::CloseFile(oobe_flag_file);
+}
+
+void WizardController::ApplyPartnerServicesCustomizations() {
+  if (services_manifest_fetcher_.get() == NULL ||
+      services_manifest_fetcher_->result().empty()) {
+    return;
+  }
+  scoped_ptr<chromeos::ServicesCustomizationDocument> customization;
+  bool manifest_loaded;
+  customization.reset(new chromeos::ServicesCustomizationDocument());
+  manifest_loaded = customization->LoadManifestFromString(
+      services_manifest_fetcher_->result());
+  DCHECK(manifest_loaded) << "Customization manifest fetch error: "
+                          << services_manifest_fetcher_->result();
+  if (!manifest_loaded)
+    return;
+  if (!customization->initial_start_page_url().empty()) {
+    // Append partner's start page url to command line so it gets opened
+    // on browser startup.
+    CommandLine::ForCurrentProcess()->AppendLooseValue(
+        UTF8ToWide(customization->initial_start_page_url()));
+  }

[Nikita (slow), 2010/07/19 12:31:19]

apps + exts + support page?

 }
 
 ///////////////////////////////////////////////////////////////////////////////
 // WizardController, chromeos::ScreenObserver overrides:
 void WizardController::OnExit(ExitCodes exit_code) {
   switch (exit_code) {
     case LOGIN_SIGN_IN_SELECTED:
       OnLoginSignInSelected();
       break;
     case LOGIN_GUEST_SELECTED:
@@ skipping 92 matching lines @@
           switches::kEnableLoginImages)) {
     std::vector<chromeos::UserManager::User> users =
         chromeos::UserManager::Get()->GetUsers();
     // ExistingUserController deletes itself.
     (new chromeos::ExistingUserController(users, screen_bounds))->Init();
     return;
   }
 
   scoped_ptr<chromeos::StartupCustomizationDocument> customization;
 
-  if (!oobe_complete) {
-    // Load partner customization startup manifest if needed.
-    if (CommandLine::ForCurrentProcess()->HasSwitch(
-        switches::kStartupManifest)) {
-      customization.reset(new chromeos::StartupCustomizationDocument());
-      FilePath manifest_path =
-          CommandLine::ForCurrentProcess()->GetSwitchValuePath(
-              switches::kStartupManifest);
-      bool manifest_loaded = customization->LoadManifestFromFile(manifest_path);
-      DCHECK(manifest_loaded) << manifest_path.value();
-    }
+  // Load partner customization startup manifest if it is available.
+  FilePath startup_manifest_path(kStartupCustomizationManifestPath);
+  if (file_util::PathExists(startup_manifest_path)) {

[Nikita (slow), 2010/07/19 12:31:19]

Please add check that startup manifest should exist before .oobe_completed file
is created (before both parts of OOBE are finished).

+    customization.reset(new chromeos::StartupCustomizationDocument());
+    bool manifest_loaded = customization->LoadManifestFromFile(
+        FilePath(kStartupCustomizationManifestPath));
+    DCHECK(manifest_loaded) << kStartupCustomizationManifestPath;
+    if (!manifest_loaded)
+      customization.reset(NULL);
+  }
 
-    // Do UX customizations if needed.
-    if (customization != NULL) {
-      // Switch to initial locale if specified by customization.
+  if (chromeos::CrosLibrary::Get()->EnsureLoaded())
+    chromeos::CrosLibrary::Get()->GetLoginLibrary()->EmitLoginPromptReady();

[Nikita (slow), 2010/07/19 12:31:19]

We should look deeper into it.
Emitting login_prompt_ready signal here might slowdown showing login window. See
discussion here
http://code.google.com/p/chromium-os/issues/detail?id=4562

[Nikita (slow), 2010/07/19 12:54:26]

Possible solution:
1. Change Chrome locale only, show welcome screen
2. Postpone switching system wide locale till underlying services are ready
(login_prompt_ready emitted).

With this solution it's fine to leave emitting login_prompt_ready after
controller->Show() as it was. 
In all cases for non-OOBE case it should be leaved unchanged.

+
+  if (customization.get() != NULL) {
+    // Switch to initial locale if specified by customization
+    // and has not been set yet.
+    const std::string current_locale =
+        g_browser_process->local_state()->GetString(prefs::kApplicationLocale);
+    if (current_locale.empty()) {
       const std::string locale = customization->initial_locale();
       if (!locale.empty()) {
         chromeos::LanguageSwitchMenu::SwitchLanguage(locale);
       }
+    }
 
-      // Set initial timezone if specified by customization.
-      const std::string timezone_name = customization->initial_timezone();
-      if (!timezone_name.empty()) {
-        icu::TimeZone* timezone = icu::TimeZone::createTimeZone(
-            icu::UnicodeString::fromUTF8(timezone_name));
-        chromeos::CrosLibrary::Get()->GetSystemLibrary()->SetTimezone(timezone);
-      }
+    // Set initial timezone if specified by customization.
+    const std::string timezone_name = customization->initial_timezone();
+    if (!timezone_name.empty()) {
+      icu::TimeZone* timezone = icu::TimeZone::createTimeZone(
+          icu::UnicodeString::fromUTF8(timezone_name));
+      chromeos::CrosLibrary::Get()->GetSystemLibrary()->SetTimezone(timezone);
     }
   }
 
   // Create and show the wizard.
   WizardController* controller = new WizardController();
-  if (!oobe_complete)
+  if (customization.get() != NULL)
     controller->SetCustomization(customization.release());
   controller->ShowBackground(screen_bounds);
   controller->Init(first_screen_name, screen_bounds);
   controller->Show();
-  if (chromeos::CrosLibrary::Get()->EnsureLoaded())
-    chromeos::CrosLibrary::Get()->GetLoginLibrary()->EmitLoginPromptReady();
 }
 
 }  // namespace browser