chrome.audio API: treat mute as system wide property

extensions/browser/api/audio/audio_api.cc

Index: extensions/browser/api/audio/audio_api.cc
diff --git a/extensions/browser/api/audio/audio_api.cc b/extensions/browser/api/audio/audio_api.cc
index 53bd0da17f5cb2838c3d4de6f8af69d55ce490e8..dabb853b1a277ce58ee9d4361b662e30a28d5d02 100644
--- a/extensions/browser/api/audio/audio_api.cc
+++ b/extensions/browser/api/audio/audio_api.cc
@@ -59,14 +59,21 @@ void AudioAPI::OnLevelChanged(const std::string& id, int level) {
 }
 
 void AudioAPI::OnMuteChanged(bool is_input, bool is_muted) {
-  if (EventRouter::Get(browser_context_)) {
-    std::unique_ptr<base::ListValue> args =
-        audio::OnMuteChanged::Create(is_input, is_muted);
-    std::unique_ptr<Event> event(new Event(events::AUDIO_ON_MUTE_CHANGED,
-                                           audio::OnMuteChanged::kEventName,
-                                           std::move(args)));
-    EventRouter::Get(browser_context_)->BroadcastEvent(std::move(event));
-  }
+  EventRouter* event_router = EventRouter::Get(browser_context_);
+  if (!event_router)
+    return;
+
+  // Dispatch onMuteChanged event.
+  audio::MuteChangedEvent raw_event;
+  raw_event.stream_type =
+      is_input ? audio::STREAM_TYPE_INPUT : audio::STREAM_TYPE_OUTPUT;
+  raw_event.is_muted = is_muted;
+  std::unique_ptr<base::ListValue> event_args =
+      audio::OnMuteChanged::Create(raw_event);
+  std::unique_ptr<Event> event(new Event(events::AUDIO_ON_MUTE_CHANGED,
+                                         audio::OnMuteChanged::kEventName,
+                                         std::move(event_args)));
+  event_router->BroadcastEvent(std::move(event));
 }
 
 void AudioAPI::OnDevicesChanged(const DeviceInfoList& devices) {
@@ -140,13 +147,52 @@ ExtensionFunction::ResponseAction AudioSetPropertiesFunction::Run() {
   int gain_value = params->properties.gain.get() ?
       *params->properties.gain : -1;
 
-  if (!service->SetDeviceProperties(params->id, params->properties.is_muted,
-                                    volume_value, gain_value)) {
-    return RespondNow(Error("Could not set properties"));
+  if (!service->SetDeviceSoundLevel(params->id, volume_value, gain_value))
+    return RespondNow(Error("Could not set volume/gain properties"));
+
+  if (params->properties.is_muted.get() &&
+      !service->SetMuteForDevice(params->id, *params->properties.is_muted)) {
+    return RespondNow(Error("Could not set mute property."));
+  }
+
+  return RespondNow(NoArguments());
+}
+
+///////////////////////////////////////////////////////////////////////////////
+
+ExtensionFunction::ResponseAction AudioSetMuteFunction::Run() {
+  std::unique_ptr<audio::SetMute::Params> params(
+      audio::SetMute::Params::Create(*args_));
+  EXTENSION_FUNCTION_VALIDATE(params.get());
+
+  AudioService* service =
+      AudioAPI::GetFactoryInstance()->Get(browser_context())->GetService();
+  DCHECK(service);
+
+  if (!service->SetMute(params->stream_type == audio::STREAM_TYPE_INPUT,
+                        params->is_muted)) {
+    return RespondNow(Error("Could not set mute state."));
   }
   return RespondNow(NoArguments());
 }
 
 ///////////////////////////////////////////////////////////////////////////////
 
+ExtensionFunction::ResponseAction AudioGetMuteFunction::Run() {
+  std::unique_ptr<audio::GetMute::Params> params(
+      audio::GetMute::Params::Create(*args_));
+  EXTENSION_FUNCTION_VALIDATE(params.get());
+
+  AudioService* service =
+      AudioAPI::GetFactoryInstance()->Get(browser_context())->GetService();
+  DCHECK(service);
+
+  bool value = false;
+  if (!service->GetMute(params->stream_type == audio::STREAM_TYPE_INPUT,
+                        &value)) {
+    return RespondNow(Error("Could not get mute state."));
+  }
+  return RespondNow(ArgumentList(audio::GetMute::Results::Create(value)));
+}
+
 }  // namespace extensions