| 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 6fe0694..803b0f9 100644 | 2 index 41f99e2..d6fd028 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 @@ -222,13 +222,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", |
| (...skipping 11 matching lines...) Expand all Loading... |
| 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 @@ -623,8 +614,6 @@ source_set("test_support") { | 28 @@ -623,8 +614,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", | |
| 36 "test/impl_side_painting_settings.h", | |
| 37 @@ -783,7 +772,6 @@ if (!is_win || link_chrome_on_windows) { | 37 @@ -783,7 +772,6 @@ if (!is_win || link_chrome_on_windows) { |
| 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 @@ -816,7 +804,6 @@ if (!is_win || link_chrome_on_windows) { | 45 @@ -815,7 +803,6 @@ if (!is_win || link_chrome_on_windows) { |
| 46 "resources/texture_uploader_unittest.cc", |
| 46 "resources/tile_manager_unittest.cc", | 47 "resources/tile_manager_unittest.cc", |
| 47 "resources/tile_priority_unittest.cc", | 48 "resources/tile_priority_unittest.cc", |
| 48 "resources/tile_task_worker_pool_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 @@ -844,7 +831,6 @@ if (!is_win || link_chrome_on_windows) { | 53 @@ -844,7 +831,6 @@ if (!is_win || link_chrome_on_windows) { |
| 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", |
| (...skipping 184 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 243 - matrix[0], matrix[4], matrix[8], matrix[12], | 243 - matrix[0], matrix[4], matrix[8], matrix[12], |
| 244 - matrix[1], matrix[5], matrix[9], matrix[13], | 244 - matrix[1], matrix[5], matrix[9], matrix[13], |
| 245 - matrix[2], matrix[6], matrix[10], matrix[14], | 245 - matrix[2], matrix[6], matrix[10], matrix[14], |
| 246 - matrix[3], matrix[7], matrix[11], matrix[15]); | 246 - matrix[3], matrix[7], matrix[11], matrix[15]); |
| 247 - if (active_video_layer_) | 247 - if (active_video_layer_) |
| 248 - active_video_layer_->SetNeedsRedraw(); | 248 - active_video_layer_->SetNeedsRedraw(); |
| 249 -} | 249 -} |
| 250 - | 250 - |
| 251 -} // namespace cc | 251 -} // namespace cc |
| 252 diff --git a/cc/output/gl_renderer.cc b/cc/output/gl_renderer.cc | 252 diff --git a/cc/output/gl_renderer.cc b/cc/output/gl_renderer.cc |
| 253 index 6142d96..5c560dc 100644 | 253 index a394a1a..2612f10 100644 |
| 254 --- a/cc/output/gl_renderer.cc | 254 --- a/cc/output/gl_renderer.cc |
| 255 +++ b/cc/output/gl_renderer.cc | 255 +++ b/cc/output/gl_renderer.cc |
| 256 @@ -13,7 +13,6 @@ | 256 @@ -13,7 +13,6 @@ |
| 257 #include "base/logging.h" | 257 #include "base/logging.h" |
| 258 #include "base/trace_event/trace_event.h" | 258 #include "base/trace_event/trace_event.h" |
| 259 #include "cc/base/math_util.h" | 259 #include "cc/base/math_util.h" |
| 260 -#include "cc/layers/video_layer_impl.h" | 260 -#include "cc/layers/video_layer_impl.h" |
| 261 #include "cc/output/compositor_frame.h" | 261 #include "cc/output/compositor_frame.h" |
| 262 #include "cc/output/compositor_frame_metadata.h" | 262 #include "cc/output/compositor_frame_metadata.h" |
| 263 #include "cc/output/context_provider.h" | 263 #include "cc/output/context_provider.h" |
| 264 diff --git a/cc/output/renderer_pixeltest.cc b/cc/output/renderer_pixeltest.cc | 264 diff --git a/cc/output/renderer_pixeltest.cc b/cc/output/renderer_pixeltest.cc |
| 265 index 743e1cf..9e9175e 100644 | 265 index 36765ce..d7d1211 100644 |
| 266 --- a/cc/output/renderer_pixeltest.cc | 266 --- a/cc/output/renderer_pixeltest.cc |
| 267 +++ b/cc/output/renderer_pixeltest.cc | 267 +++ b/cc/output/renderer_pixeltest.cc |
| 268 @@ -12,7 +12,6 @@ | 268 @@ -12,7 +12,6 @@ |
| 269 #include "cc/test/fake_picture_pile_impl.h" | 269 #include "cc/test/fake_picture_pile_impl.h" |
| 270 #include "cc/test/pixel_test.h" | 270 #include "cc/test/pixel_test.h" |
| 271 #include "gpu/command_buffer/client/gles2_interface.h" | 271 #include "gpu/command_buffer/client/gles2_interface.h" |
| 272 -#include "media/base/video_frame.h" | 272 -#include "media/base/video_frame.h" |
| 273 #include "third_party/skia/include/core/SkColorPriv.h" | 273 #include "third_party/skia/include/core/SkColorPriv.h" |
| 274 #include "third_party/skia/include/core/SkImageFilter.h" | 274 #include "third_party/skia/include/core/SkImageFilter.h" |
| 275 #include "third_party/skia/include/core/SkMatrix.h" | 275 #include "third_party/skia/include/core/SkMatrix.h" |
| 276 @@ -388,453 +387,6 @@ TEST_F(GLRendererPixelTest, NonPremultipliedTextureWithBac
kground) { | 276 @@ -386,453 +385,6 @@ TEST_F(GLRendererPixelTest, NonPremultipliedTextureWithBac
kground) { |
| 277 FuzzyPixelOffByOneComparator(true))); | 277 FuzzyPixelOffByOneComparator(true))); |
| 278 } | 278 } |
| 279 | 279 |
| 280 -class VideoGLRendererPixelTest : public GLRendererPixelTest { | 280 -class VideoGLRendererPixelTest : public GLRendererPixelTest { |
| 281 - protected: | 281 - protected: |
| 282 - void CreateTestYUVVideoDrawQuad_Striped(const SharedQuadState* shared_state, | 282 - void CreateTestYUVVideoDrawQuad_Striped(const SharedQuadState* shared_state, |
| 283 - media::VideoFrame::Format format, | 283 - media::VideoFrame::Format format, |
| 284 - bool is_transparent, | 284 - bool is_transparent, |
| 285 - const gfx::RectF& tex_coord_rect, | 285 - const gfx::RectF& tex_coord_rect, |
| 286 - RenderPass* render_pass) { | 286 - RenderPass* render_pass) { |
| (...skipping 163 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 450 - | 450 - |
| 451 - void CreateTestYUVVideoDrawQuad_FromVideoFrame( | 451 - void CreateTestYUVVideoDrawQuad_FromVideoFrame( |
| 452 - const SharedQuadState* shared_state, | 452 - const SharedQuadState* shared_state, |
| 453 - scoped_refptr<media::VideoFrame> video_frame, | 453 - scoped_refptr<media::VideoFrame> video_frame, |
| 454 - uint8 alpha_value, | 454 - uint8 alpha_value, |
| 455 - const gfx::RectF& tex_coord_rect, | 455 - const gfx::RectF& tex_coord_rect, |
| 456 - RenderPass* render_pass) { | 456 - RenderPass* render_pass) { |
| 457 - const bool with_alpha = (video_frame->format() == media::VideoFrame::YV12A)
; | 457 - const bool with_alpha = (video_frame->format() == media::VideoFrame::YV12A)
; |
| 458 - const YUVVideoDrawQuad::ColorSpace color_space = | 458 - const YUVVideoDrawQuad::ColorSpace color_space = |
| 459 - (video_frame->format() == media::VideoFrame::YV12J | 459 - (video_frame->format() == media::VideoFrame::YV12J |
| 460 - ? YUVVideoDrawQuad::JPEG | 460 - ? YUVVideoDrawQuad::REC_601_JPEG |
| 461 - : YUVVideoDrawQuad::REC_601); | 461 - : YUVVideoDrawQuad::REC_601); |
| 462 - const gfx::Rect rect(shared_state->content_bounds); | 462 - const gfx::Rect rect(shared_state->content_bounds); |
| 463 - const gfx::Rect opaque_rect(0, 0, 0, 0); | 463 - const gfx::Rect opaque_rect(0, 0, 0, 0); |
| 464 - | 464 - |
| 465 - if (with_alpha) | 465 - if (with_alpha) |
| 466 - memset(video_frame->data(media::VideoFrame::kAPlane), alpha_value, | 466 - memset(video_frame->data(media::VideoFrame::kAPlane), alpha_value, |
| 467 - video_frame->stride(media::VideoFrame::kAPlane) * | 467 - video_frame->stride(media::VideoFrame::kAPlane) * |
| 468 - video_frame->rows(media::VideoFrame::kAPlane)); | 468 - video_frame->rows(media::VideoFrame::kAPlane)); |
| 469 - | 469 - |
| 470 - VideoFrameExternalResources resources = | 470 - VideoFrameExternalResources resources = |
| (...skipping 250 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
| 721 - EXPECT_TRUE(this->RunPixelTest( | 721 - EXPECT_TRUE(this->RunPixelTest( |
| 722 - &pass_list, | 722 - &pass_list, |
| 723 - base::FilePath(FILE_PATH_LITERAL("black.png")), | 723 - base::FilePath(FILE_PATH_LITERAL("black.png")), |
| 724 - ExactPixelComparator(true))); | 724 - ExactPixelComparator(true))); |
| 725 -} | 725 -} |
| 726 - | 726 - |
| 727 TYPED_TEST(RendererPixelTest, FastPassColorFilterAlpha) { | 727 TYPED_TEST(RendererPixelTest, FastPassColorFilterAlpha) { |
| 728 gfx::Rect viewport_rect(this->device_viewport_size_); | 728 gfx::Rect viewport_rect(this->device_viewport_size_); |
| 729 | 729 |
| 730 diff --git a/cc/quads/yuv_video_draw_quad.h b/cc/quads/yuv_video_draw_quad.h | 730 diff --git a/cc/quads/yuv_video_draw_quad.h b/cc/quads/yuv_video_draw_quad.h |
| 731 index 358929e..15bce98 100644 | 731 index 99ed7e2..d57d56f 100644 |
| 732 --- a/cc/quads/yuv_video_draw_quad.h | 732 --- a/cc/quads/yuv_video_draw_quad.h |
| 733 +++ b/cc/quads/yuv_video_draw_quad.h | 733 +++ b/cc/quads/yuv_video_draw_quad.h |
| 734 @@ -8,7 +8,6 @@ | 734 @@ -8,7 +8,6 @@ |
| 735 #include "base/basictypes.h" | 735 #include "base/basictypes.h" |
| 736 #include "base/memory/scoped_ptr.h" | 736 #include "base/memory/scoped_ptr.h" |
| 737 #include "cc/base/cc_export.h" | 737 #include "cc/base/cc_export.h" |
| 738 -#include "cc/layers/video_layer_impl.h" | 738 -#include "cc/layers/video_layer_impl.h" |
| 739 #include "cc/quads/draw_quad.h" | 739 #include "cc/quads/draw_quad.h" |
| 740 | 740 |
| 741 namespace cc { | 741 namespace cc { |
| 742 diff --git a/cc/resources/drawing_display_item.cc b/cc/resources/drawing_display
_item.cc | 742 diff --git a/cc/resources/drawing_display_item.cc b/cc/resources/drawing_display
_item.cc |
| 743 index 648f9de..6dffad9 100644 | 743 index 0fe86f4..1ef149f 100644 |
| 744 --- a/cc/resources/drawing_display_item.cc | 744 --- a/cc/resources/drawing_display_item.cc |
| 745 +++ b/cc/resources/drawing_display_item.cc | 745 +++ b/cc/resources/drawing_display_item.cc |
| 746 @@ -6,6 +6,7 @@ | 746 @@ -6,6 +6,7 @@ |
| 747 | 747 |
| 748 #include <string> | 748 #include <string> |
| 749 | 749 |
| 750 +#include "base/logging.h" | 750 +#include "base/logging.h" |
| 751 #include "base/strings/stringprintf.h" | 751 #include "base/strings/stringprintf.h" |
| 752 #include "base/trace_event/trace_event_argument.h" | 752 #include "base/trace_event/trace_event_argument.h" |
| 753 #include "cc/debug/picture_debug_util.h" | 753 #include "cc/debug/picture_debug_util.h" |
| 754 diff --git a/cc/trees/layer_tree_host_impl_unittest.cc b/cc/trees/layer_tree_hos
t_impl_unittest.cc | 754 diff --git a/cc/trees/layer_tree_host_impl_unittest.cc b/cc/trees/layer_tree_hos
t_impl_unittest.cc |
| 755 index 70fa92b..38b2d66 100644 | 755 index aa46125..07fb048 100644 |
| 756 --- a/cc/trees/layer_tree_host_impl_unittest.cc | 756 --- a/cc/trees/layer_tree_host_impl_unittest.cc |
| 757 +++ b/cc/trees/layer_tree_host_impl_unittest.cc | 757 +++ b/cc/trees/layer_tree_host_impl_unittest.cc |
| 758 @@ -27,7 +27,6 @@ | 758 @@ -27,7 +27,6 @@ |
| 759 #include "cc/layers/solid_color_scrollbar_layer_impl.h" | 759 #include "cc/layers/solid_color_scrollbar_layer_impl.h" |
| 760 #include "cc/layers/texture_layer_impl.h" | 760 #include "cc/layers/texture_layer_impl.h" |
| 761 #include "cc/layers/tiled_layer_impl.h" | 761 #include "cc/layers/tiled_layer_impl.h" |
| 762 -#include "cc/layers/video_layer_impl.h" | 762 -#include "cc/layers/video_layer_impl.h" |
| 763 #include "cc/output/begin_frame_args.h" | 763 #include "cc/output/begin_frame_args.h" |
| 764 #include "cc/output/compositor_frame_ack.h" | 764 #include "cc/output/compositor_frame_ack.h" |
| 765 #include "cc/output/compositor_frame_metadata.h" | 765 #include "cc/output/compositor_frame_metadata.h" |
| (...skipping 22 matching lines...) Expand all Loading... |
| 788 namespace cc { | 788 namespace cc { |
| 789 namespace { | 789 namespace { |
| 790 @@ -96,7 +92,6 @@ class LayerTreeHostImplTest : public testing::Test, | 790 @@ -96,7 +92,6 @@ class LayerTreeHostImplTest : public testing::Test, |
| 791 reduce_memory_result_(true), | 791 reduce_memory_result_(true), |
| 792 current_limit_bytes_(0), | 792 current_limit_bytes_(0), |
| 793 current_priority_cutoff_value_(0) { | 793 current_priority_cutoff_value_(0) { |
| 794 - media::InitializeMediaLibraryForTesting(); | 794 - media::InitializeMediaLibraryForTesting(); |
| 795 } | 795 } |
| 796 | 796 |
| 797 LayerTreeSettings DefaultSettings() { | 797 LayerTreeSettings DefaultSettings() { |
| 798 @@ -5463,18 +5458,6 @@ TEST_F(LayerTreeHostImplTest, LayersFreeTextures) { | 798 @@ -5489,18 +5484,6 @@ TEST_F(LayerTreeHostImplTest, LayersFreeTextures) { |
| 799 root_layer->SetBounds(gfx::Size(10, 10)); | 799 root_layer->SetBounds(gfx::Size(10, 10)); |
| 800 root_layer->SetHasRenderSurface(true); | 800 root_layer->SetHasRenderSurface(true); |
| 801 | 801 |
| 802 - scoped_refptr<VideoFrame> softwareFrame = | 802 - scoped_refptr<VideoFrame> softwareFrame = |
| 803 - media::VideoFrame::CreateColorFrame( | 803 - media::VideoFrame::CreateColorFrame( |
| 804 - gfx::Size(4, 4), 0x80, 0x80, 0x80, base::TimeDelta()); | 804 - gfx::Size(4, 4), 0x80, 0x80, 0x80, base::TimeDelta()); |
| 805 - FakeVideoFrameProvider provider; | 805 - FakeVideoFrameProvider provider; |
| 806 - provider.set_frame(softwareFrame); | 806 - provider.set_frame(softwareFrame); |
| 807 - scoped_ptr<VideoLayerImpl> video_layer = VideoLayerImpl::Create( | 807 - scoped_ptr<VideoLayerImpl> video_layer = VideoLayerImpl::Create( |
| 808 - host_impl_->active_tree(), 4, &provider, media::VIDEO_ROTATION_0); | 808 - host_impl_->active_tree(), 4, &provider, media::VIDEO_ROTATION_0); |
| 809 - video_layer->SetBounds(gfx::Size(10, 10)); | 809 - video_layer->SetBounds(gfx::Size(10, 10)); |
| 810 - video_layer->SetContentBounds(gfx::Size(10, 10)); | 810 - video_layer->SetContentBounds(gfx::Size(10, 10)); |
| 811 - video_layer->SetDrawsContent(true); | 811 - video_layer->SetDrawsContent(true); |
| 812 - root_layer->AddChild(video_layer.Pass()); | 812 - root_layer->AddChild(video_layer.Pass()); |
| 813 - | 813 - |
| 814 scoped_ptr<IOSurfaceLayerImpl> io_surface_layer = | 814 scoped_ptr<IOSurfaceLayerImpl> io_surface_layer = |
| 815 IOSurfaceLayerImpl::Create(host_impl_->active_tree(), 5); | 815 IOSurfaceLayerImpl::Create(host_impl_->active_tree(), 5); |
| 816 io_surface_layer->SetBounds(gfx::Size(10, 10)); | 816 io_surface_layer->SetBounds(gfx::Size(10, 10)); |
| 817 @@ -6555,16 +6538,6 @@ TEST_F(LayerTreeHostImplTest, | 817 @@ -6580,16 +6563,6 @@ TEST_F(LayerTreeHostImplTest, |
| 818 scoped_ptr<SolidColorLayerImpl> root_layer = | 818 scoped_ptr<SolidColorLayerImpl> root_layer = |
| 819 SolidColorLayerImpl::Create(host_impl_->active_tree(), 1); | 819 SolidColorLayerImpl::Create(host_impl_->active_tree(), 1); |
| 820 | 820 |
| 821 - // VideoLayerImpl will not be drawn. | 821 - // VideoLayerImpl will not be drawn. |
| 822 - FakeVideoFrameProvider provider; | 822 - FakeVideoFrameProvider provider; |
| 823 - scoped_ptr<VideoLayerImpl> video_layer = VideoLayerImpl::Create( | 823 - scoped_ptr<VideoLayerImpl> video_layer = VideoLayerImpl::Create( |
| 824 - host_impl_->active_tree(), 2, &provider, media::VIDEO_ROTATION_0); | 824 - host_impl_->active_tree(), 2, &provider, media::VIDEO_ROTATION_0); |
| 825 - video_layer->SetBounds(gfx::Size(10, 10)); | 825 - video_layer->SetBounds(gfx::Size(10, 10)); |
| 826 - video_layer->SetContentBounds(gfx::Size(10, 10)); | 826 - video_layer->SetContentBounds(gfx::Size(10, 10)); |
| 827 - video_layer->SetDrawsContent(true); | 827 - video_layer->SetDrawsContent(true); |
| 828 - root_layer->AddChild(video_layer.Pass()); | 828 - root_layer->AddChild(video_layer.Pass()); |
| 829 - SetupRootLayerImpl(root_layer.Pass()); | 829 - SetupRootLayerImpl(root_layer.Pass()); |
| 830 - | 830 - |
| 831 LayerTreeHostImpl::FrameData frame; | 831 LayerTreeHostImpl::FrameData frame; |
| 832 EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame)); | 832 EXPECT_EQ(DRAW_SUCCESS, host_impl_->PrepareToDraw(&frame)); |
| 833 host_impl_->DrawLayers(&frame, gfx::FrameTime::Now()); | 833 host_impl_->DrawLayers(&frame, gfx::FrameTime::Now()); |
| 834 diff --git a/cc/trees/layer_tree_host_unittest.cc b/cc/trees/layer_tree_host_uni
ttest.cc | 834 diff --git a/cc/trees/layer_tree_host_unittest.cc b/cc/trees/layer_tree_host_uni
ttest.cc |
| 835 index 48be5b8..f9b690d 100644 | 835 index bb315e7..bb6dc17 100644 |
| 836 --- a/cc/trees/layer_tree_host_unittest.cc | 836 --- a/cc/trees/layer_tree_host_unittest.cc |
| 837 +++ b/cc/trees/layer_tree_host_unittest.cc | 837 +++ b/cc/trees/layer_tree_host_unittest.cc |
| 838 @@ -18,7 +18,6 @@ | 838 @@ -18,7 +18,6 @@ |
| 839 #include "cc/layers/painted_scrollbar_layer.h" | 839 #include "cc/layers/painted_scrollbar_layer.h" |
| 840 #include "cc/layers/picture_layer.h" | 840 #include "cc/layers/picture_layer.h" |
| 841 #include "cc/layers/solid_color_layer.h" | 841 #include "cc/layers/solid_color_layer.h" |
| 842 -#include "cc/layers/video_layer.h" | 842 -#include "cc/layers/video_layer.h" |
| 843 #include "cc/output/begin_frame_args.h" | 843 #include "cc/output/begin_frame_args.h" |
| 844 #include "cc/output/compositor_frame_ack.h" | 844 #include "cc/output/compositor_frame_ack.h" |
| 845 #include "cc/output/copy_output_request.h" | 845 #include "cc/output/copy_output_request.h" |
| 846 @@ -41,7 +40,6 @@ | 846 @@ -41,7 +40,6 @@ |
| 847 #include "cc/test/fake_picture_pile.h" | 847 #include "cc/test/fake_picture_pile.h" |
| 848 #include "cc/test/fake_proxy.h" | 848 #include "cc/test/fake_proxy.h" |
| 849 #include "cc/test/fake_scoped_ui_resource.h" | 849 #include "cc/test/fake_scoped_ui_resource.h" |
| 850 -#include "cc/test/fake_video_frame_provider.h" | 850 -#include "cc/test/fake_video_frame_provider.h" |
| 851 #include "cc/test/geometry_test_utils.h" | 851 #include "cc/test/geometry_test_utils.h" |
| 852 #include "cc/test/impl_side_painting_settings.h" | 852 #include "cc/test/impl_side_painting_settings.h" |
| 853 #include "cc/test/layer_tree_test.h" | 853 #include "cc/test/layer_tree_test.h" |
| 854 @@ -4167,28 +4165,6 @@ class LayerInvalidateCausesDraw : public LayerTreeHostTes
t { | 854 @@ -4202,28 +4200,6 @@ class LayerInvalidateCausesDraw : public LayerTreeHostTes
t { |
| 855 int num_draws_; | 855 int num_draws_; |
| 856 }; | 856 }; |
| 857 | 857 |
| 858 -// VideoLayer must support being invalidated and then passing that along | 858 -// VideoLayer must support being invalidated and then passing that along |
| 859 -// to the compositor thread, even though no resources are updated in | 859 -// to the compositor thread, even though no resources are updated in |
| 860 -// response to that invalidation. | 860 -// response to that invalidation. |
| 861 -class LayerTreeHostTestVideoLayerInvalidate : public LayerInvalidateCausesDraw
{ | 861 -class LayerTreeHostTestVideoLayerInvalidate : public LayerInvalidateCausesDraw
{ |
| 862 - public: | 862 - public: |
| 863 - void SetupTree() override { | 863 - void SetupTree() override { |
| 864 - LayerTreeHostTest::SetupTree(); | 864 - LayerTreeHostTest::SetupTree(); |
| 865 - scoped_refptr<VideoLayer> video_layer = | 865 - scoped_refptr<VideoLayer> video_layer = |
| 866 - VideoLayer::Create(&provider_, media::VIDEO_ROTATION_0); | 866 - VideoLayer::Create(&provider_, media::VIDEO_ROTATION_0); |
| 867 - video_layer->SetBounds(gfx::Size(10, 10)); | 867 - video_layer->SetBounds(gfx::Size(10, 10)); |
| 868 - video_layer->SetIsDrawable(true); | 868 - video_layer->SetIsDrawable(true); |
| 869 - layer_tree_host()->root_layer()->AddChild(video_layer); | 869 - layer_tree_host()->root_layer()->AddChild(video_layer); |
| 870 - | 870 - |
| 871 - invalidate_layer_ = video_layer; | 871 - invalidate_layer_ = video_layer; |
| 872 - } | 872 - } |
| 873 - | 873 - |
| 874 - private: | 874 - private: |
| 875 - FakeVideoFrameProvider provider_; | 875 - FakeVideoFrameProvider provider_; |
| 876 -}; | 876 -}; |
| 877 - | 877 - |
| 878 -SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestVideoLayerInvalidate); | 878 -SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestVideoLayerInvalidate); |
| 879 - | 879 - |
| 880 // IOSurfaceLayer must support being invalidated and then passing that along | 880 // IOSurfaceLayer must support being invalidated and then passing that along |
| 881 // to the compositor thread, even though no resources are updated in | 881 // to the compositor thread, even though no resources are updated in |
| 882 // response to that invalidation. | 882 // response to that invalidation. |
| 883 diff --git a/cc/trees/layer_tree_host_unittest_context.cc b/cc/trees/layer_tree_
host_unittest_context.cc | 883 diff --git a/cc/trees/layer_tree_host_unittest_context.cc b/cc/trees/layer_tree_
host_unittest_context.cc |
| 884 index c99180d..cc536f4 100644 | 884 index 596adc0..fb0c9c8 100644 |
| 885 --- a/cc/trees/layer_tree_host_unittest_context.cc | 885 --- a/cc/trees/layer_tree_host_unittest_context.cc |
| 886 +++ b/cc/trees/layer_tree_host_unittest_context.cc | 886 +++ b/cc/trees/layer_tree_host_unittest_context.cc |
| 887 @@ -15,8 +15,6 @@ | 887 @@ -15,8 +15,6 @@ |
| 888 #include "cc/layers/picture_layer.h" | 888 #include "cc/layers/picture_layer.h" |
| 889 #include "cc/layers/texture_layer.h" | 889 #include "cc/layers/texture_layer.h" |
| 890 #include "cc/layers/texture_layer_impl.h" | 890 #include "cc/layers/texture_layer_impl.h" |
| 891 -#include "cc/layers/video_layer.h" | 891 -#include "cc/layers/video_layer.h" |
| 892 -#include "cc/layers/video_layer_impl.h" | 892 -#include "cc/layers/video_layer_impl.h" |
| 893 #include "cc/output/filter_operations.h" | 893 #include "cc/output/filter_operations.h" |
| 894 #include "cc/resources/single_release_callback.h" | 894 #include "cc/resources/single_release_callback.h" |
| (...skipping 17 matching lines...) Expand all Loading... |
| 912 namespace cc { | 912 namespace cc { |
| 913 namespace { | 913 namespace { |
| 914 @@ -66,7 +60,6 @@ class LayerTreeHostContextTest : public LayerTreeTest { | 914 @@ -66,7 +60,6 @@ class LayerTreeHostContextTest : public LayerTreeTest { |
| 915 context_should_support_io_surface_(false), | 915 context_should_support_io_surface_(false), |
| 916 fallback_context_works_(false), | 916 fallback_context_works_(false), |
| 917 async_output_surface_creation_(false) { | 917 async_output_surface_creation_(false) { |
| 918 - media::InitializeMediaLibraryForTesting(); | 918 - media::InitializeMediaLibraryForTesting(); |
| 919 } | 919 } |
| 920 | 920 |
| 921 void LoseContext() { | 921 void LoseContext() { |
| 922 @@ -1055,41 +1048,6 @@ class LayerTreeHostContextTestDontUseLostResources | 922 @@ -1057,41 +1050,6 @@ class LayerTreeHostContextTestDontUseLostResources |
| 923 layer_with_mask->SetMaskLayer(mask.get()); | 923 layer_with_mask->SetMaskLayer(mask.get()); |
| 924 root->AddChild(layer_with_mask); | 924 root->AddChild(layer_with_mask); |
| 925 | 925 |
| 926 - scoped_refptr<VideoLayer> video_color = | 926 - scoped_refptr<VideoLayer> video_color = |
| 927 - VideoLayer::Create(&color_frame_provider_, media::VIDEO_ROTATION_0); | 927 - VideoLayer::Create(&color_frame_provider_, media::VIDEO_ROTATION_0); |
| 928 - video_color->SetBounds(gfx::Size(10, 10)); | 928 - video_color->SetBounds(gfx::Size(10, 10)); |
| 929 - video_color->SetIsDrawable(true); | 929 - video_color->SetIsDrawable(true); |
| 930 - root->AddChild(video_color); | 930 - root->AddChild(video_color); |
| 931 - | 931 - |
| 932 - scoped_refptr<VideoLayer> video_hw = | 932 - scoped_refptr<VideoLayer> video_hw = |
| (...skipping 21 matching lines...) Expand all Loading... |
| 954 - media::VideoFrame::ReleaseMailboxCB(), gfx::Size(4, 4), | 954 - media::VideoFrame::ReleaseMailboxCB(), gfx::Size(4, 4), |
| 955 - gfx::Rect(0, 0, 3, 2), gfx::Size(4, 4), base::TimeDelta(), false); | 955 - gfx::Rect(0, 0, 3, 2), gfx::Size(4, 4), base::TimeDelta(), false); |
| 956 - | 956 - |
| 957 - color_frame_provider_.set_frame(color_video_frame_); | 957 - color_frame_provider_.set_frame(color_video_frame_); |
| 958 - hw_frame_provider_.set_frame(hw_video_frame_); | 958 - hw_frame_provider_.set_frame(hw_video_frame_); |
| 959 - scaled_hw_frame_provider_.set_frame(scaled_hw_video_frame_); | 959 - scaled_hw_frame_provider_.set_frame(scaled_hw_video_frame_); |
| 960 - | 960 - |
| 961 if (!delegating_renderer()) { | 961 if (!delegating_renderer()) { |
| 962 // TODO(danakj): IOSurface layer can not be transported. crbug.com/239335 | 962 // TODO(danakj): IOSurface layer can not be transported. crbug.com/239335 |
| 963 scoped_refptr<IOSurfaceLayer> io_surface = IOSurfaceLayer::Create(); | 963 scoped_refptr<IOSurfaceLayer> io_surface = IOSurfaceLayer::Create(); |
| 964 @@ -1119,14 +1077,6 @@ class LayerTreeHostContextTestDontUseLostResources | 964 @@ -1121,14 +1079,6 @@ class LayerTreeHostContextTestDontUseLostResources |
| 965 | 965 |
| 966 void CommitCompleteOnThread(LayerTreeHostImpl* host_impl) override { | 966 void CommitCompleteOnThread(LayerTreeHostImpl* host_impl) override { |
| 967 LayerTreeHostContextTest::CommitCompleteOnThread(host_impl); | 967 LayerTreeHostContextTest::CommitCompleteOnThread(host_impl); |
| 968 - | 968 - |
| 969 - if (host_impl->active_tree()->source_frame_number() == 3) { | 969 - if (host_impl->active_tree()->source_frame_number() == 3) { |
| 970 - // On the third commit we're recovering from context loss. Hardware | 970 - // On the third commit we're recovering from context loss. Hardware |
| 971 - // video frames should not be reused by the VideoFrameProvider, but | 971 - // video frames should not be reused by the VideoFrameProvider, but |
| 972 - // software frames can be. | 972 - // software frames can be. |
| 973 - hw_frame_provider_.set_frame(NULL); | 973 - hw_frame_provider_.set_frame(NULL); |
| 974 - scaled_hw_frame_provider_.set_frame(NULL); | 974 - scaled_hw_frame_provider_.set_frame(NULL); |
| 975 - } | 975 - } |
| 976 } | 976 } |
| 977 | 977 |
| 978 DrawResult PrepareToDrawOnThread(LayerTreeHostImpl* host_impl, | 978 DrawResult PrepareToDrawOnThread(LayerTreeHostImpl* host_impl, |
| 979 @@ -1175,14 +1125,6 @@ class LayerTreeHostContextTestDontUseLostResources | 979 @@ -1177,14 +1127,6 @@ class LayerTreeHostContextTestDontUseLostResources |
| 980 scoped_refptr<DelegatedFrameResourceCollection> | 980 scoped_refptr<DelegatedFrameResourceCollection> |
| 981 delegated_resource_collection_; | 981 delegated_resource_collection_; |
| 982 scoped_refptr<DelegatedFrameProvider> delegated_frame_provider_; | 982 scoped_refptr<DelegatedFrameProvider> delegated_frame_provider_; |
| 983 - | 983 - |
| 984 - scoped_refptr<VideoFrame> color_video_frame_; | 984 - scoped_refptr<VideoFrame> color_video_frame_; |
| 985 - scoped_refptr<VideoFrame> hw_video_frame_; | 985 - scoped_refptr<VideoFrame> hw_video_frame_; |
| 986 - scoped_refptr<VideoFrame> scaled_hw_video_frame_; | 986 - scoped_refptr<VideoFrame> scaled_hw_video_frame_; |
| 987 - | 987 - |
| 988 - FakeVideoFrameProvider color_frame_provider_; | 988 - FakeVideoFrameProvider color_frame_provider_; |
| 989 - FakeVideoFrameProvider hw_frame_provider_; | 989 - FakeVideoFrameProvider hw_frame_provider_; |
| 990 - FakeVideoFrameProvider scaled_hw_frame_provider_; | 990 - FakeVideoFrameProvider scaled_hw_frame_provider_; |
| 991 }; | 991 }; |
| 992 | 992 |
| 993 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostContextTestDontUseLostResources); | 993 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostContextTestDontUseLostResources); |
| OLD | NEW |