Set the texture to cleared in VideoDecodeAccelerator::PictureReady.

content/common/gpu/media/gpu_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.
 
 #ifndef CONTENT_COMMON_GPU_MEDIA_GPU_VIDEO_DECODE_ACCELERATOR_H_
 #define CONTENT_COMMON_GPU_MEDIA_GPU_VIDEO_DECODE_ACCELERATOR_H_
 
 #include <vector>
 
 #include "base/compiler_specific.h"
@@ skipping 63 matching lines @@
   void OnAssignPictureBuffers(const std::vector<int32>& buffer_ids,
                               const std::vector<uint32>& texture_ids);
   void OnReusePictureBuffer(int32 picture_buffer_id);
   void OnFlush();
   void OnReset();
   void OnDestroy();
 
   // Called on IO thread when |filter_| has been removed.
   void OnFilterRemoved();
 
+  // Sets the texture to cleared. Returns false if an error has occurred.
+  bool SetTextureCleared(const media::Picture& picture);
+
   // Message to Send() when initialization is done.  Is only non-NULL during
   // initialization and is owned by the IPC channel underlying the
   // GpuCommandBufferStub.
   IPC::Message* init_done_msg_;
 
   // Route ID to communicate with the host.
   int32 host_route_id_;
 
   // Unowned pointer to the underlying GpuCommandBufferStub.
   GpuCommandBufferStub* stub_;
@@ skipping 16 matching lines @@
 
   // GPU child message loop.
   scoped_refptr<base::MessageLoopProxy> child_message_loop_;
 
   // GPU IO message loop.
   scoped_refptr<base::MessageLoopProxy> io_message_loop_;
 
   // Weak pointers will be invalidated on IO thread.
   base::WeakPtrFactory<Client> weak_factory_for_io_;
 
+  // Picture buffer ID that have not set level cleared.
+  std::vector<int32> buffer_ids_;
+
+  // Texture ID that have not set level cleared.
+  std::vector<uint32> texture_ids_;

[Ami GONE FROM CHROMIUM, 2013/09/26 16:21:56]

comments for this and previous field could be clearer (s/set level cleared/been
cleared/)

names for both fields would benefit from an uncleared_ prefix, IMO.

[wuchengli, 2013/09/30 16:11:21]

Done.

+
   DISALLOW_IMPLICIT_CONSTRUCTORS(GpuVideoDecodeAccelerator);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_COMMON_GPU_MEDIA_GPU_VIDEO_DECODE_ACCELERATOR_H_