Implement proper synchronization between HW video decode IPC and CommandBuffer.

media/video/video_decode_accelerator.h

Index: media/video/video_decode_accelerator.h
diff --git a/media/video/video_decode_accelerator.h b/media/video/video_decode_accelerator.h
index c50da976f97f88ff471e076ebc91a3133966b116..f13e7d636507bf31b1c98c773b0d4e8031594a77 100644
--- a/media/video/video_decode_accelerator.h
+++ b/media/video/video_decode_accelerator.h
@@ -257,7 +257,7 @@ class VideoDecodeAccelerator {
   //  |bitstream_buffer| is the input bitstream that is sent for decoding.
   //
   // Returns true when command successfully accepted. Otherwise false.

[scherkus (not reviewing), 2011/06/27 22:54:47]

update docs

[Ami GONE FROM CHROMIUM, 2011/06/28 00:06:35]

Done.

-  virtual bool Decode(const BitstreamBuffer& bitstream_buffer) = 0;
+  virtual void Decode(const BitstreamBuffer& bitstream_buffer) = 0;
 
   // Assigns a set of picture buffers to the video decoder.
   // AssignGLESBuffers assigns texture-backed buffers.
@@ -287,7 +287,7 @@ class VideoDecodeAccelerator {
   // flushing, the decode will call NotifyFlushDone().
   //
   // Returns true when command successfully accepted. Otherwise false.

[scherkus (not reviewing), 2011/06/27 22:54:47]

update docs

[Ami GONE FROM CHROMIUM, 2011/06/28 00:06:35]

Done.

-  virtual bool Flush() = 0;
+  virtual void Flush() = 0;
 
   // Aborts the decoder. Decode will abort the decoding as soon as possible and
   // will not output anything. NotifyAbortDone() is called  as soon as abort has
@@ -295,7 +295,7 @@ class VideoDecodeAccelerator {
   // when there has not been callbacks to dismiss them.
   //
   // Returns true when command successfully accepted. Otherwise false.

[scherkus (not reviewing), 2011/06/27 22:54:47]

update docs

[Ami GONE FROM CHROMIUM, 2011/06/28 00:06:35]

Done.

-  virtual bool Abort() = 0;
+  virtual void Abort() = 0;
 };
 
 }  // namespace media