Simplify logic behind chrome.audio.setActiveDevices

extensions/browser/api/audio/audio_service.h

 // Copyright (c) 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 EXTENSIONS_BROWSER_API_AUDIO_AUDIO_SERVICE_H_
 #define EXTENSIONS_BROWSER_API_AUDIO_AUDIO_SERVICE_H_
 
+#include <memory>
 #include <string>
 #include <vector>
 
 #include "base/callback.h"
 #include "base/macros.h"
 #include "extensions/common/api/audio.h"
 
 namespace extensions {
 
 using OutputInfo = std::vector<api::audio::OutputDeviceInfo>;
@@ skipping 30 matching lines @@
   // Called by listeners to this service to add/remove themselves as observers.
   virtual void AddObserver(Observer* observer) = 0;
   virtual void RemoveObserver(Observer* observer) = 0;
 
   // Start to query audio device information. Should be called on UI thread.
   // Populates |output_info_out| and |input_info_out| with the results.
   // Returns true on success.
   virtual bool GetInfo(OutputInfo* output_info_out,
                        InputInfo* input_info_out) = 0;
 
+  virtual bool SetActiveDeviceLists(
+      const std::unique_ptr<DeviceIdList>& input_devices,
+      const std::unique_ptr<DeviceIdList>& output_devives) = 0;

[jennyz, 2017/01/07 00:33:00]

Please add some comments about the new function.

[tbarzic, 2017/01/07 05:16:31]

Done.

+
   // Sets the active devices to the devices specified by |device_list|.
   // It can pass in the "complete" active device list of either input
   // devices, or output devices, or both. If only input devices are passed in,
   // it will only change the input devices' active status, output devices will
   // NOT be changed; similarly for the case if only output devices are passed.
   // If the devices specified in |new_active_ids| are already active, they will
   // remain active. Otherwise, the old active devices will be de-activated
   // before we activate the new devices with the same type(input/output).
   virtual void SetActiveDevices(const DeviceIdList& device_list) = 0;
 
   // Set the muted and volume/gain properties of a device.
   virtual bool SetDeviceProperties(const std::string& device_id,
                                    bool muted,
                                    int volume,
                                    int gain) = 0;
 
  protected:
   AudioService() {}
 
  private:
   DISALLOW_COPY_AND_ASSIGN(AudioService);
 };
 
 }  // namespace extensions
 
 #endif  // EXTENSIONS_BROWSER_API_AUDIO_AUDIO_SERVICE_H_