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

Side by Side Diff: media/video/gpu_memory_buffer_video_frame_pool_unittest.cc

Issue 1427543002: Modified old wait sync point functions to also accept new sync tokens. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Folded sync_point into sync_tokens Created 5 years, 1 month 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 #include "base/bind.h" 5 #include "base/bind.h"
6 #include "base/test/test_simple_task_runner.h" 6 #include "base/test/test_simple_task_runner.h"
7 #include "gpu/command_buffer/client/gles2_interface_stub.h" 7 #include "gpu/command_buffer/client/gles2_interface_stub.h"
8 #include "media/base/video_frame.h" 8 #include "media/base/video_frame.h"
9 #include "media/renderers/mock_gpu_video_accelerator_factories.h" 9 #include "media/renderers/mock_gpu_video_accelerator_factories.h"
10 #include "media/video/gpu_memory_buffer_video_frame_pool.h" 10 #include "media/video/gpu_memory_buffer_video_frame_pool.h"
(...skipping 112 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 123
124 TEST_F(GpuMemoryBufferVideoFramePoolTest, ReuseFirstResource) { 124 TEST_F(GpuMemoryBufferVideoFramePoolTest, ReuseFirstResource) {
125 scoped_refptr<VideoFrame> software_frame = CreateTestYUVVideoFrame(10); 125 scoped_refptr<VideoFrame> software_frame = CreateTestYUVVideoFrame(10);
126 scoped_refptr<VideoFrame> frame; 126 scoped_refptr<VideoFrame> frame;
127 gpu_memory_buffer_pool_->MaybeCreateHardwareFrame( 127 gpu_memory_buffer_pool_->MaybeCreateHardwareFrame(
128 software_frame, base::Bind(MaybeCreateHardwareFrameCallback, &frame)); 128 software_frame, base::Bind(MaybeCreateHardwareFrameCallback, &frame));
129 RunUntilIdle(); 129 RunUntilIdle();
130 130
131 EXPECT_NE(software_frame.get(), frame.get()); 131 EXPECT_NE(software_frame.get(), frame.get());
132 gpu::Mailbox mailbox = frame->mailbox_holder(0).mailbox; 132 gpu::Mailbox mailbox = frame->mailbox_holder(0).mailbox;
133 unsigned sync_point = frame->mailbox_holder(0).sync_point; 133 const gpu::SyncTokne sync_token = frame->mailbox_holder(0).sync_token;
piman 2015/10/29 00:08:20 typo: SyncToken
David Yen 2015/10/29 17:08:08 Done.
134 EXPECT_EQ(3u, gles2_->gen_textures); 134 EXPECT_EQ(3u, gles2_->gen_textures);
135 135
136 scoped_refptr<VideoFrame> frame2; 136 scoped_refptr<VideoFrame> frame2;
137 gpu_memory_buffer_pool_->MaybeCreateHardwareFrame( 137 gpu_memory_buffer_pool_->MaybeCreateHardwareFrame(
138 software_frame, base::Bind(MaybeCreateHardwareFrameCallback, &frame2)); 138 software_frame, base::Bind(MaybeCreateHardwareFrameCallback, &frame2));
139 RunUntilIdle(); 139 RunUntilIdle();
140 140
141 EXPECT_NE(software_frame.get(), frame2.get()); 141 EXPECT_NE(software_frame.get(), frame2.get());
142 EXPECT_NE(mailbox, frame2->mailbox_holder(0).mailbox); 142 EXPECT_NE(mailbox, frame2->mailbox_holder(0).mailbox);
143 EXPECT_EQ(6u, gles2_->gen_textures); 143 EXPECT_EQ(6u, gles2_->gen_textures);
144 144
145 frame = nullptr; 145 frame = nullptr;
146 frame2 = nullptr; 146 frame2 = nullptr;
147 RunUntilIdle(); 147 RunUntilIdle();
148 148
149 gpu_memory_buffer_pool_->MaybeCreateHardwareFrame( 149 gpu_memory_buffer_pool_->MaybeCreateHardwareFrame(
150 software_frame, base::Bind(MaybeCreateHardwareFrameCallback, &frame)); 150 software_frame, base::Bind(MaybeCreateHardwareFrameCallback, &frame));
151 RunUntilIdle(); 151 RunUntilIdle();
152 152
153 EXPECT_NE(software_frame.get(), frame.get()); 153 EXPECT_NE(software_frame.get(), frame.get());
154 EXPECT_EQ(6u, gles2_->gen_textures); 154 EXPECT_EQ(6u, gles2_->gen_textures);
155 EXPECT_EQ(frame->mailbox_holder(0).mailbox, mailbox); 155 EXPECT_EQ(frame->mailbox_holder(0).mailbox, mailbox);
156 EXPECT_NE(frame->mailbox_holder(0).sync_point, sync_point); 156 EXPECT_NE(frame->mailbox_holder(0).sync_token, sync_token);
157 } 157 }
158 158
159 TEST_F(GpuMemoryBufferVideoFramePoolTest, DropResourceWhenSizeIsDifferent) { 159 TEST_F(GpuMemoryBufferVideoFramePoolTest, DropResourceWhenSizeIsDifferent) {
160 scoped_refptr<VideoFrame> frame; 160 scoped_refptr<VideoFrame> frame;
161 gpu_memory_buffer_pool_->MaybeCreateHardwareFrame( 161 gpu_memory_buffer_pool_->MaybeCreateHardwareFrame(
162 CreateTestYUVVideoFrame(10), 162 CreateTestYUVVideoFrame(10),
163 base::Bind(MaybeCreateHardwareFrameCallback, &frame)); 163 base::Bind(MaybeCreateHardwareFrameCallback, &frame));
164 RunUntilIdle(); 164 RunUntilIdle();
165 165
166 EXPECT_EQ(3u, gles2_->gen_textures); 166 EXPECT_EQ(3u, gles2_->gen_textures);
(...skipping 43 matching lines...) Expand 10 before | Expand all | Expand 10 after
210 gpu_memory_buffer_pool_->MaybeCreateHardwareFrame( 210 gpu_memory_buffer_pool_->MaybeCreateHardwareFrame(
211 software_frame, base::Bind(MaybeCreateHardwareFrameCallback, &frame)); 211 software_frame, base::Bind(MaybeCreateHardwareFrameCallback, &frame));
212 212
213 RunUntilIdle(); 213 RunUntilIdle();
214 214
215 EXPECT_NE(software_frame.get(), frame.get()); 215 EXPECT_NE(software_frame.get(), frame.get());
216 EXPECT_EQ(3u, gles2_->gen_textures); 216 EXPECT_EQ(3u, gles2_->gen_textures);
217 } 217 }
218 218
219 } // namespace media 219 } // namespace media
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698