Update CryptAuthDeviceManager to store all synced devices instead of only unlock keys.

components/cryptauth/cryptauth_device_manager.h

Index: components/cryptauth/cryptauth_device_manager.h
diff --git a/components/cryptauth/cryptauth_device_manager.h b/components/cryptauth/cryptauth_device_manager.h
index 9f3602fd949ade0ee6064b111cfcb8ce6adc9486..075779dfd602c0019877a55adc1cfc676e0f1bf7 100644
--- a/components/cryptauth/cryptauth_device_manager.h
+++ b/components/cryptauth/cryptauth_device_manager.h
@@ -105,10 +105,14 @@ class CryptAuthDeviceManager : public SyncScheduler::Delegate,
   // has ever been recorded, then this function will also return true.
   bool IsRecoveringFromFailure() const;
 
+  // Returns a list of all remote devices that have been synced.
+  const std::vector<cryptauth::ExternalDeviceInfo> synced_devices() const;

[Tim Song, 2016/12/08 00:26:39]

We can remove the const in the return type. It doesn't add any protection.

[Kyle Horimoto, 2016/12/08 00:35:03]

Done.

+
   // Returns a list of remote devices that can unlock the user's other devices.
-  const std::vector<cryptauth::ExternalDeviceInfo>& unlock_keys() const {
-    return unlock_keys_;
-  }
+  const std::vector<cryptauth::ExternalDeviceInfo> unlock_keys() const;
+
+  // Returns a list of remote devices that can host tether hotspots.
+  const std::vector<cryptauth::ExternalDeviceInfo> tether_hosts() const;
 
  protected:
   // Creates a new SyncScheduler instance. Exposed for testing.
@@ -144,8 +148,8 @@ class CryptAuthDeviceManager : public SyncScheduler::Delegate,
   // instance.
   PrefService* const pref_service_;
 
-  // The unlock keys currently synced from CryptAuth.
-  std::vector<cryptauth::ExternalDeviceInfo> unlock_keys_;
+  // All devices currently synced from CryptAuth.
+  std::vector<cryptauth::ExternalDeviceInfo> synced_devices_;
 
   // Schedules the time between device sync attempts.
   std::unique_ptr<SyncScheduler> scheduler_;