Remove unused parameter to OnError()

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 "base/bind.h"
 #include "base/metrics/histogram.h"
 #include "base/process.h"
 #include "base/shared_memory.h"
@@ skipping 75 matching lines @@
     media::AudioInputController* controller) {
   BrowserThread::PostTask(
       BrowserThread::IO,
       FROM_HERE,
       base::Bind(
           &AudioInputRendererHost::DoSendRecordingMessage,
           this,
           make_scoped_refptr(controller)));
 }
 
 void AudioInputRendererHost::OnError(

[scherkus (not reviewing), 2013/03/21 17:37:24]

nit: this looks like it can fit on one line (i.e., <= 80 chars)

[jrummell, 2013/03/21 18:17:31]

Done.

-    media::AudioInputController* controller,
-    int error_code) {
+    media::AudioInputController* controller) {
   BrowserThread::PostTask(
       BrowserThread::IO,
       FROM_HERE,
       base::Bind(&AudioInputRendererHost::DoHandleError,
-          this, make_scoped_refptr(controller), error_code));
+          this, make_scoped_refptr(controller)));

[scherkus (not reviewing), 2013/03/21 17:37:24]

nit: can you fix indentation here?

it should line up with the &AudioInputRendererHost::DoHandleError parameter

[jrummell, 2013/03/21 18:17:31]

Done.

 }
 
 void AudioInputRendererHost::OnData(media::AudioInputController* controller,
                                     const uint8* data,
                                     uint32 size) {
   NOTREACHED() << "Only low-latency mode is supported.";
 }
 
 void AudioInputRendererHost::DoCompleteCreation(
     media::AudioInputController* controller) {
@@ skipping 50 matching lines @@
 }
 
 void AudioInputRendererHost::DoSendRecordingMessage(
     media::AudioInputController* controller) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
   // TODO(henrika): See crbug.com/115262 for details on why this method
   // should be implemented.
 }
 
 void AudioInputRendererHost::DoHandleError(
-    media::AudioInputController* controller,
-    int error_code) {
-  DLOG(WARNING) << "AudioInputRendererHost::DoHandleError(error_code="
-                << error_code << ")";
+    media::AudioInputController* controller) {
   DCHECK(BrowserThread::CurrentlyOn(BrowserThread::IO));
 
   AudioEntry* entry = LookupByController(controller);
   if (!entry)
     return;
 
   DeleteEntryOnError(entry);
 }
 
 bool AudioInputRendererHost::OnMessageReceived(const IPC::Message& message,
@@ skipping 219 matching lines @@
   // TODO(hclam): Implement a faster look up method.
   for (AudioEntryMap::iterator i = audio_entries_.begin();
        i != audio_entries_.end(); ++i) {
     if (controller == i->second->controller.get())
       return i->second;
   }
   return NULL;
 }
 
 }  // namespace content