Remove outdated devices from Android device chooser

device/bluetooth/bluetooth_adapter.cc

 // 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.
 
 #include "device/bluetooth/bluetooth_adapter.h"
 
 #include <memory>
 #include <utility>
 
 #include "base/bind.h"
@@ skipping 347 matching lines @@
       }
       continue;
     }
 
     result = BluetoothDiscoveryFilter::Merge(result.get(), curr_filter);
   }
 
   return result;
 }
 
+void BluetoothAdapter::RemoveTimedOutDevices() {
+  std::set<std::string> removed_devices;
+  for (DevicesMap::const_iterator it = devices_.begin(); it != devices_.end();
+       ++it) {
+    BluetoothDevice* device = it->second;
+
+    if (device->IsPaired() || device->IsConnected() ||
+        device->IsGattConnected())
+      continue;
+
+    base::Time last_update_time = device->GetLastUpdateTime();
+
+    bool device_expired =
+        (base::Time::NowFromSystemTime() - last_update_time) > timeoutSec;
+    VLOG(1) << "device: " << device->GetAddress()
+            << ", last_update: " << last_update_time
+            << ", exp: " << device_expired;
+
+    if (!device_expired)
+      continue;
+    removed_devices.insert(it->first);

[ortuno, 2016/06/22 17:23:11]

nit: I would change removed_devices to be a std::vector<std::unique_ptr> and
call 

removed_devices.push_back(devices_.take_and_erase(it));

This way if someone calls GetDevices inside DeviceRemoved the removed device is
not included.

[perja, 2016/06/23 14:31:25]

I used a slightly different approach. Please take a look.

+
+    FOR_EACH_OBSERVER(BluetoothAdapter::Observer, observers_,
+                      DeviceRemoved(this, device));
+  }
+
+  for (const std::string& device_address : removed_devices) {
+    size_t num_removed = devices_.erase(device_address);
+    DCHECK_EQ(num_removed, 1U);
+  }
+}
+
 // static
 void BluetoothAdapter::RecordBluetoothDiscoverySessionStartOutcome(
     UMABluetoothDiscoverySessionOutcome outcome) {
   UMA_HISTOGRAM_ENUMERATION(
       "Bluetooth.DiscoverySession.Start.Outcome", static_cast<int>(outcome),
       static_cast<int>(UMABluetoothDiscoverySessionOutcome::COUNT));
 }
 
 // static
 void BluetoothAdapter::RecordBluetoothDiscoverySessionStopOutcome(
     UMABluetoothDiscoverySessionOutcome outcome) {
   UMA_HISTOGRAM_ENUMERATION(
       "Bluetooth.DiscoverySession.Stop.Outcome", static_cast<int>(outcome),
       static_cast<int>(UMABluetoothDiscoverySessionOutcome::COUNT));
 }
 
+// static
+const base::TimeDelta BluetoothAdapter::timeoutSec =
+    base::TimeDelta::FromSeconds(180);
+
 }  // namespace device