Adding pref to store the user-selected proximity threshold.

components/proximity_auth/unlock_manager_impl.h

 // Copyright 2015 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 COMPONENTS_PROXIMITY_AUTH_UNLOCK_MANAGER_IMPL_H
 #define COMPONENTS_PROXIMITY_AUTH_UNLOCK_MANAGER_IMPL_H
 
 #include "base/macros.h"
 #include "base/memory/ref_counted.h"
 #include "base/memory/weak_ptr.h"
 #include "build/build_config.h"
 #include "components/proximity_auth/messenger_observer.h"
 #include "components/proximity_auth/proximity_auth_system.h"
 #include "components/proximity_auth/proximity_monitor_observer.h"
 #include "components/proximity_auth/remote_device_life_cycle.h"
 #include "components/proximity_auth/remote_status_update.h"
 #include "components/proximity_auth/screenlock_bridge.h"
 #include "components/proximity_auth/screenlock_state.h"
 #include "components/proximity_auth/unlock_manager.h"
 #include "device/bluetooth/bluetooth_adapter.h"
 
 #if defined(OS_CHROMEOS)
 #include "chromeos/dbus/power_manager_client.h"
 #endif
 
+class PrefService;
+
 namespace proximity_auth {
 
 class Messenger;
 class ProximityAuthClient;
 class ProximityMonitor;
 
 // The unlock manager is responsible for controlling the lock screen UI based on
 // the authentication status of the registered remote devices.
 class UnlockManagerImpl : public UnlockManager,
                           public MessengerObserver,
@@ skipping 13 matching lines @@
   // UnlockManager:
   bool IsUnlockAllowed() override;
   void SetRemoteDeviceLifeCycle(RemoteDeviceLifeCycle* life_cycle) override;
   void OnLifeCycleStateChanged() override;
   void OnAuthAttempted(mojom::AuthType auth_type) override;
 
  protected:
   // Creates a ProximityMonitor instance for the given |connection|.
   // Exposed for testing.
   virtual std::unique_ptr<ProximityMonitor> CreateProximityMonitor(
-      cryptauth::Connection* connection);
+      cryptauth::Connection* connection,
+      PrefService* pref_service);

[Tim Song, 2017/07/10 19:27:56]

You should take in the ProximityAuthPrefManager instead. We can pass in the
pref_manager_ in ProximityAuthSystem.

[sacomoto, 2017/07/11 09:56:06]

Done.

 
  private:
   // The possible lock screen states for the remote device.
   enum class RemoteScreenlockState {
     UNKNOWN,
     UNLOCKED,
     DISABLED,
     LOCKED,
   };
 
@@ skipping 117 matching lines @@
 
   // Used to vend all other weak pointers.
   base::WeakPtrFactory<UnlockManagerImpl> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(UnlockManagerImpl);
 };
 
 }  // namespace proximity_auth
 
 #endif  // COMPONENTS_PROXIMITY_AUTH_UNLOCK_MANAGER_IMPL_H