Add SupportsConfigChanges() to DemuxerStream.

media/filters/fake_demuxer_stream.cc

 // Copyright (c) 2013 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/filters/fake_demuxer_stream.h"
 
 #include "base/bind.h"
 #include "base/callback_helpers.h"
 #include "base/location.h"
 #include "base/logging.h"
@@ skipping 67 matching lines @@
 // TODO(xhwang): Support audio if needed.
 DemuxerStream::Type FakeDemuxerStream::type() {
   DCHECK(task_runner_->BelongsToCurrentThread());
   return VIDEO;
 }
 
 void FakeDemuxerStream::EnableBitstreamConverter() {
   DCHECK(task_runner_->BelongsToCurrentThread());
 }
 
+bool FakeDemuxerStream::SupportsConfigChanges() {
+  return true;

[scherkus (not reviewing), 2014/03/25 22:16:47]

is this needed for some tests?

do we need some tests to cover the false behaviour?

[rileya (GONE FROM CHROMIUM), 2014/03/25 22:43:19]

Yeah, it's used by the tests. I made it return true or false depending on
whether the FDS was initialized with multiple configs. I also added a test for
the 'false' behavior.

+}
+
 void FakeDemuxerStream::HoldNextRead() {
   DCHECK(task_runner_->BelongsToCurrentThread());
   read_to_hold_ = next_read_num_;
 }
 
 void FakeDemuxerStream::HoldNextConfigChangeRead() {
   DCHECK(task_runner_->BelongsToCurrentThread());
   // Set |read_to_hold_| to be the next config change read.
   read_to_hold_ = next_read_num_ + num_buffers_in_one_config_ -
                   next_read_num_ % (num_buffers_in_one_config_ + 1);
@@ skipping 61 matching lines @@
 
   num_buffers_left_in_current_config_--;
   if (num_buffers_left_in_current_config_ == 0)
     num_configs_left_--;
 
   num_buffers_returned_++;
   base::ResetAndReturn(&read_cb_).Run(kOk, buffer);
 }
 
 }  // namespace media