Add SupportsConfigChanges() to DemuxerStream.

media/base/demuxer_stream.h

 // Copyright (c) 2012 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.
 
 #ifndef MEDIA_BASE_DEMUXER_STREAM_H_
 #define MEDIA_BASE_DEMUXER_STREAM_H_
 
 #include "base/callback.h"
 #include "base/memory/ref_counted.h"
 #include "media/base/media_export.h"
@@ skipping 50 matching lines @@
 
   // Returns the video decoder configuration. It is an error to call this method
   // if type() != VIDEO.
   virtual VideoDecoderConfig video_decoder_config() = 0;
 
   // Returns the type of stream.
   virtual Type type() = 0;
 
   virtual void EnableBitstreamConverter() = 0;
 
+  // Whether or not this DemuxerStream allows midstream configuration changes.

[scherkus (not reviewing), 2014/03/25 21:14:53]

I know EnableBitsreamConverter() is sorely lacking in documentation, but we can
certainly make this better

the question I have is an API consumer what does this mean for me? how will a
DemuxerStream operate differently when this returns true vs. false? (e.g., will
video_decoder_config() change?)

[rileya (GONE FROM CHROMIUM), 2014/03/25 21:45:06]

Added more detail.

+  virtual bool SupportsConfigChanges();

[DaleCurtis, 2014/03/24 19:01:00]

Unfortunately, this should be = 0; and each implementation will need to define
this method.

[rileya (GONE FROM CHROMIUM), 2014/03/25 21:45:06]

Done.

+
  protected:
   // Only allow concrete implementations to get deleted.
   virtual ~DemuxerStream();
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_DEMUXER_STREAM_H_