| 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 c744c5b..16a1589 100644 | 2 index 826e12c..13e974d 100644 |
| 3 --- a/cc/BUILD.gn | 3 --- a/cc/BUILD.gn |
| 4 +++ b/cc/BUILD.gn | 4 +++ b/cc/BUILD.gn |
| 5 @@ -212,13 +212,6 @@ component("cc") { | 5 @@ -213,13 +213,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 @@ -453,8 +446,6 @@ component("cc") { | 19 @@ -456,8 +449,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 @@ -588,8 +579,6 @@ source_set("test_support") { | 28 @@ -590,8 +581,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/failure_output_surface.cc", |
| 35 "test/failure_output_surface.h", |
| 34 "test/geometry_test_utils.cc", | 36 "test/geometry_test_utils.cc", |
| 35 "test/geometry_test_utils.h", | 37 @@ -747,7 +736,6 @@ test("cc_unittests") { |
| 36 "test/test_in_process_context_provider.cc", | |
| 37 @@ -744,7 +733,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 @@ -775,7 +763,6 @@ test("cc_unittests") { | 45 @@ -778,7 +766,6 @@ test("cc_unittests") { |
| 46 "resources/texture_uploader_unittest.cc", | 46 "resources/texture_uploader_unittest.cc", |
| 47 "resources/tile_manager_unittest.cc", | 47 "resources/tile_manager_unittest.cc", |
| 48 "resources/tile_priority_unittest.cc", | 48 "resources/tile_priority_unittest.cc", |
| 49 - "resources/video_resource_updater_unittest.cc", | 49 - "resources/video_resource_updater_unittest.cc", |
| 50 "scheduler/begin_frame_source_unittest.cc", | 50 "scheduler/begin_frame_source_unittest.cc", |
| 51 "scheduler/delay_based_time_source_unittest.cc", | 51 "scheduler/delay_based_time_source_unittest.cc", |
| 52 "scheduler/scheduler_state_machine_unittest.cc", | 52 "scheduler/scheduler_state_machine_unittest.cc", |
| 53 @@ -804,7 +791,6 @@ test("cc_unittests") { | 53 @@ -807,7 +794,6 @@ test("cc_unittests") { |
| 54 "trees/layer_tree_host_unittest_picture.cc", | 54 "trees/layer_tree_host_unittest_picture.cc", |
| 55 "trees/layer_tree_host_unittest_proxy.cc", | 55 "trees/layer_tree_host_unittest_proxy.cc", |
| 56 "trees/layer_tree_host_unittest_scroll.cc", | 56 "trees/layer_tree_host_unittest_scroll.cc", |
| 57 - "trees/layer_tree_host_unittest_video.cc", | 57 - "trees/layer_tree_host_unittest_video.cc", |
| 58 "trees/layer_tree_impl_unittest.cc", | 58 "trees/layer_tree_impl_unittest.cc", |
| 59 "trees/occlusion_tracker_unittest.cc", | 59 "trees/occlusion_tracker_unittest.cc", |
| 60 "trees/occlusion_unittest.cc", | 60 "trees/occlusion_unittest.cc", |
| 61 diff --git a/cc/layers/video_frame_provider.h b/cc/layers/video_frame_provider.h | 61 diff --git a/cc/layers/video_frame_provider.h b/cc/layers/video_frame_provider.h |
| 62 deleted file mode 100644 | 62 deleted file mode 100644 |
| 63 index 784d951..0000000 | 63 index 784d951..0000000 |
| (...skipping 158 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 222 - matrix[0], matrix[4], matrix[8], matrix[12], | 222 - matrix[0], matrix[4], matrix[8], matrix[12], |
| 223 - matrix[1], matrix[5], matrix[9], matrix[13], | 223 - matrix[1], matrix[5], matrix[9], matrix[13], |
| 224 - matrix[2], matrix[6], matrix[10], matrix[14], | 224 - matrix[2], matrix[6], matrix[10], matrix[14], |
| 225 - matrix[3], matrix[7], matrix[11], matrix[15]); | 225 - matrix[3], matrix[7], matrix[11], matrix[15]); |
| 226 - if (active_video_layer_) | 226 - if (active_video_layer_) |
| 227 - active_video_layer_->SetNeedsRedraw(); | 227 - active_video_layer_->SetNeedsRedraw(); |
| 228 -} | 228 -} |
| 229 - | 229 - |
| 230 -} // namespace cc | 230 -} // namespace cc |
| 231 diff --git a/cc/output/gl_renderer.cc b/cc/output/gl_renderer.cc | 231 diff --git a/cc/output/gl_renderer.cc b/cc/output/gl_renderer.cc |
| 232 index 27f40eb..49af1c3 100644 | 232 index ee94a4c..7896bb8 100644 |
| 233 --- a/cc/output/gl_renderer.cc | 233 --- a/cc/output/gl_renderer.cc |
| 234 +++ b/cc/output/gl_renderer.cc | 234 +++ b/cc/output/gl_renderer.cc |
| 235 @@ -13,7 +13,6 @@ | 235 @@ -13,7 +13,6 @@ |
| 236 #include "base/debug/trace_event.h" | 236 #include "base/debug/trace_event.h" |
| 237 #include "base/logging.h" | 237 #include "base/logging.h" |
| 238 #include "cc/base/math_util.h" | 238 #include "cc/base/math_util.h" |
| 239 -#include "cc/layers/video_layer_impl.h" | 239 -#include "cc/layers/video_layer_impl.h" |
| 240 #include "cc/output/compositor_frame.h" | 240 #include "cc/output/compositor_frame.h" |
| 241 #include "cc/output/compositor_frame_metadata.h" | 241 #include "cc/output/compositor_frame_metadata.h" |
| 242 #include "cc/output/context_provider.h" | 242 #include "cc/output/context_provider.h" |
| 243 diff --git a/cc/output/renderer_pixeltest.cc b/cc/output/renderer_pixeltest.cc | 243 diff --git a/cc/output/renderer_pixeltest.cc b/cc/output/renderer_pixeltest.cc |
| 244 index 0c2bc3d..d112ed1 100644 | 244 index b51aef6..58205a9 100644 |
| 245 --- a/cc/output/renderer_pixeltest.cc | 245 --- a/cc/output/renderer_pixeltest.cc |
| 246 +++ b/cc/output/renderer_pixeltest.cc | 246 +++ b/cc/output/renderer_pixeltest.cc |
| 247 @@ -12,7 +12,6 @@ | 247 @@ -12,7 +12,6 @@ |
| 248 #include "cc/test/fake_picture_pile_impl.h" | 248 #include "cc/test/fake_picture_pile_impl.h" |
| 249 #include "cc/test/pixel_test.h" | 249 #include "cc/test/pixel_test.h" |
| 250 #include "gpu/command_buffer/client/gles2_interface.h" | 250 #include "gpu/command_buffer/client/gles2_interface.h" |
| 251 -#include "media/base/video_frame.h" | 251 -#include "media/base/video_frame.h" |
| 252 #include "third_party/skia/include/core/SkColorPriv.h" | 252 #include "third_party/skia/include/core/SkColorPriv.h" |
| 253 #include "third_party/skia/include/core/SkImageFilter.h" | 253 #include "third_party/skia/include/core/SkImageFilter.h" |
| 254 #include "third_party/skia/include/core/SkMatrix.h" | 254 #include "third_party/skia/include/core/SkMatrix.h" |
| 255 @@ -385,346 +384,6 @@ TEST_F(GLRendererPixelTest, NonPremultipliedTextureWithBac
kground) { | 255 @@ -386,346 +385,6 @@ TEST_F(GLRendererPixelTest, NonPremultipliedTextureWithBac
kground) { |
| 256 FuzzyPixelOffByOneComparator(true))); | 256 FuzzyPixelOffByOneComparator(true))); |
| 257 } | 257 } |
| 258 | 258 |
| 259 -class VideoGLRendererPixelTest : public GLRendererPixelTest { | 259 -class VideoGLRendererPixelTest : public GLRendererPixelTest { |
| 260 - protected: | 260 - protected: |
| 261 - void CreateTestYUVVideoDrawQuad_Striped(const SharedQuadState* shared_state, | 261 - void CreateTestYUVVideoDrawQuad_Striped(const SharedQuadState* shared_state, |
| 262 - media::VideoFrame::Format format, | 262 - media::VideoFrame::Format format, |
| 263 - bool is_transparent, | 263 - bool is_transparent, |
| 264 - const gfx::RectF& tex_coord_rect, | 264 - const gfx::RectF& tex_coord_rect, |
| 265 - RenderPass* render_pass) { | 265 - RenderPass* render_pass) { |
| (...skipping 351 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 617 +++ b/cc/resources/drawing_display_item.cc | 617 +++ b/cc/resources/drawing_display_item.cc |
| 618 @@ -4,6 +4,7 @@ | 618 @@ -4,6 +4,7 @@ |
| 619 | 619 |
| 620 #include "cc/resources/drawing_display_item.h" | 620 #include "cc/resources/drawing_display_item.h" |
| 621 | 621 |
| 622 +#include "base/logging.h" | 622 +#include "base/logging.h" |
| 623 #include "third_party/skia/include/core/SkCanvas.h" | 623 #include "third_party/skia/include/core/SkCanvas.h" |
| 624 #include "third_party/skia/include/core/SkMatrix.h" | 624 #include "third_party/skia/include/core/SkMatrix.h" |
| 625 #include "third_party/skia/include/core/SkPicture.h" | 625 #include "third_party/skia/include/core/SkPicture.h" |
| 626 diff --git a/cc/trees/layer_tree_host_impl_unittest.cc b/cc/trees/layer_tree_hos
t_impl_unittest.cc | 626 diff --git a/cc/trees/layer_tree_host_impl_unittest.cc b/cc/trees/layer_tree_hos
t_impl_unittest.cc |
| 627 index f31cde0..4e070e2 100644 | 627 index fc4780f..61bda89 100644 |
| 628 --- a/cc/trees/layer_tree_host_impl_unittest.cc | 628 --- a/cc/trees/layer_tree_host_impl_unittest.cc |
| 629 +++ b/cc/trees/layer_tree_host_impl_unittest.cc | 629 +++ b/cc/trees/layer_tree_host_impl_unittest.cc |
| 630 @@ -27,7 +27,6 @@ | 630 @@ -27,7 +27,6 @@ |
| 631 #include "cc/layers/solid_color_scrollbar_layer_impl.h" | 631 #include "cc/layers/solid_color_scrollbar_layer_impl.h" |
| 632 #include "cc/layers/texture_layer_impl.h" | 632 #include "cc/layers/texture_layer_impl.h" |
| 633 #include "cc/layers/tiled_layer_impl.h" | 633 #include "cc/layers/tiled_layer_impl.h" |
| 634 -#include "cc/layers/video_layer_impl.h" | 634 -#include "cc/layers/video_layer_impl.h" |
| 635 #include "cc/output/begin_frame_args.h" | 635 #include "cc/output/begin_frame_args.h" |
| 636 #include "cc/output/compositor_frame_ack.h" | 636 #include "cc/output/compositor_frame_ack.h" |
| 637 #include "cc/output/compositor_frame_metadata.h" | 637 #include "cc/output/compositor_frame_metadata.h" |
| (...skipping 22 matching lines...) Expand all Loading... |
| 660 namespace cc { | 660 namespace cc { |
| 661 namespace { | 661 namespace { |
| 662 @@ -94,7 +90,6 @@ class LayerTreeHostImplTest : public testing::Test, | 662 @@ -94,7 +90,6 @@ class LayerTreeHostImplTest : public testing::Test, |
| 663 reduce_memory_result_(true), | 663 reduce_memory_result_(true), |
| 664 current_limit_bytes_(0), | 664 current_limit_bytes_(0), |
| 665 current_priority_cutoff_value_(0) { | 665 current_priority_cutoff_value_(0) { |
| 666 - media::InitializeMediaLibraryForTesting(); | 666 - media::InitializeMediaLibraryForTesting(); |
| 667 } | 667 } |
| 668 | 668 |
| 669 LayerTreeSettings DefaultSettings() { | 669 LayerTreeSettings DefaultSettings() { |
| 670 @@ -5247,18 +5242,6 @@ TEST_F(LayerTreeHostImplTest, LayersFreeTextures) { | 670 @@ -5248,18 +5243,6 @@ TEST_F(LayerTreeHostImplTest, LayersFreeTextures) { |
| 671 LayerImpl::Create(host_impl_->active_tree(), 1); | 671 LayerImpl::Create(host_impl_->active_tree(), 1); |
| 672 root_layer->SetBounds(gfx::Size(10, 10)); | 672 root_layer->SetBounds(gfx::Size(10, 10)); |
| 673 | 673 |
| 674 - scoped_refptr<VideoFrame> softwareFrame = | 674 - scoped_refptr<VideoFrame> softwareFrame = |
| 675 - media::VideoFrame::CreateColorFrame( | 675 - media::VideoFrame::CreateColorFrame( |
| 676 - gfx::Size(4, 4), 0x80, 0x80, 0x80, base::TimeDelta()); | 676 - gfx::Size(4, 4), 0x80, 0x80, 0x80, base::TimeDelta()); |
| 677 - FakeVideoFrameProvider provider; | 677 - FakeVideoFrameProvider provider; |
| 678 - provider.set_frame(softwareFrame); | 678 - provider.set_frame(softwareFrame); |
| 679 - scoped_ptr<VideoLayerImpl> video_layer = VideoLayerImpl::Create( | 679 - scoped_ptr<VideoLayerImpl> video_layer = VideoLayerImpl::Create( |
| 680 - host_impl_->active_tree(), 4, &provider, media::VIDEO_ROTATION_0); | 680 - host_impl_->active_tree(), 4, &provider, media::VIDEO_ROTATION_0); |
| 681 - video_layer->SetBounds(gfx::Size(10, 10)); | 681 - video_layer->SetBounds(gfx::Size(10, 10)); |
| 682 - video_layer->SetContentBounds(gfx::Size(10, 10)); | 682 - video_layer->SetContentBounds(gfx::Size(10, 10)); |
| 683 - video_layer->SetDrawsContent(true); | 683 - video_layer->SetDrawsContent(true); |
| 684 - root_layer->AddChild(video_layer.Pass()); | 684 - root_layer->AddChild(video_layer.Pass()); |
| 685 - | 685 - |
| 686 scoped_ptr<IOSurfaceLayerImpl> io_surface_layer = | 686 scoped_ptr<IOSurfaceLayerImpl> io_surface_layer = |
| 687 IOSurfaceLayerImpl::Create(host_impl_->active_tree(), 5); | 687 IOSurfaceLayerImpl::Create(host_impl_->active_tree(), 5); |
| 688 io_surface_layer->SetBounds(gfx::Size(10, 10)); | 688 io_surface_layer->SetBounds(gfx::Size(10, 10)); |
| 689 @@ -6321,16 +6304,6 @@ TEST_F(LayerTreeHostImplTest, | 689 @@ -6322,16 +6305,6 @@ TEST_F(LayerTreeHostImplTest, |
| 690 scoped_ptr<SolidColorLayerImpl> root_layer = | 690 scoped_ptr<SolidColorLayerImpl> root_layer = |
| 691 SolidColorLayerImpl::Create(host_impl_->active_tree(), 1); | 691 SolidColorLayerImpl::Create(host_impl_->active_tree(), 1); |
| 692 | 692 |
| 693 - // VideoLayerImpl will not be drawn. | 693 - // VideoLayerImpl will not be drawn. |
| 694 - FakeVideoFrameProvider provider; | 694 - FakeVideoFrameProvider provider; |
| 695 - scoped_ptr<VideoLayerImpl> video_layer = VideoLayerImpl::Create( | 695 - scoped_ptr<VideoLayerImpl> video_layer = VideoLayerImpl::Create( |
| 696 - host_impl_->active_tree(), 2, &provider, media::VIDEO_ROTATION_0); | 696 - host_impl_->active_tree(), 2, &provider, media::VIDEO_ROTATION_0); |
| 697 - video_layer->SetBounds(gfx::Size(10, 10)); | 697 - video_layer->SetBounds(gfx::Size(10, 10)); |
| 698 - video_layer->SetContentBounds(gfx::Size(10, 10)); | 698 - video_layer->SetContentBounds(gfx::Size(10, 10)); |
| 699 - video_layer->SetDrawsContent(true); | 699 - video_layer->SetDrawsContent(true); |
| 700 - root_layer->AddChild(video_layer.Pass()); | 700 - root_layer->AddChild(video_layer.Pass()); |
| 701 - SetupRootLayerImpl(root_layer.Pass()); | 701 - SetupRootLayerImpl(root_layer.Pass()); |
| 702 - | 702 - |
| 703 LayerTreeHostImpl::FrameData frame; | 703 LayerTreeHostImpl::FrameData frame; |
| 704 EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame)); | 704 EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame)); |
| 705 host_impl_->DrawLayers(&frame, gfx::FrameTime::Now()); | 705 host_impl_->DrawLayers(&frame, gfx::FrameTime::Now()); |
| 706 diff --git a/cc/trees/layer_tree_host_unittest.cc b/cc/trees/layer_tree_host_uni
ttest.cc | 706 diff --git a/cc/trees/layer_tree_host_unittest.cc b/cc/trees/layer_tree_host_uni
ttest.cc |
| 707 index 712909f..51c5d7e 100644 | 707 index a343466..eee6275 100644 |
| 708 --- a/cc/trees/layer_tree_host_unittest.cc | 708 --- a/cc/trees/layer_tree_host_unittest.cc |
| 709 +++ b/cc/trees/layer_tree_host_unittest.cc | 709 +++ b/cc/trees/layer_tree_host_unittest.cc |
| 710 @@ -18,7 +18,6 @@ | 710 @@ -18,7 +18,6 @@ |
| 711 #include "cc/layers/painted_scrollbar_layer.h" | 711 #include "cc/layers/painted_scrollbar_layer.h" |
| 712 #include "cc/layers/picture_layer.h" | 712 #include "cc/layers/picture_layer.h" |
| 713 #include "cc/layers/solid_color_layer.h" | 713 #include "cc/layers/solid_color_layer.h" |
| 714 -#include "cc/layers/video_layer.h" | 714 -#include "cc/layers/video_layer.h" |
| 715 #include "cc/output/begin_frame_args.h" | 715 #include "cc/output/begin_frame_args.h" |
| 716 #include "cc/output/compositor_frame_ack.h" | 716 #include "cc/output/compositor_frame_ack.h" |
| 717 #include "cc/output/copy_output_request.h" | 717 #include "cc/output/copy_output_request.h" |
| 718 @@ -41,7 +40,6 @@ | 718 @@ -41,7 +40,6 @@ |
| 719 #include "cc/test/fake_picture_pile.h" | 719 #include "cc/test/fake_picture_pile.h" |
| 720 #include "cc/test/fake_proxy.h" | 720 #include "cc/test/fake_proxy.h" |
| 721 #include "cc/test/fake_scoped_ui_resource.h" | 721 #include "cc/test/fake_scoped_ui_resource.h" |
| 722 -#include "cc/test/fake_video_frame_provider.h" | 722 -#include "cc/test/fake_video_frame_provider.h" |
| 723 #include "cc/test/geometry_test_utils.h" | 723 #include "cc/test/geometry_test_utils.h" |
| 724 #include "cc/test/layer_tree_test.h" | 724 #include "cc/test/layer_tree_test.h" |
| 725 #include "cc/test/test_shared_bitmap_manager.h" | 725 #include "cc/test/test_shared_bitmap_manager.h" |
| 726 @@ -4189,28 +4187,6 @@ class LayerInvalidateCausesDraw : public LayerTreeHostTes
t { | 726 @@ -4183,28 +4181,6 @@ class LayerInvalidateCausesDraw : public LayerTreeHostTes
t { |
| 727 int num_draws_; | 727 int num_draws_; |
| 728 }; | 728 }; |
| 729 | 729 |
| 730 -// VideoLayer must support being invalidated and then passing that along | 730 -// VideoLayer must support being invalidated and then passing that along |
| 731 -// to the compositor thread, even though no resources are updated in | 731 -// to the compositor thread, even though no resources are updated in |
| 732 -// response to that invalidation. | 732 -// response to that invalidation. |
| 733 -class LayerTreeHostTestVideoLayerInvalidate : public LayerInvalidateCausesDraw
{ | 733 -class LayerTreeHostTestVideoLayerInvalidate : public LayerInvalidateCausesDraw
{ |
| 734 - public: | 734 - public: |
| 735 - void SetupTree() override { | 735 - void SetupTree() override { |
| 736 - LayerTreeHostTest::SetupTree(); | 736 - LayerTreeHostTest::SetupTree(); |
| 737 - scoped_refptr<VideoLayer> video_layer = | 737 - scoped_refptr<VideoLayer> video_layer = |
| 738 - VideoLayer::Create(&provider_, media::VIDEO_ROTATION_0); | 738 - VideoLayer::Create(&provider_, media::VIDEO_ROTATION_0); |
| 739 - video_layer->SetBounds(gfx::Size(10, 10)); | 739 - video_layer->SetBounds(gfx::Size(10, 10)); |
| 740 - video_layer->SetIsDrawable(true); | 740 - video_layer->SetIsDrawable(true); |
| 741 - layer_tree_host()->root_layer()->AddChild(video_layer); | 741 - layer_tree_host()->root_layer()->AddChild(video_layer); |
| 742 - | 742 - |
| 743 - invalidate_layer_ = video_layer; | 743 - invalidate_layer_ = video_layer; |
| 744 - } | 744 - } |
| 745 - | 745 - |
| 746 - private: | 746 - private: |
| 747 - FakeVideoFrameProvider provider_; | 747 - FakeVideoFrameProvider provider_; |
| 748 -}; | 748 -}; |
| 749 - | 749 - |
| 750 -SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestVideoLayerInvalidate); | 750 -SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestVideoLayerInvalidate); |
| 751 - | 751 - |
| 752 // IOSurfaceLayer must support being invalidated and then passing that along | 752 // IOSurfaceLayer must support being invalidated and then passing that along |
| 753 // to the compositor thread, even though no resources are updated in | 753 // to the compositor thread, even though no resources are updated in |
| 754 // response to that invalidation. | 754 // response to that invalidation. |
| 755 diff --git a/cc/trees/layer_tree_host_unittest_context.cc b/cc/trees/layer_tree_
host_unittest_context.cc | 755 diff --git a/cc/trees/layer_tree_host_unittest_context.cc b/cc/trees/layer_tree_
host_unittest_context.cc |
| 756 index 2d23d75..b646e6a 100644 | 756 index 24ffcd4..cc4f28f 100644 |
| 757 --- a/cc/trees/layer_tree_host_unittest_context.cc | 757 --- a/cc/trees/layer_tree_host_unittest_context.cc |
| 758 +++ b/cc/trees/layer_tree_host_unittest_context.cc | 758 +++ b/cc/trees/layer_tree_host_unittest_context.cc |
| 759 @@ -15,8 +15,6 @@ | 759 @@ -15,8 +15,6 @@ |
| 760 #include "cc/layers/picture_layer.h" | 760 #include "cc/layers/picture_layer.h" |
| 761 #include "cc/layers/texture_layer.h" | 761 #include "cc/layers/texture_layer.h" |
| 762 #include "cc/layers/texture_layer_impl.h" | 762 #include "cc/layers/texture_layer_impl.h" |
| 763 -#include "cc/layers/video_layer.h" | 763 -#include "cc/layers/video_layer.h" |
| 764 -#include "cc/layers/video_layer_impl.h" | 764 -#include "cc/layers/video_layer_impl.h" |
| 765 #include "cc/output/filter_operations.h" | 765 #include "cc/output/filter_operations.h" |
| 766 #include "cc/resources/single_release_callback.h" | 766 #include "cc/resources/single_release_callback.h" |
| 767 #include "cc/test/fake_content_layer.h" | 767 #include "cc/test/failure_output_surface.h" |
| 768 @@ -32,7 +30,6 @@ | 768 @@ -33,7 +31,6 @@ |
| 769 #include "cc/test/fake_picture_layer_impl.h" | 769 #include "cc/test/fake_picture_layer_impl.h" |
| 770 #include "cc/test/fake_scoped_ui_resource.h" | 770 #include "cc/test/fake_scoped_ui_resource.h" |
| 771 #include "cc/test/fake_scrollbar.h" | 771 #include "cc/test/fake_scrollbar.h" |
| 772 -#include "cc/test/fake_video_frame_provider.h" | 772 -#include "cc/test/fake_video_frame_provider.h" |
| 773 #include "cc/test/layer_tree_test.h" | 773 #include "cc/test/layer_tree_test.h" |
| 774 #include "cc/test/render_pass_test_common.h" | 774 #include "cc/test/render_pass_test_common.h" |
| 775 #include "cc/test/test_context_provider.h" | 775 #include "cc/test/test_context_provider.h" |
| 776 @@ -42,9 +39,6 @@ | 776 @@ -43,9 +40,6 @@ |
| 777 #include "cc/trees/layer_tree_impl.h" | 777 #include "cc/trees/layer_tree_impl.h" |
| 778 #include "cc/trees/single_thread_proxy.h" | 778 #include "cc/trees/single_thread_proxy.h" |
| 779 #include "gpu/GLES2/gl2extchromium.h" | 779 #include "gpu/GLES2/gl2extchromium.h" |
| 780 -#include "media/base/media.h" | 780 -#include "media/base/media.h" |
| 781 - | 781 - |
| 782 -using media::VideoFrame; | 782 -using media::VideoFrame; |
| 783 | 783 |
| 784 namespace cc { | 784 namespace cc { |
| 785 namespace { | 785 namespace { |
| 786 @@ -65,7 +59,6 @@ class LayerTreeHostContextTest : public LayerTreeTest { | 786 @@ -66,7 +60,6 @@ class LayerTreeHostContextTest : public LayerTreeTest { |
| 787 context_should_support_io_surface_(false), | 787 context_should_support_io_surface_(false), |
| 788 fallback_context_works_(false), | 788 fallback_context_works_(false), |
| 789 async_output_surface_creation_(false) { | 789 async_output_surface_creation_(false) { |
| 790 - media::InitializeMediaLibraryForTesting(); | 790 - media::InitializeMediaLibraryForTesting(); |
| 791 } | 791 } |
| 792 | 792 |
| 793 void LoseContext() { | 793 void LoseContext() { |
| 794 @@ -1083,49 +1076,6 @@ class LayerTreeHostContextTestDontUseLostResources | 794 @@ -1041,49 +1034,6 @@ class LayerTreeHostContextTestDontUseLostResources |
| 795 layer_with_mask->SetMaskLayer(mask.get()); | 795 layer_with_mask->SetMaskLayer(mask.get()); |
| 796 root->AddChild(layer_with_mask); | 796 root->AddChild(layer_with_mask); |
| 797 | 797 |
| 798 - scoped_refptr<VideoLayer> video_color = | 798 - scoped_refptr<VideoLayer> video_color = |
| 799 - VideoLayer::Create(&color_frame_provider_, media::VIDEO_ROTATION_0); | 799 - VideoLayer::Create(&color_frame_provider_, media::VIDEO_ROTATION_0); |
| 800 - video_color->SetBounds(gfx::Size(10, 10)); | 800 - video_color->SetBounds(gfx::Size(10, 10)); |
| 801 - video_color->SetIsDrawable(true); | 801 - video_color->SetIsDrawable(true); |
| 802 - root->AddChild(video_color); | 802 - root->AddChild(video_color); |
| 803 - | 803 - |
| 804 - scoped_refptr<VideoLayer> video_hw = | 804 - scoped_refptr<VideoLayer> video_hw = |
| (...skipping 29 matching lines...) Expand all Loading... |
| 834 - base::TimeDelta(), | 834 - base::TimeDelta(), |
| 835 - VideoFrame::ReadPixelsCB()); | 835 - VideoFrame::ReadPixelsCB()); |
| 836 - | 836 - |
| 837 - color_frame_provider_.set_frame(color_video_frame_); | 837 - color_frame_provider_.set_frame(color_video_frame_); |
| 838 - hw_frame_provider_.set_frame(hw_video_frame_); | 838 - hw_frame_provider_.set_frame(hw_video_frame_); |
| 839 - scaled_hw_frame_provider_.set_frame(scaled_hw_video_frame_); | 839 - scaled_hw_frame_provider_.set_frame(scaled_hw_video_frame_); |
| 840 - | 840 - |
| 841 if (!delegating_renderer()) { | 841 if (!delegating_renderer()) { |
| 842 // TODO(danakj): IOSurface layer can not be transported. crbug.com/239335 | 842 // TODO(danakj): IOSurface layer can not be transported. crbug.com/239335 |
| 843 scoped_refptr<IOSurfaceLayer> io_surface = IOSurfaceLayer::Create(); | 843 scoped_refptr<IOSurfaceLayer> io_surface = IOSurfaceLayer::Create(); |
| 844 @@ -1155,14 +1105,6 @@ class LayerTreeHostContextTestDontUseLostResources | 844 @@ -1113,14 +1063,6 @@ class LayerTreeHostContextTestDontUseLostResources |
| 845 | 845 |
| 846 void CommitCompleteOnThread(LayerTreeHostImpl* host_impl) override { | 846 void CommitCompleteOnThread(LayerTreeHostImpl* host_impl) override { |
| 847 LayerTreeHostContextTest::CommitCompleteOnThread(host_impl); | 847 LayerTreeHostContextTest::CommitCompleteOnThread(host_impl); |
| 848 - | 848 - |
| 849 - if (host_impl->active_tree()->source_frame_number() == 3) { | 849 - if (host_impl->active_tree()->source_frame_number() == 3) { |
| 850 - // On the third commit we're recovering from context loss. Hardware | 850 - // On the third commit we're recovering from context loss. Hardware |
| 851 - // video frames should not be reused by the VideoFrameProvider, but | 851 - // video frames should not be reused by the VideoFrameProvider, but |
| 852 - // software frames can be. | 852 - // software frames can be. |
| 853 - hw_frame_provider_.set_frame(NULL); | 853 - hw_frame_provider_.set_frame(NULL); |
| 854 - scaled_hw_frame_provider_.set_frame(NULL); | 854 - scaled_hw_frame_provider_.set_frame(NULL); |
| 855 - } | 855 - } |
| 856 } | 856 } |
| 857 | 857 |
| 858 DrawResult PrepareToDrawOnThread(LayerTreeHostImpl* host_impl, | 858 DrawResult PrepareToDrawOnThread(LayerTreeHostImpl* host_impl, |
| 859 @@ -1212,14 +1154,6 @@ class LayerTreeHostContextTestDontUseLostResources | 859 @@ -1169,14 +1111,6 @@ class LayerTreeHostContextTestDontUseLostResources |
| 860 scoped_refptr<DelegatedFrameResourceCollection> | 860 scoped_refptr<DelegatedFrameResourceCollection> |
| 861 delegated_resource_collection_; | 861 delegated_resource_collection_; |
| 862 scoped_refptr<DelegatedFrameProvider> delegated_frame_provider_; | 862 scoped_refptr<DelegatedFrameProvider> delegated_frame_provider_; |
| 863 - | 863 - |
| 864 - scoped_refptr<VideoFrame> color_video_frame_; | 864 - scoped_refptr<VideoFrame> color_video_frame_; |
| 865 - scoped_refptr<VideoFrame> hw_video_frame_; | 865 - scoped_refptr<VideoFrame> hw_video_frame_; |
| 866 - scoped_refptr<VideoFrame> scaled_hw_video_frame_; | 866 - scoped_refptr<VideoFrame> scaled_hw_video_frame_; |
| 867 - | 867 - |
| 868 - FakeVideoFrameProvider color_frame_provider_; | 868 - FakeVideoFrameProvider color_frame_provider_; |
| 869 - FakeVideoFrameProvider hw_frame_provider_; | 869 - FakeVideoFrameProvider hw_frame_provider_; |
| 870 - FakeVideoFrameProvider scaled_hw_frame_provider_; | 870 - FakeVideoFrameProvider scaled_hw_frame_provider_; |
| 871 }; | 871 }; |
| 872 | 872 |
| 873 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostContextTestDontUseLostResources); | 873 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostContextTestDontUseLostResources); |
| OLD | NEW |