Refactor AudioParameters member setting.

chrome/browser/media/webrtc_browsertest_audio.cc

 // Copyright 2014 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 "chrome/browser/media/webrtc_browsertest_audio.h"
 
 #include "base/files/file.h"
 #include "base/files/file_path.h"
 #include "media/audio/audio_parameters.h"
 #include "media/audio/audio_power_monitor.h"
@@ skipping 61 matching lines @@
 
   // Set the filter coefficient to the whole file's duration; this will make the
   // power monitor take the entire file into account.
   media::AudioPowerMonitor power_monitor(wav_audio_handler->sample_rate(),
                                          file_duration);
   power_monitor.Scan(*audio_bus, audio_bus->frames());
 
   file_parameters->Reset(
       media::AudioParameters::AUDIO_PCM_LOW_LATENCY,
       media::GuessChannelLayout(wav_audio_handler->num_channels()),
-      wav_audio_handler->num_channels(), wav_audio_handler->sample_rate(),
-      wav_audio_handler->bits_per_sample(), wav_audio_handler->total_frames());
+      wav_audio_handler->sample_rate(), wav_audio_handler->bits_per_sample(),
+      wav_audio_handler->total_frames());
+  file_parameters->set_channels_for_discrete(wav_audio_handler->num_channels());

[DaleCurtis, 2015/09/04 17:03:39]

Hmm, I'm not a fan of this calling pattern, but it's not really any worse than
what we have today... and I really like the 1-constructor cleanup. :|

[ajm, 2015/09/04 19:02:53]

Yeah I'm not completely satisfied with it either, but it was the least bad thing
I could come up with. Open to alternatives.

[tommi (sloooow) - chröme, 2015/09/05 09:49:57]

Ack.  No better suggestions atm.

 
   return power_monitor.ReadCurrentPowerAndClip().first;
 }
 
 }  // namespace test