Remove memset from VideoFrame and mark buffer as unpoisoned

media/base/video_frame_pool.cc

 // Copyright 2013 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.
 
 #include "media/base/video_frame_pool.h"
 
 #include <list>
 
 #include "base/bind.h"
 #include "base/memory/ref_counted.h"
 #include "base/synchronization/lock.h"
 
 namespace media {
 
 class VideoFramePool::PoolImpl
     : public base::RefCountedThreadSafe<VideoFramePool::PoolImpl> {
  public:
   PoolImpl();
 
   // Returns a frame from the pool that matches the specified
   // parameters or creates a new frame if no suitable frame exists in
   // the pool. The pool is drained if no matching frame is found.
   scoped_refptr<VideoFrame> CreateFrame(VideoFrame::Format format,
                                         const gfx::Size& coded_size,
                                         const gfx::Rect& visible_rect,
                                         const gfx::Size& natural_size,
-                                        base::TimeDelta timestamp);
+                                        base::TimeDelta timestamp,
+                                        bool zero_out_memory);

[DaleCurtis, 2015/07/09 22:21:16]

This is a bit confusing since it would only happen on new frames and not reused
frames, so we either need a better name or this should be a constructor argument
with some more details. Either way a comment should be added explaining the
parameter precisely. Maybe "zero_initialize_new_allocations" ? Though it is a
bit wordy.

[emircan, 2015/07/09 23:16:27]

True, I will go with "zero_init_new_allocations".

 
   // Shuts down the frame pool and releases all frames in |frames_|.
   // Once this is called frames will no longer be inserted back into
   // |frames_|.
   void Shutdown();
 
   size_t GetPoolSizeForTesting() const { return frames_.size(); }
 
  private:
   friend class base::RefCountedThreadSafe<VideoFramePool::PoolImpl>;
@@ skipping 15 matching lines @@
 
 VideoFramePool::PoolImpl::~PoolImpl() {
   DCHECK(is_shutdown_);
 }
 
 scoped_refptr<VideoFrame> VideoFramePool::PoolImpl::CreateFrame(
     VideoFrame::Format format,
     const gfx::Size& coded_size,
     const gfx::Rect& visible_rect,
     const gfx::Size& natural_size,
-    base::TimeDelta timestamp) {
+    base::TimeDelta timestamp,
+    bool zero_out_memory) {
   base::AutoLock auto_lock(lock_);
   DCHECK(!is_shutdown_);
 
   scoped_refptr<VideoFrame> frame;
 
   while (!frame.get() && !frames_.empty()) {
       scoped_refptr<VideoFrame> pool_frame = frames_.front();
       frames_.pop_front();
 
       if (pool_frame->format() == format &&
           pool_frame->coded_size() == coded_size &&
           pool_frame->visible_rect() == visible_rect &&
           pool_frame->natural_size() == natural_size) {
         frame = pool_frame;
         frame->set_timestamp(timestamp);
         break;
       }
   }
 
   if (!frame.get()) {
     frame = VideoFrame::CreateFrame(
         format, coded_size, visible_rect, natural_size, timestamp);
+    if (zero_out_memory)
+      memset(frame->data(0), 0, VideoFrame::AlignedAllocationSize(frame));
   }
 
   scoped_refptr<VideoFrame> wrapped_frame = VideoFrame::WrapVideoFrame(
       frame, frame->visible_rect(), frame->natural_size());
   wrapped_frame->AddDestructionObserver(
       base::Bind(&VideoFramePool::PoolImpl::FrameReleased, this, frame));
   return wrapped_frame;
 }
 
 void VideoFramePool::PoolImpl::Shutdown() {
@@ skipping 16 matching lines @@
 
 VideoFramePool::~VideoFramePool() {
   pool_->Shutdown();
 }
 
 scoped_refptr<VideoFrame> VideoFramePool::CreateFrame(
     VideoFrame::Format format,
     const gfx::Size& coded_size,
     const gfx::Rect& visible_rect,
     const gfx::Size& natural_size,
-    base::TimeDelta timestamp) {
+    base::TimeDelta timestamp,
+    bool zero_out_memory) {
   return pool_->CreateFrame(format, coded_size, visible_rect, natural_size,
-                            timestamp);
+                            timestamp, zero_out_memory);
 }
 
 size_t VideoFramePool::GetPoolSizeForTesting() const {
   return pool_->GetPoolSizeForTesting();
 }
 
 }  // namespace media