Changing overstrict CHECKs to just setting the object into an invalid state.

media/audio/linux/alsa_output.cc

 // Copyright (c) 2009 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.
 //
 // The audio stream implementation is made difficult because different methods
 // are available for calling depending on what state the stream is.  Here is the
 // state transition table for the stream.
 //
 // STATE_CREATED -> Open() -> STATE_OPENED
 // STATE_OPENED -> Start() -> STATE_STARTED
@@ skipping 72 matching lines @@
       source_callback_(NULL),
       playback_thread_("PlaybackThread"),
       channels_(channels),
       sample_rate_(sample_rate),
       bits_per_sample_(bits_per_sample),
       min_buffer_frames_((min_buffer_ms * sample_rate_) /
                          base::Time::kMillisecondsPerSecond),
       packet_size_(0),
       device_write_suspended_(true),  // Start suspended.
       resources_released_(false) {
-  CHECK(channels_ == 2) << "Only 2-channel audio is supported right now.";
-  CHECK(AudioManager::AUDIO_PCM_LINEAR == format)
-      << "Only linear PCM supported.";
-  CHECK(bits_per_sample % 8 == 0) << "Only allow byte-aligned samples";
-
   // Reference self to avoid accidental deletion before the message loop is
   // done.
   AddRef();
 
+  // Sanity check input values.
+  if (channels_ != 2) {
+    LOG(WARNING) << "Only 2-channel audio is supported right now.";
+    state_ = STATE_ERROR;
+  }
+
+  if (AudioManager::AUDIO_PCM_LINEAR != format) {
+    LOG(WARNING) << "Only linear PCM supported.";
+    state_ = STATE_ERROR;
+  }
+
+  if (bits_per_sample % 8 != 0) {
+    // We do this explicitly just incase someone messes up the switch below.
+    LOG(WARNING) << "Only allow byte-aligned samples";
+    state_ = STATE_ERROR;
+  }
+
   switch (bits_per_sample) {
     case 8:
       pcm_format_ = SND_PCM_FORMAT_S8;
       break;
 
     case 16:
       pcm_format_ = SND_PCM_FORMAT_S16;
       break;
 
     case 24:
@@ skipping 400 matching lines @@
   NOTIMPLEMENTED();
 }
 
 void AlsaPCMOutputStream::GetVolume(double* left_level, double* right_level) {
   NOTIMPLEMENTED();
 }
 
 size_t AlsaPCMOutputStream::GetNumBuffers() {
   return 0;
 }