VideoCaptureBufferPool: Refactor to allow support of non-ShMem backed buffers

content/browser/renderer_host/media/video_capture_buffer_pool_unittest.cc

 // Copyright (c) 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.
 
 // Unit test for VideoCaptureBufferPool.
 
 #include "content/browser/renderer_host/media/video_capture_buffer_pool.h"
 
 #include "base/bind.h"
 #include "base/memory/ref_counted.h"
@@ skipping 27 matching lines @@
   };
   VideoCaptureBufferPoolTest()
       : expected_dropped_id_(0),
         pool_(new VideoCaptureBufferPool(3)) {}
 
   void ExpectDroppedId(int expected_dropped_id) {
     expected_dropped_id_ = expected_dropped_id;
   }
 
   scoped_ptr<Buffer> ReserveI420Buffer(const gfx::Size& dimensions) {
-    const size_t frame_bytes =
-        media::VideoFrame::AllocationSize(media::VideoFrame::I420, dimensions);
     // To verify that ReserveI420Buffer always sets |buffer_id_to_drop|,
     // initialize it to something different than the expected value.
     int buffer_id_to_drop = ~expected_dropped_id_;
-    int buffer_id = pool_->ReserveForProducer(frame_bytes, &buffer_id_to_drop);
+    int buffer_id = pool_->ReserveForProducer(media::PIXEL_FORMAT_I420,
+                                              dimensions, &buffer_id_to_drop);
     if (buffer_id == VideoCaptureBufferPool::kInvalidId)
       return scoped_ptr<Buffer>();
 
     void* memory;
     size_t size;
     pool_->GetBufferInfo(buffer_id, &memory, &size);
     EXPECT_EQ(expected_dropped_id_, buffer_id_to_drop);
     return scoped_ptr<Buffer>(new Buffer(pool_, buffer_id, memory, size));
   }
 
@@ skipping 129 matching lines @@
   memset(buffer2->data(), 0x22, buffer2->size());
   memset(buffer4->data(), 0x55, buffer4->size());
 
   buffer2.reset();
 
   memset(buffer4->data(), 0x77, buffer4->size());
   buffer4.reset();
 }
 
 } // namespace content