Make check for 'opened' consistent across all audio stream implementations of Start() and log an ...

media/audio/mac/audio_output_mac.cc

 // Copyright (c) 2011 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/audio/mac/audio_output_mac.h"
 
 #include "base/basictypes.h"
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "base/memory/scoped_ptr.h"
@@ skipping 461 matching lines @@
       // indeed been closed.
       if (!audio_stream->GetSource())
         return;
     }
     audio_stream->HandleError(err);
   }
 }
 
 void PCMQueueOutAudioOutputStream::Start(AudioSourceCallback* callback) {
   DCHECK(callback);
+  DLOG_IF(ERROR, !audio_queue_) << "Open() has not been called successfully";
+  if (!audio_queue_)
+    return;
+
   OSStatus err = noErr;
   SetSource(callback);
   pending_bytes_ = 0;
   // Ask the source to pre-fill all our buffers before playing.
   for (uint32 ix = 0; ix != kNumBuffers; ++ix) {
     buffer_[ix]->mAudioDataByteSize = 0;
     // Caller waits for 1st packet to become available, but not for others,
     // so we wait for them here.
     if (ix != 0) {
       AudioSourceCallback* source = GetSource();
@@ skipping 21 matching lines @@
 void PCMQueueOutAudioOutputStream::SetSource(AudioSourceCallback* source) {
   base::AutoLock lock(source_lock_);
   source_ = source;
 }
 
 AudioOutputStream::AudioSourceCallback*
 PCMQueueOutAudioOutputStream::GetSource() {
   base::AutoLock lock(source_lock_);
   return source_;
 }