Implement sourceAddId() & sourceRemoveId()

media/filters/chunk_demuxer.h

Index: media/filters/chunk_demuxer.h
diff --git a/media/filters/chunk_demuxer.h b/media/filters/chunk_demuxer.h
index ae690b912fc798d8070bd238587783ecca8b4a19..87166b9f814f45cc35fb9a98bdb783a73f906168 100644
--- a/media/filters/chunk_demuxer.h
+++ b/media/filters/chunk_demuxer.h
@@ -6,6 +6,7 @@
 #define MEDIA_FILTERS_CHUNK_DEMUXER_H_
 
 #include <list>
+#include <string>
 
 #include "base/synchronization/lock.h"
 #include "media/base/byte_queue.h"
@@ -22,6 +23,13 @@ class FFmpegURLProtocol;
 // from JavaScript to the media stack.
 class MEDIA_EXPORT ChunkDemuxer : public Demuxer, public StreamParserHost {
  public:
+  // Status codes for AddId() method.
+  enum AddIdStatus {
+    AisNoError,         // ID added w/o error.

[scherkus (not reviewing), 2012/04/12 20:58:32]

I'm calling silliness on the naming here!

1) Why not "Status" -- are we going to have multiple status codes very soon? I
don't see anything in the spec that hints at additional status codes
2) Constants are kCamelCase
3) "Ais" is confusing: is it "A prefix, the word 'is'" (as in "A, isNoError") or
short for AddIdStatus (as in "AddIdStatus NoError")
4) kOK is a pithier way of saying AisNoError (bonus: matches the new status
codes added to BufferedResourceLoader!)

If we go with just "Status" we could clear up all this stuff by having:

enum Status {
  kOK,
  kNotSupported,
  kReachedIdLimit,
};

[acolwell GONE FROM CHROMIUM, 2012/04/12 22:43:12]

Done.

+    AisNotSupported,    // Type specified is not supported.
+    AisReachedIdLimit,  // Reached ID limit. We can't handle any more IDs.
+  };
+
   explicit ChunkDemuxer(ChunkDemuxerClient* client);
   virtual ~ChunkDemuxer();
 
@@ -43,7 +51,9 @@ class MEDIA_EXPORT ChunkDemuxer : public Demuxer, public StreamParserHost {
 
   // Appends media data to the stream. Returns false if this method
   // is called in an invalid state.

[scherkus (not reviewing), 2012/04/12 20:58:32]

docs need updating

[acolwell GONE FROM CHROMIUM, 2012/04/12 22:43:12]

Done.

-  bool AppendData(const uint8* data, size_t length);
+  AddIdStatus AddId(const std::string& id, const std::string& type);
+  void RemoveId(const std::string& id);
+  bool AppendData(const std::string& id, const uint8* data, size_t length);
   void EndOfStream(PipelineStatus status);
   bool HasEnded();
   void Shutdown();
@@ -95,6 +105,9 @@ class MEDIA_EXPORT ChunkDemuxer : public Demuxer, public StreamParserHost {
 
   ByteQueue byte_queue_;
 
+  // TODO(acolwell): Remove this when fixing http://crbug.com/122909 .
+  std::string source_id_;
+
   DISALLOW_COPY_AND_ASSIGN(ChunkDemuxer);
 };