Introduce AudioRendererSink::IsOptimizedForHardwareParameters

media/blink/webaudiosourceprovider_impl.cc

 // 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.
 
 #include "media/blink/webaudiosourceprovider_impl.h"
 
 #include <vector>
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
@@ skipping 234 matching lines @@
 }
 
 bool WebAudioSourceProviderImpl::SetVolume(double volume) {
   base::AutoLock auto_lock(sink_lock_);
   volume_ = volume;
   if (!client_ && sink_)
     sink_->SetVolume(volume);
   return true;
 }
 
-OutputDeviceInfo WebAudioSourceProviderImpl::GetOutputDeviceInfo() {
+OutputDeviceInfo WebAudioSourceProviderImpl::GetOutputDeviceInfo(
+    const AudioParameters& preferred_params) {
   base::AutoLock auto_lock(sink_lock_);
-  return sink_ ? sink_->GetOutputDeviceInfo()
-               : OutputDeviceInfo(OUTPUT_DEVICE_STATUS_ERROR_NOT_FOUND);
+
+  OutputDeviceInfo sink_info = sink_->GetOutputDeviceInfo();

[o1ka, 2017/04/26 14:27:54]

What if |sink_| is nullptr?

[flim-chromium, 2017/04/26 16:24:09]

Fixed, thanks for spotting this!

+
+  if (client_) {
+    return OutputDeviceInfo(sink_info.device_id(), sink_info.device_status(),
+                            preferred_params);
+  }
+
+  if (sink_) {
+    return sink_info;
+  }
+
+  return OutputDeviceInfo(OUTPUT_DEVICE_STATUS_ERROR_NOT_FOUND);
 }
 
 bool WebAudioSourceProviderImpl::CurrentThreadIsRenderingThread() {
   NOTIMPLEMENTED();
   return false;
 }
 
 void WebAudioSourceProviderImpl::SwitchOutputDevice(
     const std::string& device_id,
     const url::Origin& security_origin,
@@ skipping 63 matching lines @@
 
   return num_rendered_frames;
 }
 
 void WebAudioSourceProviderImpl::TeeFilter::OnRenderError() {
   DCHECK(IsInitialized());
   renderer_->OnRenderError();
 }
 
 }  // namespace media