Read CrOS install attributes cache on startup.

chrome/browser/policy/enterprise_install_attributes.h

 // Copyright (c) 2012 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.
 
 #ifndef CHROME_BROWSER_POLICY_ENTERPRISE_INSTALL_ATTRIBUTES_H_
 #define CHROME_BROWSER_POLICY_ENTERPRISE_INSTALL_ATTRIBUTES_H_
 
+#include <map>
 #include <string>
 
 #include "base/basictypes.h"
 #include "base/compiler_specific.h"
+#include "base/file_path.h"
 #include "chrome/browser/policy/cloud_policy_constants.h"
 
+class FilePath;

[Joao da Silva, 2013/01/17 19:53:25]

file_path.h already included

[Mattias Nissler (ping if slow), 2013/01/17 20:34:43]

Done.

+
 namespace chromeos {
 class CryptohomeLibrary;
 }
 
 namespace policy {
 
 // Brokers access to the enterprise-related installation-time attributes on
 // ChromeOS.
 class EnterpriseInstallAttributes {
  public:
   // Return codes for LockDevice().
   enum LockResult {
     LOCK_SUCCESS,
     LOCK_NOT_READY,
     LOCK_BACKEND_ERROR,
     LOCK_WRONG_USER,
   };
 
+  // Standard cache file name.
+  static const FilePath::CharType kCacheFilePath[];
+
   explicit EnterpriseInstallAttributes(chromeos::CryptohomeLibrary* cryptohome);
 
+  // Reads data from the cache file. The cache file is used to work around slow
+  // cryptohome startup, which takes a while to register its DBus interface.
+  // See http://crosbug.com/37367 for background on this.
+  void ReadCacheFile(const FilePath& cache_file);

[Joao da Silva, 2013/01/17 19:53:25]

Why the argument and the static const? Just ReadCacheFile()? Or did you mean to
write tests for this? :-)

[Mattias Nissler (ping if slow), 2013/01/17 20:34:43]

Wrote the test :)

+
   // Locks the device to be an enterprise device registered by the given user.
   // This can also be called after the lock has already been taken, in which
   // case it checks that the passed user agrees with the locked attribute.
   LockResult LockDevice(const std::string& user,
                         DeviceMode device_mode,
                         const std::string& device_id) WARN_UNUSED_RESULT;
 
   // Checks whether this is an enterprise device.
   bool IsEnterpriseDevice();
 
@@ skipping 12 matching lines @@
 
   // Gets the mode the device was enrolled to. The return value for devices that
   // are not locked yet will be DEVICE_MODE_UNKNOWN.
   DeviceMode GetMode();
 
  private:
   // Makes sure the local caches for enterprise-related install attributes are
   // up-to-date with what cryptohome has.
   void ReadImmutableAttributes();
 
+  // Decodes the install attributes provided in |attr_map|.
+  void DecodeInstallAttributes(
+      const std::map<std::string, std::string>& attr_map);
+
   chromeos::CryptohomeLibrary* cryptohome_;
 
+  bool attrs_read_;
+
   bool device_locked_;
   std::string registration_user_;
   std::string registration_domain_;
   std::string registration_device_id_;
   DeviceMode registration_mode_;
 
   DISALLOW_COPY_AND_ASSIGN(EnterpriseInstallAttributes);
 };
 
 }  // namespace policy
 
 #endif  // CHROME_BROWSER_POLICY_ENTERPRISE_INSTALL_ATTRIBUTES_H_