VAVDA: Redesign stage 1.

content/common/gpu/media/h264_dpb.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.

[Ami GONE FROM CHROMIUM, 2013/05/22 23:59:47]

WARNING: due to the vagaries of rietveld some of my comments are on the
patchset#1 version of files, some are on the patchset #2 version of files.  I
recommend you either use the email interface to make sure you see all comments,
or else view the PS#1->PS#2 diff view in rietveld.

 // 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 an H.264 Decoded Picture Buffer
 // used in H264 decoders.
 
 #ifndef CONTENT_COMMON_GPU_MEDIA_H264_DPB_H_
 #define CONTENT_COMMON_GPU_MEDIA_H264_DPB_H_
 
 #include <vector>
@@ skipping 52 matching lines @@
 // Stores decoded pictures that will be used for future display
 // and/or reference.
 class H264DPB {
  public:
   H264DPB();
   ~H264DPB();
 
   void set_max_num_pics(size_t max_num_pics);
   size_t max_num_pics() { return max_num_pics_; }
 
-  // Remove unused (not reference and already outputted) pictures from DPB.
-  void RemoveUnused();
+  // Remove unused (not reference and already outputted) pictures from DPB
+  // and free it.
+  void DeleteUnused();
 
-  // Remove a picture by its pic_order_cnt.
-  void RemoveByPOC(int poc);
+  // Remove a picture by its pic_order_cnt and free it.
+  void DeleteByPOC(int poc);
 
   // Clear DPB.
   void Clear();
 
   // Store picture in DPB. DPB takes ownership of its resources.
   void StorePic(H264Picture* pic);
 
   // Return the number of reference pictures in DPB.
   int CountRefPics();
 
@@ skipping 37 matching lines @@
  private:
   Pictures pics_;
   size_t max_num_pics_;
 
   DISALLOW_COPY_AND_ASSIGN(H264DPB);
 };
 
 }  // namespace content
 
 #endif  // CONTENT_COMMON_GPU_MEDIA_H264_DPB_H_