Translate device-local account IDs to user IDs

chrome/browser/chromeos/app_mode/kiosk_app_launcher.cc

 // Copyright 2013 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/app_mode/kiosk_app_launcher.h"
 
+#include <vector>
+
 #include "base/chromeos/chromeos_version.h"
 #include "base/logging.h"
 #include "base/memory/weak_ptr.h"
 #include "base/message_loop.h"
 #include "chrome/browser/chromeos/app_mode/startup_app_launcher.h"
 #include "chrome/browser/chromeos/login/login_display_host_impl.h"
 #include "chrome/browser/chromeos/login/login_utils.h"
 #include "chrome/browser/chromeos/login/user_manager.h"
+#include "chrome/browser/chromeos/policy/device_local_account.h"
+#include "chrome/browser/chromeos/settings/cros_settings.h"
 #include "chrome/browser/chromeos/ui/app_launch_view.h"
 #include "chrome/browser/lifetime/application_lifetime.h"
 #include "chromeos/cryptohome/async_method_caller.h"
 #include "chromeos/cryptohome/cryptohome_library.h"
 #include "chromeos/dbus/cryptohome_client.h"
 #include "chromeos/dbus/dbus_thread_manager.h"
 #include "content/public/browser/browser_thread.h"
 
 using content::BrowserThread;
 
 namespace chromeos {
 
 namespace {
 
-std::string GetAppUserNameFromAppId(const std::string& app_id) {
-  return app_id + "@" + UserManager::kKioskAppUserDomain;
-}
-
 }  // namespace
 
 // static
 KioskAppLauncher* KioskAppLauncher::running_instance_ = NULL;
 
 ////////////////////////////////////////////////////////////////////////////////
 // KioskAppLauncher::CryptohomedChecker ensures cryptohome daemon is up
 // and running by issuing an IsMounted call. If the call does not go through
 // and chromeos::DBUS_METHOD_CALL_SUCCESS is not returned, it will retry after
 // some time out and at the maximum five times before it gives up. Upon
@@ skipping 64 matching lines @@
  public:
   explicit ProfileLoader(KioskAppLauncher* launcher)
       : launcher_(launcher) {
   }
 
   virtual ~ProfileLoader() {
     LoginUtils::Get()->DelegateDeleted(this);
   }
 
   void Start() {
+    const std::vector<policy::DeviceLocalAccount> device_local_accounts =

[Mattias Nissler (ping if slow), 2013/05/17 14:29:44]

We might better go through KioskAppManager to find the username here?

[xiyuan, 2013/05/17 15:46:52]

Let's add a new method to KioskAppManager and move the logic there.

[bartfab (slow), 2013/05/17 16:08:47]

Done.

I reused the existing KioskAppManager::GetApp(). I think this is even neater
than adding a new special-purpose method.

While here, I also made some of the dependencies more explicit - part of the
neverending crusade against global singletons :).

+        policy::GetDeviceLocalAccounts(CrosSettings::Get());
+    for (std::vector<policy::DeviceLocalAccount>::const_iterator
+             it = device_local_accounts.begin();
+         it != device_local_accounts.end(); ++it) {
+      if (it->type == policy::DeviceLocalAccount::TYPE_KIOSK_APP &&
+          it->kiosk_app_id == launcher_->app_id_) {
+        user_id_ = it->user_id;
+        break;
+      }
+    }
+
     cryptohome::AsyncMethodCaller::GetInstance()->AsyncGetSanitizedUsername(
-        GetAppUserNameFromAppId(launcher_->app_id_),
+        user_id_,
         base::Bind(&ProfileLoader::OnUsernameHashRetrieved,
                    base::Unretained(this)));
   }
 
  private:
   void OnUsernameHashRetrieved(bool success,
                                const std::string& username_hash) {
     if (!success) {
-      LOG(ERROR) << "Unable to retrieve username hash for user '" <<
-          GetAppUserNameFromAppId(launcher_->app_id_) << "'";
+      LOG(ERROR) << "Unable to retrieve username hash for user '" << user_id_
+                 << "'.";

[xiyuan, 2013/05/17 15:46:52]

nit: think "'" is more correct since we have an opening "'" before |user_id_|.

[bartfab (slow), 2013/05/17 16:08:47]

It is a single quote already - this is probably hard to see in an editor as
there are so many quotes in a row :).

       launcher_->ReportLaunchResult(
           KioskAppLaunchError::UNABLE_TO_RETRIEVE_HASH);
       return;
     }
     LoginUtils::Get()->PrepareProfile(
-        UserContext(GetAppUserNameFromAppId(launcher_->app_id_),
+        UserContext(user_id_,
                     std::string(),   // password
                     std::string(),   // auth_code
                     username_hash),
         std::string(),  // display email
         false,  // using_oauth
         false,  // has_cookies
         this);
   }
 
   // LoginUtils::Delegate overrides:
   virtual void OnProfilePrepared(Profile* profile) OVERRIDE {
     launcher_->OnProfilePrepared(profile);
   }
 
   KioskAppLauncher* launcher_;
+  std::string user_id_;
+
   DISALLOW_COPY_AND_ASSIGN(ProfileLoader);
 };
 
 ////////////////////////////////////////////////////////////////////////////////
 // KioskAppLauncher
 
 KioskAppLauncher::KioskAppLauncher(const std::string& app_id)
     : app_id_(app_id),
       remove_attempted_(false) {
 }
@@ skipping 91 matching lines @@
   (new chromeos::StartupAppLauncher(profile, app_id_))->Start();
 
   if (LoginDisplayHostImpl::default_host())
     LoginDisplayHostImpl::default_host()->OnSessionStart();
   UserManager::Get()->SessionStarted();
 
   ReportLaunchResult(KioskAppLaunchError::NONE);
 }
 
 }  // namespace chromeos