Fix a signed settings serialization issue.

chrome/browser/chromeos/login/session_manager_observer.cc

Index: chrome/browser/chromeos/login/session_manager_observer.cc
diff --git a/chrome/browser/chromeos/login/session_manager_observer.cc b/chrome/browser/chromeos/login/session_manager_observer.cc
index bd778ee85642d6437d9cc7221089c4991c827b96..baca35bfcd5728d77d62784bb011342ee558574e 100644
--- a/chrome/browser/chromeos/login/session_manager_observer.cc
+++ b/chrome/browser/chromeos/login/session_manager_observer.cc
@@ -53,19 +53,19 @@ void SessionManagerObserver::OwnerKeySet(bool success) {
   if (!success)
     result = chrome::NOTIFICATION_OWNER_KEY_FETCH_ATTEMPT_FAILED;
 
+  // We stored some settings in transient storage before owner was assigned.
+  // Now owner is assigned and key is generated and we should persist
+  // those settings into signed storage.
+  if (success && g_browser_process && g_browser_process->local_state()) {

[Ivan Korotkov, 2011/12/06 15:30:27]

One-line branches don't need braces.

+    signed_settings_cache::Finalize(g_browser_process->local_state());
+  }
+
   // Whether we exported the public key or not, send a notification
   // indicating that we're done with this attempt.
   content::NotificationService::current()->Notify(
       result,
       content::NotificationService::AllSources(),
       content::NotificationService::NoDetails());
-
-  // We stored some settings in transient storage before owner was assigned.
-  // Now owner is assigned and key is generated and we should persist
-  // those settings into signed storage.
-  if (g_browser_process && g_browser_process->local_state()) {
-    signed_settings_cache::Finalize(g_browser_process->local_state());
-  }
 }
 
 void SessionManagerObserver::PropertyChangeComplete(bool success) {