Restricting output device authorization timeout to Windows only.

content/renderer/media/audio_device_factory.cc

 // Copyright (c) 2012 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 "content/renderer/media/audio_device_factory.h"
 
 #include <algorithm>
 
 #include "base/logging.h"
 #include "content/common/content_constants_internal.h"
 #include "content/renderer/media/audio_input_message_filter.h"
 #include "content/renderer/media/audio_message_filter.h"
 #include "content/renderer/media/audio_renderer_mixer_manager.h"
 #include "content/renderer/render_thread_impl.h"
 #include "media/audio/audio_input_device.h"
 #include "media/audio/audio_output_device.h"
 #include "media/base/audio_latency.h"
 #include "media/base/audio_renderer_mixer_input.h"
 #include "url/origin.h"
 
 namespace content {
 
 // static
 AudioDeviceFactory* AudioDeviceFactory::factory_ = NULL;
 
 namespace {
+#if defined(OS_WIN)
 const int64_t kMaxAuthorizationTimeoutMs = 900;

[Guido Urdaneta, 2016/06/30 09:11:02]

Add a comment explaining why the timeout is necessary on Windows.
Make a reference to the bug.

+#else
+const int64_t kMaxAuthorizationTimeoutMs = 0;  // No timeout.
+#endif  // defined(OS_WIN)
 
 media::AudioLatency::LatencyType GetSourceLatencyType(
     AudioDeviceFactory::SourceType source) {
   switch (source) {
     case AudioDeviceFactory::kSourceWebAudioInteractive:
       return media::AudioLatency::LATENCY_INTERACTIVE;
     case AudioDeviceFactory::kSourceNone:
     case AudioDeviceFactory::kSourceWebRtc:
     case AudioDeviceFactory::kSourceNonRtcAudioTrack:
     case AudioDeviceFactory::kSourceWebAudioBalanced:
@@ skipping 162 matching lines @@
                                                device_id, security_origin);
     if (sink)
       return sink;
   }
 
   return NewOutputDevice(render_frame_id, session_id, device_id,
                          security_origin);
 }
 
 }  // namespace content