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

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

Issue 951673002: Revert "Pull chromium at 2c3ffb2355a27c32f45e508ef861416b820c823b" (Closed) Base URL: git@github.com:domokit/mojo.git@master
Patch Set: Created 5 years, 10 months 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') | mojo/tools/roll/net_sql.patch » ('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 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
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
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
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
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
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
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
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);
OLDNEW
« no previous file with comments | « mojo/services/surfaces/public/interfaces/quads.mojom ('k') | mojo/tools/roll/net_sql.patch » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698