Refine VRDevice Class

device/vr/vr_device_manager.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 DEVICE_VR_VR_DEVICE_MANAGER_H
 #define DEVICE_VR_VR_DEVICE_MANAGER_H
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 54 matching lines @@
 
   using ProviderList = std::vector<std::unique_ptr<VRDeviceProvider>>;
   ProviderList providers_;
 
   // Devices are owned by their providers.
   using DeviceMap = std::map<unsigned int, VRDevice*>;
   DeviceMap devices_;
 
   bool vr_initialized_;
 
-  using ServiceList = std::vector<VRServiceImpl*>;
-  ServiceList services_;
+  using ServiceSet = std::set<VRServiceImpl*>;

[mthiesse, 2016/11/30 13:59:04]

nit: type alias is also pretty useless here, might be better to just remove it.

+  ServiceSet services_;
 
   // For testing. If true will not delete self when consumer count reaches 0.
   bool keep_alive_;
 
   bool has_scheduled_poll_;
 
   bool has_activate_listeners_;
 
   base::ThreadChecker thread_checker_;
 
   base::RepeatingTimer timer_;
 
   DISALLOW_COPY_AND_ASSIGN(VRDeviceManager);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_BROWSER_VR_VR_DEVICE_MANAGER_H