Chromecast: Add stream id interface in DecoderBufferBase class

chromecast/media/cma/base/decoder_buffer_adapter.h

Index: chromecast/media/cma/base/decoder_buffer_adapter.h
diff --git a/chromecast/media/cma/base/decoder_buffer_adapter.h b/chromecast/media/cma/base/decoder_buffer_adapter.h
index 054e60ee59ea1a9adc865434aca0558d27081cea..e81a07fe6120e2721d9b1d4e298259a63d7d18b8 100644
--- a/chromecast/media/cma/base/decoder_buffer_adapter.h
+++ b/chromecast/media/cma/base/decoder_buffer_adapter.h
@@ -22,8 +22,11 @@ class DecoderBufferAdapter : public DecoderBufferBase {
  public:
   explicit DecoderBufferAdapter(

[gunsch, 2015/05/21 21:06:34]

why not just change the primary ctor instead of having two ctors?

[erickung1, 2015/05/21 21:38:34]

I found out some test code and component which doesn't really require stream_id
in the decoder buffer. It makes thing odd to force all of them adding something
like new DecoderBufferAdapter(-1, buffer) in the code

But Please let me know. I don't have strong opinion on both solutions.

       const scoped_refptr< ::media::DecoderBuffer>& buffer);
+  DecoderBufferAdapter(
+      int type, const scoped_refptr< ::media::DecoderBuffer>& buffer);

[gunsch, 2015/05/21 21:06:34]

type --> stream_id

[erickung1, 2015/05/21 21:38:34]

Done.

 
   // DecoderBufferBase implementation.
+  int stream_id() const override;

[gunsch, 2015/05/21 21:06:34]

what values can a stream_id have?

if only 2-3 possible values, could this be an enum?

[erickung1, 2015/05/21 21:38:34]

we'll use two possible value.
However, I think about leaving it more flexible which can be scale later on. 

[gunsch, 2015/05/21 21:45:29]

what is inflexible about enum? you can always add more enum values...

"-1" has no inherently semantic indication as far as "stream ids" go, so it's
completely unclear to the reader what it means when seeing it in the code.
Having "-1" littered around the code seems like a pretty clear anti-pattern.

[erickung1, 2015/05/21 23:32:21]

Make sense. I change the type to enum

   base::TimeDelta timestamp() const override;
   const uint8* data() const override;
   uint8* writable_data() const override;
@@ -34,6 +37,7 @@ class DecoderBufferAdapter : public DecoderBufferBase {
  private:
   ~DecoderBufferAdapter() override;
 
+  int stream_id_;
   scoped_refptr< ::media::DecoderBuffer> const buffer_;
 
   DISALLOW_COPY_AND_ASSIGN(DecoderBufferAdapter);