Fix "unused variable" warnings.

media/cast/sender/audio_encoder.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 "media/cast/sender/audio_encoder.h"
 
 #include <stdint.h>
 
 #include <algorithm>
 #include <limits>
@@ skipping 345 matching lines @@
                  num_channels,
                  sampling_rate,
                  kAccessUnitSamples,
                  callback),
         input_buffer_(AudioBus::Create(num_channels, kAccessUnitSamples)),
         input_bus_(AudioBus::CreateWrapper(num_channels)),
         max_access_unit_size_(0),
         output_buffer_(nullptr),
         converter_(nullptr),
         file_(nullptr),
-        num_access_units_(0),
-        can_resume_(true) {
+        num_access_units_(0) {
     if (ImplBase::operational_status_ != STATUS_UNINITIALIZED) {
       return;
     }
     if (!Initialize(sampling_rate, bitrate)) {
       ImplBase::operational_status_ = STATUS_INVALID_CONFIGURATION;
       return;
     }
     ImplBase::operational_status_ = STATUS_INITIALIZED;
   }
 
@@ skipping 70 matching lines @@
     // or compatible with the output sampling rate or channels).
     if (bitrate > 0) {
       prop_size = sizeof(int);
       if (AudioConverterSetProperty(
               converter_, kAudioConverterEncodeBitRate, prop_size, &bitrate) !=
           noErr) {
         return false;
       }
     }
 
-#if defined(OS_IOS)
-    // See the comment next to |can_resume_| for details on resumption. Some
-    // converters can return kAudioConverterErr_PropertyNotSupported, in which
-    // case resumption is implicitly supported. This is the only location where
-    // the implementation modifies |can_resume_|.
-    uint32_t can_resume;
-    prop_size = sizeof(can_resume);
-    OSStatus oserr = AudioConverterGetProperty(
-        converter_,
-        kAudioConverterPropertyCanResumeFromInterruption,
-        &prop_size,
-        &can_resume);
-    if (oserr == noErr) {
-      const_cast<bool&>(can_resume_) = can_resume != 0;

[Peter Kasting, 2016/05/19 07:23:29]

This member is never read, and the bug associated with doing anything with it
was WontFixed last year.

[hubbe, 2016/05/19 17:21:41]

Can you point me to the bug in question?

[Peter Kasting, 2016/05/19 18:10:18]

It's the one referred to in the other deleted block below:
https://crbug.com/424787

-    }
-#endif
-
     // Figure out the maximum size of an access unit that the encoder can
     // produce. |mBytesPerPacket| will be 0 for variable size configurations,
     // in which case we must query the value.
     uint32_t max_access_unit_size = out_asbd.mBytesPerPacket;
     if (max_access_unit_size == 0) {
       prop_size = sizeof(max_access_unit_size);
       if (AudioConverterGetProperty(
               converter_,
               kAudioConverterPropertyMaximumOutputPacketSize,
               &prop_size,
@@ skipping 232 matching lines @@
 
   // The |AudioFile| is responsible for ADTS packetization.
   AudioFileID file_;
 
   // An |AudioBufferList| passed to the converter to store encoded samples.
   AudioBufferList converter_abl_;
 
   // The number of access units emitted so far by the encoder.
   uint64_t num_access_units_;
 
-  // On iOS, audio codecs can be interrupted by other services (such as an
-  // audio alert or phone call). Depending on the underlying hardware and
-  // configuration, the codec may have to be thrown away and re-initialized
-  // after such an interruption. This flag tracks if we can resume or not from
-  // such an interruption. It is initialized to true, which is the only possible
-  // value on OS X and on most modern iOS hardware.
-  // TODO(jfroy): Implement encoder re-initialization after interruption.
-  //              https://crbug.com/424787
-  const bool can_resume_;
-
   DISALLOW_COPY_AND_ASSIGN(AppleAacImpl);
 };
 #endif  // defined(OS_MACOSX)
 
 class AudioEncoder::Pcm16Impl : public AudioEncoder::ImplBase {
  public:
   Pcm16Impl(const scoped_refptr<CastEnvironment>& cast_environment,
             int num_channels,
             int sampling_rate,
             const FrameEncodedCallback& callback)
@@ skipping 119 matching lines @@
   cast_environment_->PostTask(CastEnvironment::AUDIO,
                               FROM_HERE,
                               base::Bind(&AudioEncoder::ImplBase::EncodeAudio,
                                          impl_,
                                          base::Passed(&audio_bus),
                                          recorded_time));
 }
 
 }  // namespace cast
 }  // namespace media