Set the texture to cleared in VideoDecodeAccelerator::PictureReady.

content/common/gpu/media/exynos_video_decode_accelerator.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.
 //
 // This file contains an implementation of VideoDecoderAccelerator
 // that utilizes the hardware video decoder present on the Exynos SoC.
 
 #ifndef CONTENT_COMMON_GPU_MEDIA_EXYNOS_VIDEO_DECODE_ACCELERATOR_H_
 #define CONTENT_COMMON_GPU_MEDIA_EXYNOS_VIDEO_DECODE_ACCELERATOR_H_
 
 #include <list>
 #include <vector>
 
 #include "base/callback_forward.h"
 #include "base/memory/linked_ptr.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/threading/thread.h"
 #include "content/common/content_export.h"
 #include "content/common/gpu/media/video_decode_accelerator_impl.h"
 #include "media/base/limits.h"
 #include "media/base/video_decoder_config.h"
+#include "media/video/picture.h"
 #include "ui/gfx/size.h"
 #include "ui/gl/gl_bindings.h"
 
 namespace base {
 class MessageLoopProxy;
 }
 
 namespace content {
 class H264Parser;
 
@@ skipping 138 matching lines @@
   // Record for GSC output buffers.
   struct GscOutputRecord {
     GscOutputRecord();
     ~GscOutputRecord();
     bool at_device;        // held by device.
     bool at_client;        // held by client.
     int fd;                // file descriptor from backing EGLImage.
     EGLImageKHR egl_image; // backing EGLImage.
     EGLSyncKHR egl_sync;   // sync the compositor's use of the EGLImage.
     int32 picture_id;      // picture buffer id as returned to PictureReady().
+    bool cleared;          // the texture has been cleared
   };
 
   //
   // Decoding tasks, to be run on decode_thread_.
   //
 
   // Enqueue a BitstreamBuffer to decode.  This will enqueue a buffer to the
   // decoder_input_queue_, then queue a DecodeBufferTask() to actually decode
   // the buffer.
   void DecodeTask(const media::BitstreamBuffer& bitstream_buffer);
@@ skipping 119 matching lines @@
   // Methods run on child thread.
   //
 
   // Destroy buffers.
   void DestroyMfcInputBuffers();
   void DestroyMfcOutputBuffers();
   void DestroyGscInputBuffers();
   void DestroyGscOutputBuffers();
   void ResolutionChangeDestroyBuffers();
 
+  // Send decoded buffers to PictureReady.
+  void SendPictureReady();
+
+  // Callback that indicates a picture has been cleared.
+  void PictureCleared();
+
   // Our original calling message loop for the child thread.
   scoped_refptr<base::MessageLoopProxy> child_message_loop_proxy_;
 
   // Message loop of the IO thread.
   scoped_refptr<base::MessageLoopProxy> io_message_loop_proxy_;
 
   // WeakPtr<> pointing to |this| for use in posting tasks from the decoder or
   // device worker threads back to the child thread.  Because the worker threads
   // are members of this class, any task running on those threads is guaranteed
   // that this object is still alive.  As a result, tasks posted from the child
@@ skipping 101 matching lines @@
 
   // GSC output buffer state.
   bool gsc_output_streamon_;
   // GSC output buffers enqueued to device.
   int gsc_output_buffer_queued_count_;
   // Output buffers ready to use.  We need a FIFO here.
   std::list<int> gsc_free_output_buffers_;
   // Mapping of int index to GSC output buffer record.
   std::vector<GscOutputRecord> gsc_output_buffer_map_;
 
+  // Pictures that are ready but not sent to PictureReady yet.
+  std::queue<std::pair<bool, media::Picture> > pending_picture_ready_;

[Ami GONE FROM CHROMIUM, 2013/09/30 18:03:03]

Doco what the bool is

[piman, 2013/09/30 21:49:13]

nit: Better yet, make it a struct with explicit field names rather than "first"
and "second"

[wuchengli, 2013/10/01 03:49:43]

Done.

+
+  // The number of pictures that are sent to PictureReady and will be cleared.
+  int picture_clearing_count_;
+
   // Output picture size.
   gfx::Size frame_buffer_size_;
 
   //
   // The device polling thread handles notifications of V4L2 device changes.
   //
 
   // The thread.
   base::Thread device_poll_thread_;
   // eventfd fd to signal device poll thread when its poll() should be
@@ skipping 13 matching lines @@
 
   // The codec we'll be decoding for.
   media::VideoCodecProfile video_profile_;
 
   DISALLOW_COPY_AND_ASSIGN(ExynosVideoDecodeAccelerator);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_COMMON_GPU_MEDIA_EXYNOS_VIDEO_DECODE_ACCELERATOR_H_