Fix null argument to base::GetProcId in RenderProcessHostImpl::CreateMessageFilters.

content/browser/renderer_host/media/audio_input_renderer_host.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/browser/renderer_host/media/audio_input_renderer_host.h"
 
 #include <utility>
 
 #include "base/bind.h"
 #include "base/command_line.h"
@@ skipping 76 matching lines @@
       shared_memory_segment_count(0),
       pending_close(false),
       has_keyboard_mic(false) {
 }
 
 AudioInputRendererHost::AudioEntry::~AudioEntry() {
 }
 
 AudioInputRendererHost::AudioInputRendererHost(
     int render_process_id,
-    int32_t renderer_pid,
     media::AudioManager* audio_manager,
     MediaStreamManager* media_stream_manager,
     AudioMirroringManager* audio_mirroring_manager,
     media::UserInputMonitor* user_input_monitor)
     : BrowserMessageFilter(AudioMsgStart),
       render_process_id_(render_process_id),
-      renderer_pid_(renderer_pid),
+      renderer_pid_(0),
       audio_manager_(audio_manager),
       media_stream_manager_(media_stream_manager),
       audio_mirroring_manager_(audio_mirroring_manager),
       user_input_monitor_(user_input_monitor),
       audio_log_(MediaInternals::GetInstance()->CreateAudioLog(
           media::AudioLogFactory::AUDIO_INPUT_CONTROLLER)) {}
 
 AudioInputRendererHost::~AudioInputRendererHost() {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   DCHECK(audio_entries_.empty());
 }
 
 #if BUILDFLAG(ENABLE_WEBRTC)
 void AudioInputRendererHost::EnableDebugRecording(const base::FilePath& file) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  if (audio_entries_.empty())
+    return;
   base::FilePath file_with_extensions =
       GetDebugRecordingFilePathWithExtensions(file);
   for (const auto& entry : audio_entries_)
     EnableDebugRecordingForId(file_with_extensions, entry.first);
 }
 
 void AudioInputRendererHost::DisableDebugRecording() {
   for (const auto& entry : audio_entries_) {
     entry.second->controller->DisableDebugRecording();
   }
@@ skipping 418 matching lines @@
         ->UnregisterKeyboardMicStream();
   }
 #endif
 }
 
 #if BUILDFLAG(ENABLE_WEBRTC)
 void AudioInputRendererHost::MaybeEnableDebugRecordingForId(int stream_id) {
   DCHECK_CURRENTLY_ON(BrowserThread::UI);
   if (WebRTCInternals::GetInstance()->IsAudioDebugRecordingsEnabled()) {
     BrowserThread::PostTask(
-        BrowserThread::IO,
-        FROM_HERE,
+        BrowserThread::IO, FROM_HERE,
         base::Bind(
-            &AudioInputRendererHost::EnableDebugRecordingForId,
+            &AudioInputRendererHost::
+                AddExtensionsToPathAndEnableDebugRecordingForId,
             this,
-            GetDebugRecordingFilePathWithExtensions(
-                WebRTCInternals::GetInstance()->
-                    GetAudioDebugRecordingsFilePath()),
+            WebRTCInternals::GetInstance()->GetAudioDebugRecordingsFilePath(),
             stream_id));
   }
 }
 
 #if defined(OS_WIN)
 #define IntToStringType base::IntToString16
 #else
 #define IntToStringType base::IntToString
 #endif
 
 base::FilePath AudioInputRendererHost::GetDebugRecordingFilePathWithExtensions(
     const base::FilePath& file) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  // We expect |renderer_pid_| to be set.
+  DCHECK_GT(renderer_pid_, 0);
   return file.AddExtension(IntToStringType(renderer_pid_))
              .AddExtension(kDebugRecordingFileNameAddition);
 }
 
 void AudioInputRendererHost::EnableDebugRecordingForId(
     const base::FilePath& file_name,
     int stream_id) {
   DCHECK_CURRENTLY_ON(BrowserThread::IO);
   AudioEntry* entry = LookupById(stream_id);
   if (!entry)
     return;
   entry->controller->EnableDebugRecording(
       file_name.AddExtension(IntToStringType(stream_id))
           .AddExtension(kDebugRecordingFileNameExtension));
 }
 
 #undef IntToStringType
 
+void AudioInputRendererHost::AddExtensionsToPathAndEnableDebugRecordingForId(
+    const base::FilePath& file,
+    int stream_id) {
+  DCHECK_CURRENTLY_ON(BrowserThread::IO);
+  EnableDebugRecordingForId(GetDebugRecordingFilePathWithExtensions(file),
+                            stream_id);
+}
+
 #endif  // BUILDFLAG(ENABLE_WEBRTC)
 
 }  // namespace content