Add initial implementation of SourceBufferStream

media/base/buffers.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.
 
 // Defines a base class for representing timestamped media data. Every buffer
 // contains a timestamp in microseconds describing the relative position of
 // the buffer within the media stream, and the duration in microseconds for
 // the length of time the buffer will be rendered.
 //
 // Timestamps are derived directly from the encoded media file and are commonly
@@ skipping 55 matching lines @@
     timestamp_ = timestamp;
   }
 
   base::TimeDelta GetDuration() const {
     return duration_;
   }
   void SetDuration(const base::TimeDelta& duration) {
     duration_ = duration;
   }
 
+  // TODO(vrk): Only ChunkDemuxer and friends care about these methods.
+  // Put these methods inside a derived class of Buffer?
+  bool IsKeyframe() const { return is_keyframe_; }
+  void SetKeyframe(bool is_keyframe) {

[acolwell GONE FROM CHROMIUM, 2012/04/26 00:52:34]

Could we get away with just providing this info via the constructor and do away
with the setter?

[vrk (LEFT CHROMIUM), 2012/04/27 23:19:30]

Done, but it's a little icky since I have to add a DataBuffer constructor as
well. Let me know if you still prefer it this way.

+    is_keyframe_ = is_keyframe;
+  }
+
  protected:
   friend class base::RefCountedThreadSafe<Buffer>;
   Buffer(base::TimeDelta timestamp, base::TimeDelta duration);
   virtual ~Buffer();
 
  private:
   base::TimeDelta timestamp_;
   base::TimeDelta duration_;
+  bool is_keyframe_;
 
   DISALLOW_COPY_AND_ASSIGN(Buffer);
 };
 
 }  // namespace media
 
 #endif  // MEDIA_BASE_BUFFERS_H_