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

Side by Side Diff: mojo/tools/roll/cc_strip_video.patch

Issue 812543002: Update from https://crrev.com/308331 (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 6 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « mojo/services/surfaces/public/interfaces/quads.mojom ('k') | net/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
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
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
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
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
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);
OLDNEW
« no previous file with comments | « mojo/services/surfaces/public/interfaces/quads.mojom ('k') | net/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698