Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(349)

Side by Side Diff: content/browser/renderer_host/media/video_capture_buffer_pool_unittest.cc

Issue 1064703002: VideoCaptureBufferPool: Refactor to allow support of non-ShMem backed buffers (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright (c) 2013 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 // Unit test for VideoCaptureBufferPool. 5 // Unit test for VideoCaptureBufferPool.
6 6
7 #include "content/browser/renderer_host/media/video_capture_buffer_pool.h" 7 #include "content/browser/renderer_host/media/video_capture_buffer_pool.h"
8 8
9 #include "base/bind.h" 9 #include "base/bind.h"
10 #include "base/memory/ref_counted.h" 10 #include "base/memory/ref_counted.h"
(...skipping 27 matching lines...) Expand all
38 }; 38 };
39 VideoCaptureBufferPoolTest() 39 VideoCaptureBufferPoolTest()
40 : expected_dropped_id_(0), 40 : expected_dropped_id_(0),
41 pool_(new VideoCaptureBufferPool(3)) {} 41 pool_(new VideoCaptureBufferPool(3)) {}
42 42
43 void ExpectDroppedId(int expected_dropped_id) { 43 void ExpectDroppedId(int expected_dropped_id) {
44 expected_dropped_id_ = expected_dropped_id; 44 expected_dropped_id_ = expected_dropped_id;
45 } 45 }
46 46
47 scoped_ptr<Buffer> ReserveI420Buffer(const gfx::Size& dimensions) { 47 scoped_ptr<Buffer> ReserveI420Buffer(const gfx::Size& dimensions) {
48 const size_t frame_bytes = 48 const media::VideoCaptureFormat format(dimensions, 0.0,
49 media::VideoFrame::AllocationSize(media::VideoFrame::I420, dimensions); 49 media::PIXEL_FORMAT_I420);
50 // To verify that ReserveI420Buffer always sets |buffer_id_to_drop|, 50 // To verify that ReserveI420Buffer always sets |buffer_id_to_drop|,
51 // initialize it to something different than the expected value. 51 // initialize it to something different than the expected value.
52 int buffer_id_to_drop = ~expected_dropped_id_; 52 int buffer_id_to_drop = ~expected_dropped_id_;
53 int buffer_id = pool_->ReserveForProducer(frame_bytes, &buffer_id_to_drop); 53 int buffer_id = pool_->ReserveForProducer(format, &buffer_id_to_drop);
54 if (buffer_id == VideoCaptureBufferPool::kInvalidId) 54 if (buffer_id == VideoCaptureBufferPool::kInvalidId)
55 return scoped_ptr<Buffer>(); 55 return scoped_ptr<Buffer>();
56 56
57 void* memory; 57 void* memory;
58 size_t size; 58 size_t size;
59 pool_->GetBufferInfo(buffer_id, &memory, &size); 59 pool_->GetBufferInfo(buffer_id, &memory, &size);
60 EXPECT_EQ(expected_dropped_id_, buffer_id_to_drop); 60 EXPECT_EQ(expected_dropped_id_, buffer_id_to_drop);
61 return scoped_ptr<Buffer>(new Buffer(pool_, buffer_id, memory, size)); 61 return scoped_ptr<Buffer>(new Buffer(pool_, buffer_id, memory, size));
62 } 62 }
63 63
(...skipping 129 matching lines...) Expand 10 before | Expand all | Expand 10 after
193 memset(buffer2->data(), 0x22, buffer2->size()); 193 memset(buffer2->data(), 0x22, buffer2->size());
194 memset(buffer4->data(), 0x55, buffer4->size()); 194 memset(buffer4->data(), 0x55, buffer4->size());
195 195
196 buffer2.reset(); 196 buffer2.reset();
197 197
198 memset(buffer4->data(), 0x77, buffer4->size()); 198 memset(buffer4->data(), 0x77, buffer4->size());
199 buffer4.reset(); 199 buffer4.reset();
200 } 200 }
201 201
202 } // namespace content 202 } // namespace content
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698