| OLD | NEW |
| 1 diff --git a/cc/BUILD.gn b/cc/BUILD.gn | 1 diff --git a/cc/BUILD.gn b/cc/BUILD.gn |
| 2 index 0411b45..4ee07db 100644 | 2 index bd8bfdd..6e55656 100644 |
| 3 --- a/cc/BUILD.gn | 3 --- a/cc/BUILD.gn |
| 4 +++ b/cc/BUILD.gn | 4 +++ b/cc/BUILD.gn |
| 5 @@ -222,13 +222,6 @@ component("cc") { | 5 @@ -189,13 +189,6 @@ component("cc") { |
| 6 "layers/ui_resource_layer.h", | 6 "layers/ui_resource_layer.h", |
| 7 "layers/ui_resource_layer_impl.cc", | 7 "layers/ui_resource_layer_impl.cc", |
| 8 "layers/ui_resource_layer_impl.h", | 8 "layers/ui_resource_layer_impl.h", |
| 9 - "layers/video_frame_provider.h", | 9 - "layers/video_frame_provider.h", |
| 10 - "layers/video_frame_provider_client_impl.cc", | 10 - "layers/video_frame_provider_client_impl.cc", |
| 11 - "layers/video_frame_provider_client_impl.h", | 11 - "layers/video_frame_provider_client_impl.h", |
| 12 - "layers/video_layer.cc", | 12 - "layers/video_layer.cc", |
| 13 - "layers/video_layer.h", | 13 - "layers/video_layer.h", |
| 14 - "layers/video_layer_impl.cc", | 14 - "layers/video_layer_impl.cc", |
| 15 - "layers/video_layer_impl.h", | 15 - "layers/video_layer_impl.h", |
| 16 "output/begin_frame_args.cc", | 16 "output/begin_frame_args.cc", |
| 17 "output/begin_frame_args.h", | 17 "output/begin_frame_args.h", |
| 18 "output/bsp_tree.cc", | 18 "output/bsp_tree.cc", |
| 19 @@ -486,8 +479,6 @@ component("cc") { | 19 @@ -458,8 +451,6 @@ component("cc") { |
| 20 "resources/ui_resource_client.h", | 20 "resources/ui_resource_client.h", |
| 21 "resources/ui_resource_request.cc", | 21 "resources/ui_resource_request.cc", |
| 22 "resources/ui_resource_request.h", | 22 "resources/ui_resource_request.h", |
| 23 - "resources/video_resource_updater.cc", | 23 - "resources/video_resource_updater.cc", |
| 24 - "resources/video_resource_updater.h", | 24 - "resources/video_resource_updater.h", |
| 25 "resources/zero_copy_tile_task_worker_pool.cc", | 25 "resources/zero_copy_tile_task_worker_pool.cc", |
| 26 "resources/zero_copy_tile_task_worker_pool.h", | 26 "resources/zero_copy_tile_task_worker_pool.h", |
| 27 "scheduler/begin_frame_source.cc", | 27 "scheduler/begin_frame_source.cc", |
| 28 @@ -628,8 +619,6 @@ source_set("test_support") { | 28 @@ -606,8 +597,6 @@ source_set("test_support") { |
| 29 "test/fake_tile_manager_client.h", | 29 "test/fake_tile_manager_client.h", |
| 30 "test/fake_ui_resource_layer_tree_host_impl.cc", | 30 "test/fake_ui_resource_layer_tree_host_impl.cc", |
| 31 "test/fake_ui_resource_layer_tree_host_impl.h", | 31 "test/fake_ui_resource_layer_tree_host_impl.h", |
| 32 - "test/fake_video_frame_provider.cc", | 32 - "test/fake_video_frame_provider.cc", |
| 33 - "test/fake_video_frame_provider.h", | 33 - "test/fake_video_frame_provider.h", |
| 34 "test/geometry_test_utils.cc", | 34 "test/geometry_test_utils.cc", |
| 35 "test/geometry_test_utils.h", | 35 "test/geometry_test_utils.h", |
| 36 "test/impl_side_painting_settings.h", | 36 "test/impl_side_painting_settings.h", |
| 37 @@ -787,7 +776,6 @@ test("cc_unittests") { | 37 @@ -766,7 +755,6 @@ test("cc_unittests") { |
| 38 "layers/tiled_layer_unittest.cc", | 38 "layers/tiled_layer_unittest.cc", |
| 39 "layers/ui_resource_layer_impl_unittest.cc", | 39 "layers/ui_resource_layer_impl_unittest.cc", |
| 40 "layers/ui_resource_layer_unittest.cc", | 40 "layers/ui_resource_layer_unittest.cc", |
| 41 - "layers/video_layer_impl_unittest.cc", | 41 - "layers/video_layer_impl_unittest.cc", |
| 42 "output/begin_frame_args_unittest.cc", | 42 "output/begin_frame_args_unittest.cc", |
| 43 "output/delegating_renderer_unittest.cc", | 43 "output/delegating_renderer_unittest.cc", |
| 44 "output/filter_operations_unittest.cc", | 44 "output/filter_operations_unittest.cc", |
| 45 @@ -848,7 +836,6 @@ test("cc_unittests") { | 45 @@ -827,7 +815,6 @@ test("cc_unittests") { |
| 46 "trees/layer_tree_host_unittest_picture.cc", | 46 "trees/layer_tree_host_unittest_picture.cc", |
| 47 "trees/layer_tree_host_unittest_proxy.cc", | 47 "trees/layer_tree_host_unittest_proxy.cc", |
| 48 "trees/layer_tree_host_unittest_scroll.cc", | 48 "trees/layer_tree_host_unittest_scroll.cc", |
| 49 - "trees/layer_tree_host_unittest_video.cc", | 49 - "trees/layer_tree_host_unittest_video.cc", |
| 50 "trees/layer_tree_impl_unittest.cc", | 50 "trees/layer_tree_impl_unittest.cc", |
| 51 "trees/occlusion_tracker_unittest.cc", | 51 "trees/occlusion_tracker_unittest.cc", |
| 52 "trees/occlusion_unittest.cc", | 52 "trees/occlusion_unittest.cc", |
| 53 diff --git a/cc/layers/video_frame_provider.h b/cc/layers/video_frame_provider.h | 53 diff --git a/cc/layers/video_frame_provider.h b/cc/layers/video_frame_provider.h |
| 54 deleted file mode 100644 | 54 deleted file mode 100644 |
| 55 index 784d951..0000000 | 55 index 784d951..0000000 |
| (...skipping 178 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 234 - stream_texture_matrix_ = gfx::Transform( | 234 - stream_texture_matrix_ = gfx::Transform( |
| 235 - matrix[0], matrix[4], matrix[8], matrix[12], | 235 - matrix[0], matrix[4], matrix[8], matrix[12], |
| 236 - matrix[1], matrix[5], matrix[9], matrix[13], | 236 - matrix[1], matrix[5], matrix[9], matrix[13], |
| 237 - matrix[2], matrix[6], matrix[10], matrix[14], | 237 - matrix[2], matrix[6], matrix[10], matrix[14], |
| 238 - matrix[3], matrix[7], matrix[11], matrix[15]); | 238 - matrix[3], matrix[7], matrix[11], matrix[15]); |
| 239 - if (active_video_layer_) | 239 - if (active_video_layer_) |
| 240 - active_video_layer_->SetNeedsRedraw(); | 240 - active_video_layer_->SetNeedsRedraw(); |
| 241 -} | 241 -} |
| 242 - | 242 - |
| 243 -} // namespace cc | 243 -} // namespace cc |
| 244 diff --git a/cc/output/gl_renderer.cc b/cc/output/gl_renderer.cc | |
| 245 index 613ea4b..3813e1d 100644 | |
| 246 --- a/cc/output/gl_renderer.cc | |
| 247 +++ b/cc/output/gl_renderer.cc | |
| 248 @@ -18,7 +18,6 @@ | |
| 249 #include "build/build_config.h" | |
| 250 #include "base/trace_event/trace_event.h" | |
| 251 #include "cc/base/math_util.h" | |
| 252 -#include "cc/layers/video_layer_impl.h" | |
| 253 #include "cc/output/compositor_frame.h" | |
| 254 #include "cc/output/compositor_frame_metadata.h" | |
| 255 #include "cc/output/context_provider.h" | |
| 256 diff --git a/cc/output/renderer_pixeltest.cc b/cc/output/renderer_pixeltest.cc | 244 diff --git a/cc/output/renderer_pixeltest.cc b/cc/output/renderer_pixeltest.cc |
| 257 index 295fc84..ebda8d6 100644 | 245 index d02df4f..9166ea8 100644 |
| 258 --- a/cc/output/renderer_pixeltest.cc | 246 --- a/cc/output/renderer_pixeltest.cc |
| 259 +++ b/cc/output/renderer_pixeltest.cc | 247 +++ b/cc/output/renderer_pixeltest.cc |
| 260 @@ -12,7 +12,6 @@ | 248 @@ -11,7 +11,6 @@ |
| 261 #include "cc/test/fake_picture_pile_impl.h" | 249 #include "cc/test/fake_picture_pile_impl.h" |
| 262 #include "cc/test/pixel_test.h" | 250 #include "cc/test/pixel_test.h" |
| 263 #include "gpu/command_buffer/client/gles2_interface.h" | 251 #include "gpu/command_buffer/client/gles2_interface.h" |
| 264 -#include "media/base/video_frame.h" | 252 -#include "media/base/video_frame.h" |
| 265 #include "third_party/skia/include/core/SkColorPriv.h" | 253 #include "third_party/skia/include/core/SkColorPriv.h" |
| 266 #include "third_party/skia/include/core/SkImageFilter.h" | 254 #include "third_party/skia/include/core/SkImageFilter.h" |
| 267 #include "third_party/skia/include/core/SkMatrix.h" | 255 #include "third_party/skia/include/core/SkMatrix.h" |
| 268 @@ -902,243 +901,6 @@ TEST_F(GLRendererPixelTest, NonPremultipliedTextureWithBac
kground) { | 256 @@ -907,267 +906,6 @@ TEST_F(GLRendererPixelTest, NonPremultipliedTextureWithBac
kground) { |
| 269 FuzzyPixelOffByOneComparator(true))); | 257 FuzzyPixelOffByOneComparator(true))); |
| 270 } | 258 } |
| 271 | 259 |
| 272 -class VideoGLRendererPixelTest : public GLRendererPixelTest { | 260 -class VideoGLRendererPixelTest : public GLRendererPixelTest { |
| 273 - protected: | 261 - protected: |
| 274 - void CreateEdgeBleedPass(media::VideoFrame::Format format, | 262 - void CreateEdgeBleedPass(media::VideoFrame::Format format, |
| 275 - RenderPassList* pass_list) { | 263 - RenderPassList* pass_list) { |
| 276 - gfx::Rect rect(200, 200); | 264 - gfx::Rect rect(200, 200); |
| 277 - | 265 - |
| 278 - RenderPassId id(1, 1); | 266 - RenderPassId id(1, 1); |
| (...skipping 14 matching lines...) Expand all Loading... |
| 293 - static_cast<float>(green_rect.y()) / background_size.height(), | 281 - static_cast<float>(green_rect.y()) / background_size.height(), |
| 294 - static_cast<float>(green_rect.width()) / background_size.width(), | 282 - static_cast<float>(green_rect.width()) / background_size.width(), |
| 295 - static_cast<float>(green_rect.height()) / background_size.height()); | 283 - static_cast<float>(green_rect.height()) / background_size.height()); |
| 296 - | 284 - |
| 297 - // YUV of (149,43,21) should be green (0,255,0) in RGB. | 285 - // YUV of (149,43,21) should be green (0,255,0) in RGB. |
| 298 - // Create a video frame that has a non-green background rect, with a | 286 - // Create a video frame that has a non-green background rect, with a |
| 299 - // green sub-rectangle that should be the only thing displayed in | 287 - // green sub-rectangle that should be the only thing displayed in |
| 300 - // the final image. Bleeding will appear on all four sides of the video | 288 - // the final image. Bleeding will appear on all four sides of the video |
| 301 - // if the tex coords are not clamped. | 289 - // if the tex coords are not clamped. |
| 302 - CreateTestYUVVideoDrawQuad_TwoColor( | 290 - CreateTestYUVVideoDrawQuad_TwoColor( |
| 303 - shared_state, format, false, tex_coord_rect, background_size, 0, 0, 0, | 291 - shared_state, format, false, tex_coord_rect, background_size, |
| 304 - green_rect, 149, 43, 21, pass.get(), video_resource_updater_.get(), | 292 - gfx::Rect(background_size), 0, 0, 0, green_rect, 149, 43, 21, |
| 305 - resource_provider_.get()); | 293 - pass.get(), video_resource_updater_.get(), resource_provider_.get()); |
| 306 - pass_list->push_back(pass.Pass()); | 294 - pass_list->push_back(pass.Pass()); |
| 307 - } | 295 - } |
| 308 - | 296 - |
| 309 - void SetUp() override { | 297 - void SetUp() override { |
| 310 - GLRendererPixelTest::SetUp(); | 298 - GLRendererPixelTest::SetUp(); |
| 311 - video_resource_updater_.reset(new VideoResourceUpdater( | 299 - video_resource_updater_.reset(new VideoResourceUpdater( |
| 312 - output_surface_->context_provider(), resource_provider_.get())); | 300 - output_surface_->context_provider(), resource_provider_.get())); |
| 313 - } | 301 - } |
| 314 - | 302 - |
| 315 - scoped_ptr<VideoResourceUpdater> video_resource_updater_; | 303 - scoped_ptr<VideoResourceUpdater> video_resource_updater_; |
| 316 -}; | 304 -}; |
| 317 - | 305 - |
| 318 -TEST_F(VideoGLRendererPixelTest, SimpleYUVRect) { | 306 -TEST_F(VideoGLRendererPixelTest, SimpleYUVRect) { |
| 319 - gfx::Rect rect(this->device_viewport_size_); | 307 - gfx::Rect rect(this->device_viewport_size_); |
| 320 - | 308 - |
| 321 - RenderPassId id(1, 1); | 309 - RenderPassId id(1, 1); |
| 322 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); | 310 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); |
| 323 - | 311 - |
| 324 - SharedQuadState* shared_state = | 312 - SharedQuadState* shared_state = |
| 325 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); | 313 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); |
| 326 - | 314 - |
| 327 - CreateTestYUVVideoDrawQuad_Striped(shared_state, media::VideoFrame::YV12, | 315 - CreateTestYUVVideoDrawQuad_Striped(shared_state, media::VideoFrame::YV12, |
| 328 - false, gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), | 316 - false, gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), |
| 329 - pass.get(), video_resource_updater_.get(), | 317 - pass.get(), video_resource_updater_.get(), |
| 330 - rect, resource_provider_.get()); | 318 - rect, rect, resource_provider_.get()); |
| 331 - | 319 - |
| 332 - RenderPassList pass_list; | 320 - RenderPassList pass_list; |
| 333 - pass_list.push_back(pass.Pass()); | 321 - pass_list.push_back(pass.Pass()); |
| 334 - | 322 - |
| 335 - EXPECT_TRUE( | 323 - EXPECT_TRUE( |
| 336 - this->RunPixelTest(&pass_list, | 324 - this->RunPixelTest(&pass_list, |
| 337 - base::FilePath(FILE_PATH_LITERAL("yuv_stripes.png")), | 325 - base::FilePath(FILE_PATH_LITERAL("yuv_stripes.png")), |
| 338 - FuzzyPixelOffByOneComparator(true))); | 326 - FuzzyPixelOffByOneComparator(true))); |
| 339 -} | 327 -} |
| 340 - | 328 - |
| 329 -TEST_F(VideoGLRendererPixelTest, ClippedYUVRect) { |
| 330 - gfx::Rect viewport(this->device_viewport_size_); |
| 331 - gfx::Rect draw_rect(this->device_viewport_size_.width() * 1.5, |
| 332 - this->device_viewport_size_.height() * 1.5); |
| 333 - |
| 334 - RenderPassId id(1, 1); |
| 335 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, viewport); |
| 336 - |
| 337 - SharedQuadState* shared_state = |
| 338 - CreateTestSharedQuadState(gfx::Transform(), viewport, pass.get()); |
| 339 - |
| 340 - CreateTestYUVVideoDrawQuad_Striped(shared_state, media::VideoFrame::YV12, |
| 341 - false, gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), |
| 342 - pass.get(), video_resource_updater_.get(), |
| 343 - draw_rect, viewport, |
| 344 - resource_provider_.get()); |
| 345 - RenderPassList pass_list; |
| 346 - pass_list.push_back(pass.Pass()); |
| 347 - |
| 348 - EXPECT_TRUE(this->RunPixelTest( |
| 349 - &pass_list, base::FilePath(FILE_PATH_LITERAL("yuv_stripes_clipped.png")), |
| 350 - FuzzyPixelOffByOneComparator(true))); |
| 351 -} |
| 352 - |
| 341 -TEST_F(VideoGLRendererPixelTest, OffsetYUVRect) { | 353 -TEST_F(VideoGLRendererPixelTest, OffsetYUVRect) { |
| 342 - gfx::Rect rect(this->device_viewport_size_); | 354 - gfx::Rect rect(this->device_viewport_size_); |
| 343 - | 355 - |
| 344 - RenderPassId id(1, 1); | 356 - RenderPassId id(1, 1); |
| 345 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); | 357 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); |
| 346 - | 358 - |
| 347 - SharedQuadState* shared_state = | 359 - SharedQuadState* shared_state = |
| 348 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); | 360 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); |
| 349 - | 361 - |
| 350 - // Intentionally sets frame format to I420 for testing coverage. | 362 - // Intentionally sets frame format to I420 for testing coverage. |
| 351 - CreateTestYUVVideoDrawQuad_Striped( | 363 - CreateTestYUVVideoDrawQuad_Striped( |
| 352 - shared_state, media::VideoFrame::I420, false, | 364 - shared_state, media::VideoFrame::I420, false, |
| 353 - gfx::RectF(0.125f, 0.25f, 0.75f, 0.5f), pass.get(), | 365 - gfx::RectF(0.125f, 0.25f, 0.75f, 0.5f), pass.get(), |
| 354 - video_resource_updater_.get(), rect, resource_provider_.get()); | 366 - video_resource_updater_.get(), rect, rect, resource_provider_.get()); |
| 355 - | 367 - |
| 356 - RenderPassList pass_list; | 368 - RenderPassList pass_list; |
| 357 - pass_list.push_back(pass.Pass()); | 369 - pass_list.push_back(pass.Pass()); |
| 358 - | 370 - |
| 359 - EXPECT_TRUE(this->RunPixelTest( | 371 - EXPECT_TRUE(this->RunPixelTest( |
| 360 - &pass_list, | 372 - &pass_list, |
| 361 - base::FilePath(FILE_PATH_LITERAL("yuv_stripes_offset.png")), | 373 - base::FilePath(FILE_PATH_LITERAL("yuv_stripes_offset.png")), |
| 362 - FuzzyPixelOffByOneComparator(true))); | 374 - FuzzyPixelOffByOneComparator(true))); |
| 363 -} | 375 -} |
| 364 - | 376 - |
| 365 -TEST_F(VideoGLRendererPixelTest, SimpleYUVRectBlack) { | 377 -TEST_F(VideoGLRendererPixelTest, SimpleYUVRectBlack) { |
| 366 - gfx::Rect rect(this->device_viewport_size_); | 378 - gfx::Rect rect(this->device_viewport_size_); |
| 367 - | 379 - |
| 368 - RenderPassId id(1, 1); | 380 - RenderPassId id(1, 1); |
| 369 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); | 381 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); |
| 370 - | 382 - |
| 371 - SharedQuadState* shared_state = | 383 - SharedQuadState* shared_state = |
| 372 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); | 384 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); |
| 373 - | 385 - |
| 374 - // In MPEG color range YUV values of (15,128,128) should produce black. | 386 - // In MPEG color range YUV values of (15,128,128) should produce black. |
| 375 - CreateTestYUVVideoDrawQuad_Solid( | 387 - CreateTestYUVVideoDrawQuad_Solid( |
| 376 - shared_state, media::VideoFrame::YV12, false, | 388 - shared_state, media::VideoFrame::YV12, false, |
| 377 - gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), 15, 128, 128, pass.get(), | 389 - gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), 15, 128, 128, pass.get(), |
| 378 - video_resource_updater_.get(), rect, resource_provider_.get()); | 390 - video_resource_updater_.get(), rect, rect, resource_provider_.get()); |
| 379 - | 391 - |
| 380 - RenderPassList pass_list; | 392 - RenderPassList pass_list; |
| 381 - pass_list.push_back(pass.Pass()); | 393 - pass_list.push_back(pass.Pass()); |
| 382 - | 394 - |
| 383 - // If we didn't get black out of the YUV values above, then we probably have
a | 395 - // If we didn't get black out of the YUV values above, then we probably have
a |
| 384 - // color range issue. | 396 - // color range issue. |
| 385 - EXPECT_TRUE(this->RunPixelTest(&pass_list, | 397 - EXPECT_TRUE(this->RunPixelTest(&pass_list, |
| 386 - base::FilePath(FILE_PATH_LITERAL("black.png"))
, | 398 - base::FilePath(FILE_PATH_LITERAL("black.png"))
, |
| 387 - FuzzyPixelOffByOneComparator(true))); | 399 - FuzzyPixelOffByOneComparator(true))); |
| 388 -} | 400 -} |
| 389 - | 401 - |
| 390 -TEST_F(VideoGLRendererPixelTest, SimpleYUVJRect) { | 402 -TEST_F(VideoGLRendererPixelTest, SimpleYUVJRect) { |
| 391 - gfx::Rect rect(this->device_viewport_size_); | 403 - gfx::Rect rect(this->device_viewport_size_); |
| 392 - | 404 - |
| 393 - RenderPassId id(1, 1); | 405 - RenderPassId id(1, 1); |
| 394 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); | 406 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); |
| 395 - | 407 - |
| 396 - SharedQuadState* shared_state = | 408 - SharedQuadState* shared_state = |
| 397 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); | 409 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); |
| 398 - | 410 - |
| 399 - // YUV of (149,43,21) should be green (0,255,0) in RGB. | 411 - // YUV of (149,43,21) should be green (0,255,0) in RGB. |
| 400 - CreateTestYUVVideoDrawQuad_Solid( | 412 - CreateTestYUVVideoDrawQuad_Solid( |
| 401 - shared_state, media::VideoFrame::YV12J, false, | 413 - shared_state, media::VideoFrame::YV12J, false, |
| 402 - gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), 149, 43, 21, pass.get(), | 414 - gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), 149, 43, 21, pass.get(), |
| 403 - video_resource_updater_.get(), rect, resource_provider_.get()); | 415 - video_resource_updater_.get(), rect, rect, resource_provider_.get()); |
| 404 - | 416 - |
| 405 - RenderPassList pass_list; | 417 - RenderPassList pass_list; |
| 406 - pass_list.push_back(pass.Pass()); | 418 - pass_list.push_back(pass.Pass()); |
| 407 - | 419 - |
| 408 - EXPECT_TRUE(this->RunPixelTest(&pass_list, | 420 - EXPECT_TRUE(this->RunPixelTest(&pass_list, |
| 409 - base::FilePath(FILE_PATH_LITERAL("green.png"))
, | 421 - base::FilePath(FILE_PATH_LITERAL("green.png"))
, |
| 410 - FuzzyPixelOffByOneComparator(true))); | 422 - FuzzyPixelOffByOneComparator(true))); |
| 411 -} | 423 -} |
| 412 - | 424 - |
| 413 -// Test that a YUV video doesn't bleed outside of its tex coords when the | 425 -// Test that a YUV video doesn't bleed outside of its tex coords when the |
| (...skipping 20 matching lines...) Expand all Loading... |
| 434 - RenderPassId id(1, 1); | 446 - RenderPassId id(1, 1); |
| 435 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); | 447 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); |
| 436 - | 448 - |
| 437 - SharedQuadState* shared_state = | 449 - SharedQuadState* shared_state = |
| 438 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); | 450 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); |
| 439 - | 451 - |
| 440 - // Dark grey in JPEG color range (in MPEG, this is black). | 452 - // Dark grey in JPEG color range (in MPEG, this is black). |
| 441 - CreateTestYUVVideoDrawQuad_Solid( | 453 - CreateTestYUVVideoDrawQuad_Solid( |
| 442 - shared_state, media::VideoFrame::YV12J, false, | 454 - shared_state, media::VideoFrame::YV12J, false, |
| 443 - gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), 15, 128, 128, pass.get(), | 455 - gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), 15, 128, 128, pass.get(), |
| 444 - video_resource_updater_.get(), rect, resource_provider_.get()); | 456 - video_resource_updater_.get(), rect, rect, resource_provider_.get()); |
| 445 - | 457 - |
| 446 - RenderPassList pass_list; | 458 - RenderPassList pass_list; |
| 447 - pass_list.push_back(pass.Pass()); | 459 - pass_list.push_back(pass.Pass()); |
| 448 - | 460 - |
| 449 - EXPECT_TRUE( | 461 - EXPECT_TRUE( |
| 450 - this->RunPixelTest(&pass_list, | 462 - this->RunPixelTest(&pass_list, |
| 451 - base::FilePath(FILE_PATH_LITERAL("dark_grey.png")), | 463 - base::FilePath(FILE_PATH_LITERAL("dark_grey.png")), |
| 452 - FuzzyPixelOffByOneComparator(true))); | 464 - FuzzyPixelOffByOneComparator(true))); |
| 453 -} | 465 -} |
| 454 - | 466 - |
| 455 -TEST_F(VideoGLRendererPixelTest, SimpleYUVARect) { | 467 -TEST_F(VideoGLRendererPixelTest, SimpleYUVARect) { |
| 456 - gfx::Rect rect(this->device_viewport_size_); | 468 - gfx::Rect rect(this->device_viewport_size_); |
| 457 - | 469 - |
| 458 - RenderPassId id(1, 1); | 470 - RenderPassId id(1, 1); |
| 459 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); | 471 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); |
| 460 - | 472 - |
| 461 - SharedQuadState* shared_state = | 473 - SharedQuadState* shared_state = |
| 462 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); | 474 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); |
| 463 - | 475 - |
| 464 - CreateTestYUVVideoDrawQuad_Striped(shared_state, media::VideoFrame::YV12A, | 476 - CreateTestYUVVideoDrawQuad_Striped(shared_state, media::VideoFrame::YV12A, |
| 465 - false, gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), | 477 - false, gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), |
| 466 - pass.get(), video_resource_updater_.get(), | 478 - pass.get(), video_resource_updater_.get(), |
| 467 - rect, resource_provider_.get()); | 479 - rect, rect, resource_provider_.get()); |
| 468 - | 480 - |
| 469 - SolidColorDrawQuad* color_quad = | 481 - SolidColorDrawQuad* color_quad = |
| 470 - pass->CreateAndAppendDrawQuad<SolidColorDrawQuad>(); | 482 - pass->CreateAndAppendDrawQuad<SolidColorDrawQuad>(); |
| 471 - color_quad->SetNew(shared_state, rect, rect, SK_ColorWHITE, false); | 483 - color_quad->SetNew(shared_state, rect, rect, SK_ColorWHITE, false); |
| 472 - | 484 - |
| 473 - RenderPassList pass_list; | 485 - RenderPassList pass_list; |
| 474 - pass_list.push_back(pass.Pass()); | 486 - pass_list.push_back(pass.Pass()); |
| 475 - | 487 - |
| 476 - EXPECT_TRUE(this->RunPixelTest( | 488 - EXPECT_TRUE(this->RunPixelTest( |
| 477 - &pass_list, | 489 - &pass_list, |
| 478 - base::FilePath(FILE_PATH_LITERAL("yuv_stripes_alpha.png")), | 490 - base::FilePath(FILE_PATH_LITERAL("yuv_stripes_alpha.png")), |
| 479 - FuzzyPixelOffByOneComparator(true))); | 491 - FuzzyPixelOffByOneComparator(true))); |
| 480 -} | 492 -} |
| 481 - | 493 - |
| 482 -TEST_F(VideoGLRendererPixelTest, FullyTransparentYUVARect) { | 494 -TEST_F(VideoGLRendererPixelTest, FullyTransparentYUVARect) { |
| 483 - gfx::Rect rect(this->device_viewport_size_); | 495 - gfx::Rect rect(this->device_viewport_size_); |
| 484 - | 496 - |
| 485 - RenderPassId id(1, 1); | 497 - RenderPassId id(1, 1); |
| 486 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); | 498 - scoped_ptr<RenderPass> pass = CreateTestRootRenderPass(id, rect); |
| 487 - | 499 - |
| 488 - SharedQuadState* shared_state = | 500 - SharedQuadState* shared_state = |
| 489 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); | 501 - CreateTestSharedQuadState(gfx::Transform(), rect, pass.get()); |
| 490 - | 502 - |
| 491 - CreateTestYUVVideoDrawQuad_Striped(shared_state, media::VideoFrame::YV12A, | 503 - CreateTestYUVVideoDrawQuad_Striped(shared_state, media::VideoFrame::YV12A, |
| 492 - true, gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), | 504 - true, gfx::RectF(0.0f, 0.0f, 1.0f, 1.0f), |
| 493 - pass.get(), video_resource_updater_.get(), | 505 - pass.get(), video_resource_updater_.get(), |
| 494 - rect, resource_provider_.get()); | 506 - rect, rect, resource_provider_.get()); |
| 495 - | 507 - |
| 496 - SolidColorDrawQuad* color_quad = | 508 - SolidColorDrawQuad* color_quad = |
| 497 - pass->CreateAndAppendDrawQuad<SolidColorDrawQuad>(); | 509 - pass->CreateAndAppendDrawQuad<SolidColorDrawQuad>(); |
| 498 - color_quad->SetNew(shared_state, rect, rect, SK_ColorBLACK, false); | 510 - color_quad->SetNew(shared_state, rect, rect, SK_ColorBLACK, false); |
| 499 - | 511 - |
| 500 - RenderPassList pass_list; | 512 - RenderPassList pass_list; |
| 501 - pass_list.push_back(pass.Pass()); | 513 - pass_list.push_back(pass.Pass()); |
| 502 - | 514 - |
| 503 - EXPECT_TRUE(this->RunPixelTest( | 515 - EXPECT_TRUE(this->RunPixelTest( |
| 504 - &pass_list, | 516 - &pass_list, |
| 505 - base::FilePath(FILE_PATH_LITERAL("black.png")), | 517 - base::FilePath(FILE_PATH_LITERAL("black.png")), |
| 506 - ExactPixelComparator(true))); | 518 - ExactPixelComparator(true))); |
| 507 -} | 519 -} |
| 508 - | 520 - |
| 509 TYPED_TEST(RendererPixelTest, FastPassColorFilterAlpha) { | 521 TYPED_TEST(RendererPixelTest, FastPassColorFilterAlpha) { |
| 510 gfx::Rect viewport_rect(this->device_viewport_size_); | 522 gfx::Rect viewport_rect(this->device_viewport_size_); |
| 511 | 523 |
| 512 diff --git a/cc/quads/yuv_video_draw_quad.h b/cc/quads/yuv_video_draw_quad.h | |
| 513 index 358929e..15bce98 100644 | |
| 514 --- a/cc/quads/yuv_video_draw_quad.h | |
| 515 +++ b/cc/quads/yuv_video_draw_quad.h | |
| 516 @@ -8,7 +8,6 @@ | |
| 517 #include "base/basictypes.h" | |
| 518 #include "base/memory/scoped_ptr.h" | |
| 519 #include "cc/base/cc_export.h" | |
| 520 -#include "cc/layers/video_layer_impl.h" | |
| 521 #include "cc/quads/draw_quad.h" | |
| 522 | |
| 523 namespace cc { | |
| 524 diff --git a/cc/resources/drawing_display_item.cc b/cc/resources/drawing_display
_item.cc | 524 diff --git a/cc/resources/drawing_display_item.cc b/cc/resources/drawing_display
_item.cc |
| 525 index 648f9de..6dffad9 100644 | 525 index 648f9de..6dffad9 100644 |
| 526 --- a/cc/resources/drawing_display_item.cc | 526 --- a/cc/resources/drawing_display_item.cc |
| 527 +++ b/cc/resources/drawing_display_item.cc | 527 +++ b/cc/resources/drawing_display_item.cc |
| 528 @@ -6,6 +6,7 @@ | 528 @@ -6,6 +6,7 @@ |
| 529 | 529 |
| 530 #include <string> | 530 #include <string> |
| 531 | 531 |
| 532 +#include "base/logging.h" | 532 +#include "base/logging.h" |
| 533 #include "base/strings/stringprintf.h" | 533 #include "base/strings/stringprintf.h" |
| 534 #include "base/trace_event/trace_event_argument.h" | 534 #include "base/trace_event/trace_event_argument.h" |
| 535 #include "cc/debug/picture_debug_util.h" | 535 #include "cc/debug/picture_debug_util.h" |
| 536 diff --git a/cc/trees/layer_tree_host_impl_unittest.cc b/cc/trees/layer_tree_hos
t_impl_unittest.cc | 536 diff --git a/cc/trees/layer_tree_host_impl_unittest.cc b/cc/trees/layer_tree_hos
t_impl_unittest.cc |
| 537 index 4289a99..4e3230b 100644 | 537 index 8056387..e8471e2 100644 |
| 538 --- a/cc/trees/layer_tree_host_impl_unittest.cc | 538 --- a/cc/trees/layer_tree_host_impl_unittest.cc |
| 539 +++ b/cc/trees/layer_tree_host_impl_unittest.cc | 539 +++ b/cc/trees/layer_tree_host_impl_unittest.cc |
| 540 @@ -27,7 +27,6 @@ | 540 @@ -26,7 +26,6 @@ |
| 541 #include "cc/layers/solid_color_scrollbar_layer_impl.h" | 541 #include "cc/layers/solid_color_scrollbar_layer_impl.h" |
| 542 #include "cc/layers/texture_layer_impl.h" | 542 #include "cc/layers/texture_layer_impl.h" |
| 543 #include "cc/layers/tiled_layer_impl.h" | 543 #include "cc/layers/tiled_layer_impl.h" |
| 544 -#include "cc/layers/video_layer_impl.h" | 544 -#include "cc/layers/video_layer_impl.h" |
| 545 #include "cc/output/begin_frame_args.h" | 545 #include "cc/output/begin_frame_args.h" |
| 546 #include "cc/output/compositor_frame_ack.h" | 546 #include "cc/output/compositor_frame_ack.h" |
| 547 #include "cc/output/compositor_frame_metadata.h" | 547 #include "cc/output/compositor_frame_metadata.h" |
| 548 @@ -47,7 +46,6 @@ | 548 @@ -47,7 +46,6 @@ |
| 549 #include "cc/test/fake_picture_layer_impl.h" | 549 #include "cc/test/fake_picture_layer_impl.h" |
| 550 #include "cc/test/fake_picture_pile_impl.h" | 550 #include "cc/test/fake_picture_pile_impl.h" |
| (...skipping 56 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 607 - video_layer->SetBounds(gfx::Size(10, 10)); | 607 - video_layer->SetBounds(gfx::Size(10, 10)); |
| 608 - video_layer->SetContentBounds(gfx::Size(10, 10)); | 608 - video_layer->SetContentBounds(gfx::Size(10, 10)); |
| 609 - video_layer->SetDrawsContent(true); | 609 - video_layer->SetDrawsContent(true); |
| 610 - root_layer->AddChild(video_layer.Pass()); | 610 - root_layer->AddChild(video_layer.Pass()); |
| 611 - SetupRootLayerImpl(root_layer.Pass()); | 611 - SetupRootLayerImpl(root_layer.Pass()); |
| 612 - | 612 - |
| 613 LayerTreeHostImpl::FrameData frame; | 613 LayerTreeHostImpl::FrameData frame; |
| 614 EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame)); | 614 EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame)); |
| 615 host_impl_->DrawLayers(&frame, gfx::FrameTime::Now()); | 615 host_impl_->DrawLayers(&frame, gfx::FrameTime::Now()); |
| 616 diff --git a/cc/trees/layer_tree_host_unittest.cc b/cc/trees/layer_tree_host_uni
ttest.cc | 616 diff --git a/cc/trees/layer_tree_host_unittest.cc b/cc/trees/layer_tree_host_uni
ttest.cc |
| 617 index 68a1f58..2ec6e66 100644 | 617 index aacabd7..9416474 100644 |
| 618 --- a/cc/trees/layer_tree_host_unittest.cc | 618 --- a/cc/trees/layer_tree_host_unittest.cc |
| 619 +++ b/cc/trees/layer_tree_host_unittest.cc | 619 +++ b/cc/trees/layer_tree_host_unittest.cc |
| 620 @@ -18,7 +18,6 @@ | 620 @@ -17,7 +17,6 @@ |
| 621 #include "cc/layers/painted_scrollbar_layer.h" | 621 #include "cc/layers/painted_scrollbar_layer.h" |
| 622 #include "cc/layers/picture_layer.h" | 622 #include "cc/layers/picture_layer.h" |
| 623 #include "cc/layers/solid_color_layer.h" | 623 #include "cc/layers/solid_color_layer.h" |
| 624 -#include "cc/layers/video_layer.h" | 624 -#include "cc/layers/video_layer.h" |
| 625 #include "cc/output/begin_frame_args.h" | 625 #include "cc/output/begin_frame_args.h" |
| 626 #include "cc/output/compositor_frame_ack.h" | 626 #include "cc/output/compositor_frame_ack.h" |
| 627 #include "cc/output/copy_output_request.h" | 627 #include "cc/output/copy_output_request.h" |
| 628 @@ -41,7 +40,6 @@ | 628 @@ -41,7 +40,6 @@ |
| 629 #include "cc/test/fake_picture_pile.h" | 629 #include "cc/test/fake_picture_pile.h" |
| 630 #include "cc/test/fake_proxy.h" | 630 #include "cc/test/fake_proxy.h" |
| 631 #include "cc/test/fake_scoped_ui_resource.h" | 631 #include "cc/test/fake_scoped_ui_resource.h" |
| 632 -#include "cc/test/fake_video_frame_provider.h" | 632 -#include "cc/test/fake_video_frame_provider.h" |
| 633 #include "cc/test/geometry_test_utils.h" | 633 #include "cc/test/geometry_test_utils.h" |
| 634 #include "cc/test/impl_side_painting_settings.h" | 634 #include "cc/test/impl_side_painting_settings.h" |
| 635 #include "cc/test/layer_tree_test.h" | 635 #include "cc/test/layer_tree_test.h" |
| 636 @@ -4170,28 +4168,6 @@ class LayerInvalidateCausesDraw : public LayerTreeHostTes
t { | 636 @@ -4256,28 +4254,6 @@ class LayerInvalidateCausesDraw : public LayerTreeHostTes
t { |
| 637 int num_draws_; | 637 int num_draws_; |
| 638 }; | 638 }; |
| 639 | 639 |
| 640 -// VideoLayer must support being invalidated and then passing that along | 640 -// VideoLayer must support being invalidated and then passing that along |
| 641 -// to the compositor thread, even though no resources are updated in | 641 -// to the compositor thread, even though no resources are updated in |
| 642 -// response to that invalidation. | 642 -// response to that invalidation. |
| 643 -class LayerTreeHostTestVideoLayerInvalidate : public LayerInvalidateCausesDraw
{ | 643 -class LayerTreeHostTestVideoLayerInvalidate : public LayerInvalidateCausesDraw
{ |
| 644 - public: | 644 - public: |
| 645 - void SetupTree() override { | 645 - void SetupTree() override { |
| 646 - LayerTreeHostTest::SetupTree(); | 646 - LayerTreeHostTest::SetupTree(); |
| (...skipping 119 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 766 - scoped_refptr<VideoFrame> color_video_frame_; | 766 - scoped_refptr<VideoFrame> color_video_frame_; |
| 767 - scoped_refptr<VideoFrame> hw_video_frame_; | 767 - scoped_refptr<VideoFrame> hw_video_frame_; |
| 768 - scoped_refptr<VideoFrame> scaled_hw_video_frame_; | 768 - scoped_refptr<VideoFrame> scaled_hw_video_frame_; |
| 769 - | 769 - |
| 770 - FakeVideoFrameProvider color_frame_provider_; | 770 - FakeVideoFrameProvider color_frame_provider_; |
| 771 - FakeVideoFrameProvider hw_frame_provider_; | 771 - FakeVideoFrameProvider hw_frame_provider_; |
| 772 - FakeVideoFrameProvider scaled_hw_frame_provider_; | 772 - FakeVideoFrameProvider scaled_hw_frame_provider_; |
| 773 }; | 773 }; |
| 774 | 774 |
| 775 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostContextTestDontUseLostResources); | 775 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostContextTestDontUseLostResources); |
| OLD | NEW |