Comb up ownership things.

chrome/browser/chromeos/locale_change_guard.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/chromeos/locale_change_guard.h"
 
 #include "base/utf_string_conversions.h"
 #include "chrome/app/chrome_command_ids.h"
 #include "chrome/browser/browser_process.h"
+#include "chrome/browser/chromeos/login/user_manager.h"
 #include "chrome/browser/metrics/user_metrics.h"
 #include "chrome/browser/notifications/notification_delegate.h"
 #include "chrome/browser/prefs/pref_service.h"
 #include "chrome/browser/profiles/profile.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/common/pref_names.h"
 #include "content/browser/tab_contents/tab_contents.h"
 #include "content/common/notification_service.h"
 #include "content/common/notification_source.h"
 #include "grit/app_resources.h"
@@ skipping 17 matching lines @@
   DISALLOW_COPY_AND_ASSIGN(Delegate);
 };
 
 LocaleChangeGuard::LocaleChangeGuard(Profile* profile)
     : profile_(profile),
       note_(NULL),
       reverted_(false) {
   DCHECK(profile_);
   registrar_.Add(this, NotificationType::LOAD_COMPLETED_MAIN_FRAME,
                  NotificationService::AllSources());
+  registrar_.Add(this, NotificationType::OWNERSHIP_CHECKED,
+                 NotificationService::AllSources());
 }
 
 void LocaleChangeGuard::RevertLocaleChange(const ListValue* list) {
   if (note_ == NULL ||
       profile_ == NULL ||
       from_locale_.empty() ||
       to_locale_.empty()) {
     NOTREACHED();
     return;
   }
   if (reverted_)
     return;
-
-  PrefService* prefs = profile_->GetPrefs();
-  if (prefs == NULL)
-    return;
-
   reverted_ = true;
   UserMetrics::RecordAction(UserMetricsAction("LanguageChange_Revert"));
   profile_->ChangeAppLocale(
       from_locale_, Profile::APP_LOCALE_CHANGED_VIA_REVERT);
 
   Browser* browser = Browser::GetTabbedBrowser(profile_, false);
   if (browser)
     browser->ExecuteCommand(IDC_EXIT);
 }
 
 void LocaleChangeGuard::Observe(NotificationType type,
                                 const NotificationSource& source,
                                 const NotificationDetails& details) {
-  if (type != NotificationType::LOAD_COMPLETED_MAIN_FRAME) {
-    NOTREACHED();
-    return;
-  }
   if (profile_ == NULL) {
     NOTREACHED();
     return;
   }
+  switch (type.value) {
+    case NotificationType::LOAD_COMPLETED_MAIN_FRAME:
+      // We need to perform locale change check only once, so unsubscribe.
+      registrar_.Remove(this, NotificationType::LOAD_COMPLETED_MAIN_FRAME,
+                        NotificationService::AllSources());
+      Check();
+      break;
+    case NotificationType::OWNERSHIP_CHECKED:
+      if (UserManager::Get()->current_user_is_owner()) {
+        PrefService* local_state = g_browser_process->local_state();
+        if (local_state) {
+          PrefService* prefs = profile_->GetPrefs();
+          if (prefs == NULL)

[altimofeev, 2011/03/16 11:35:54]

You can add NOTREACHED() here, since GetPrefs() should init them for you.

[Denis Lagno, 2011/03/16 11:55:14]

Done.

+            return;
+          std::string owner_locale =
+              prefs->GetString(prefs::kApplicationLocale);
+          if (!owner_locale.empty()) {
+            local_state->SetString(prefs::kOwnerLocale, owner_locale);
+            local_state->ScheduleSavePersistentPrefs();
+          }
+        }
+      }
+      break;
+    default:
+      NOTREACHED();
+      break;
+  }
+}
 
-  // We need to perform locale change check only once: so we want to
-  // unsubscribe from notifications in any case.
-  registrar_.RemoveAll();
-
+void LocaleChangeGuard::Check() {
   if (note_ != NULL || !from_locale_.empty() || !to_locale_.empty()) {
-    // Somehow we are notified more than once. Once is enough.
+    // Somehow we are invoked more than once. Once is enough.
     return;
   }
 
   std::string cur_locale = g_browser_process->GetApplicationLocale();
   if (cur_locale.empty()) {
     NOTREACHED();
     return;
   }
 
   PrefService* prefs = profile_->GetPrefs();
@@ skipping 63 matching lines @@
   if (by_user)
     master_->AcceptLocaleChange();
 }
 
 std::string LocaleChangeGuard::Delegate::id() const {
   // Arbitrary unique Id.
   return "8c386938-1e3f-11e0-ac7b-18a90520e2e5";
 }
 
 }  // namespace chromeos