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

Side by Side Diff: cc/trees/layer_tree_host_unittest.cc

Issue 2449853004: Getting rid of DelegatedFrameData (Closed)
Patch Set: nit Created 4 years, 1 month 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 | « cc/trees/layer_tree_host_impl.cc ('k') | cc/trees/layer_tree_host_unittest_context.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2011 The Chromium Authors. All rights reserved. 1 // Copyright 2011 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "cc/trees/layer_tree_host.h" 5 #include "cc/trees/layer_tree_host.h"
6 6
7 #include <stddef.h> 7 #include <stddef.h>
8 #include <stdint.h> 8 #include <stdint.h>
9 9
10 #include <algorithm> 10 #include <algorithm>
(...skipping 6923 matching lines...) Expand 10 before | Expand all | Expand 10 after
6934 EXPECT_EQ(DRAW_SUCCESS, draw_result); 6934 EXPECT_EQ(DRAW_SUCCESS, draw_result);
6935 EXPECT_EQ(0, num_swaps_); 6935 EXPECT_EQ(0, num_swaps_);
6936 drawn_viewport_ = host_impl->DeviceViewport(); 6936 drawn_viewport_ = host_impl->DeviceViewport();
6937 return draw_result; 6937 return draw_result;
6938 } 6938 }
6939 6939
6940 void DisplayReceivedCompositorFrameOnThread( 6940 void DisplayReceivedCompositorFrameOnThread(
6941 const CompositorFrame& frame) override { 6941 const CompositorFrame& frame) override {
6942 EXPECT_EQ(1, ++num_swaps_); 6942 EXPECT_EQ(1, ++num_swaps_);
6943 6943
6944 DelegatedFrameData* last_frame_data = frame.delegated_frame_data.get(); 6944 EXPECT_EQ(drawn_viewport_, frame.render_pass_list.back()->output_rect);
6945 ASSERT_TRUE(frame.delegated_frame_data);
6946 EXPECT_EQ(drawn_viewport_,
6947 last_frame_data->render_pass_list.back()->output_rect);
6948 EXPECT_EQ(0.5f, frame.metadata.min_page_scale_factor); 6945 EXPECT_EQ(0.5f, frame.metadata.min_page_scale_factor);
6949 EXPECT_EQ(4.f, frame.metadata.max_page_scale_factor); 6946 EXPECT_EQ(4.f, frame.metadata.max_page_scale_factor);
6950 6947
6951 EXPECT_EQ(0u, frame.delegated_frame_data->resource_list.size()); 6948 EXPECT_EQ(0u, frame.resource_list.size());
6952 EXPECT_EQ(1u, frame.delegated_frame_data->render_pass_list.size()); 6949 EXPECT_EQ(1u, frame.render_pass_list.size());
6953 6950
6954 EndTest(); 6951 EndTest();
6955 } 6952 }
6956 6953
6957 void AfterTest() override {} 6954 void AfterTest() override {}
6958 6955
6959 int num_swaps_ = 0; 6956 int num_swaps_ = 0;
6960 gfx::Rect drawn_viewport_; 6957 gfx::Rect drawn_viewport_;
6961 }; 6958 };
6962 6959
(...skipping 17 matching lines...) Expand all
6980 6977
6981 RenderPass* pass = AddRenderPass(&frame->render_passes, RenderPassId(1, 1), 6978 RenderPass* pass = AddRenderPass(&frame->render_passes, RenderPassId(1, 1),
6982 gfx::Rect(3, 3, 10, 10), gfx::Transform()); 6979 gfx::Rect(3, 3, 10, 10), gfx::Transform());
6983 AddOneOfEveryQuadType(pass, host_impl->resource_provider(), child_pass->id, 6980 AddOneOfEveryQuadType(pass, host_impl->resource_provider(), child_pass->id,
6984 &mailbox_sync_token); 6981 &mailbox_sync_token);
6985 return draw_result; 6982 return draw_result;
6986 } 6983 }
6987 6984
6988 void DisplayReceivedCompositorFrameOnThread( 6985 void DisplayReceivedCompositorFrameOnThread(
6989 const CompositorFrame& frame) override { 6986 const CompositorFrame& frame) override {
6990 ASSERT_TRUE(frame.delegated_frame_data); 6987 EXPECT_EQ(2u, frame.render_pass_list.size());
6991
6992 EXPECT_EQ(2u, frame.delegated_frame_data->render_pass_list.size());
6993 // Each render pass has 10 resources in it. And the root render pass has a 6988 // Each render pass has 10 resources in it. And the root render pass has a
6994 // mask resource used when drawing the child render pass. The number 10 may 6989 // mask resource used when drawing the child render pass. The number 10 may
6995 // change if AppendOneOfEveryQuadType() is updated, and the value here 6990 // change if AppendOneOfEveryQuadType() is updated, and the value here
6996 // should be updated accordingly. 6991 // should be updated accordingly.
6997 EXPECT_EQ(21u, frame.delegated_frame_data->resource_list.size()); 6992 EXPECT_EQ(21u, frame.resource_list.size());
6998 6993
6999 EndTest(); 6994 EndTest();
7000 } 6995 }
7001 6996
7002 void AfterTest() override {} 6997 void AfterTest() override {}
7003 }; 6998 };
7004 6999
7005 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestSubmitFrameResources); 7000 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestSubmitFrameResources);
7006 7001
7007 } // namespace 7002 } // namespace
7008 } // namespace cc 7003 } // namespace cc
OLDNEW
« no previous file with comments | « cc/trees/layer_tree_host_impl.cc ('k') | cc/trees/layer_tree_host_unittest_context.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698