Add Device Policy Handler for Bluetooth, and allow disabling the Bluetooth adapter on Chrome OS

device/bluetooth/bluetooth_adapter_bluez.h

 // Copyright 2013 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_BLUETOOTH_BLUETOOTH_ADAPTER_BLUEZ_H_
 #define DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_BLUEZ_H_
 
 #include <stdint.h>
 
 #include <map>
@@ skipping 68 matching lines @@
 
   // BluetoothAdapter:
   void Shutdown() override;
   std::string GetAddress() const override;
   std::string GetName() const override;
   void SetName(const std::string& name,
                const base::Closure& callback,
                const ErrorCallback& error_callback) override;
   bool IsInitialized() const override;
   bool IsPresent() const override;
+  void SetDisabled(bool disabled) override;
   bool IsPowered() const override;
   void SetPowered(bool powered,
                   const base::Closure& callback,
                   const ErrorCallback& error_callback) override;
   bool IsDiscoverable() const override;
   void SetDiscoverable(bool discoverable,
                        const base::Closure& callback,
                        const ErrorCallback& error_callback) override;
   bool IsDiscovering() const override;
   void CreateRfcommService(
@@ skipping 240 matching lines @@
   // ended (with either success or failure).
   void ProcessQueuedDiscoveryRequests();
 
   InitCallback init_callback_;
 
   bool initialized_;
 
   // Set in |Shutdown()|, makes IsPresent()| return false.
   bool dbus_is_shutdown_;
 
+  // Set in |SetDisabled()|, makes |IsPresent()| return false
+  // (just disables the interface, doesn't actually shut it down).

[Andrew T Wilson (Slow), 2016/04/13 13:55:11]

One question - why did we decide to change the value of IsPresent() rather than
asking callers to call IsDisabled() to check whether it's disabled?

[Ivan Šandrk, 2016/04/13 17:19:04]

It seemed like a simpler solution since no code changes elsewhere were needed,
and it also imposes less on users of the Bluetooth adapter (maybe someone
forgets to call IsDisabled together with IsPresent). But it's not the cleanest
solution, that's true.

+  bool is_disabled_;
+
   // Number of discovery sessions that have been added.
   int num_discovery_sessions_;
 
   // True, if there is a pending request to start or stop discovery.
   bool discovery_request_pending_;
 
   // List of queued requests to add new discovery sessions. While there is a
   // pending request to BlueZ to start or stop discovery, many requests from
   // within Chrome to start or stop discovery sessions may occur. We only
   // queue requests to add new sessions to be processed later. All requests to
@@ skipping 27 matching lines @@
   // Note: This should remain the last member so it'll be destroyed and
   // invalidate its weak pointers before any other members are destroyed.
   base::WeakPtrFactory<BluetoothAdapterBlueZ> weak_ptr_factory_;
 
   DISALLOW_COPY_AND_ASSIGN(BluetoothAdapterBlueZ);
 };
 
 }  // namespace bluez
 
 #endif  // DEVICE_BLUETOOTH_BLUETOOTH_ADAPTER_BLUEZ_H_