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

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

Issue 2503203002: Revert "Getting rid of DelegatedFrameData" (Closed)
Patch Set: 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 EXPECT_EQ(drawn_viewport_, frame.render_pass_list.back()->output_rect); 6944 DelegatedFrameData* last_frame_data = frame.delegated_frame_data.get();
6945 ASSERT_TRUE(frame.delegated_frame_data);
6946 EXPECT_EQ(drawn_viewport_,
6947 last_frame_data->render_pass_list.back()->output_rect);
6945 EXPECT_EQ(0.5f, frame.metadata.min_page_scale_factor); 6948 EXPECT_EQ(0.5f, frame.metadata.min_page_scale_factor);
6946 EXPECT_EQ(4.f, frame.metadata.max_page_scale_factor); 6949 EXPECT_EQ(4.f, frame.metadata.max_page_scale_factor);
6947 6950
6948 EXPECT_EQ(0u, frame.resource_list.size()); 6951 EXPECT_EQ(0u, frame.delegated_frame_data->resource_list.size());
6949 EXPECT_EQ(1u, frame.render_pass_list.size()); 6952 EXPECT_EQ(1u, frame.delegated_frame_data->render_pass_list.size());
6950 6953
6951 EndTest(); 6954 EndTest();
6952 } 6955 }
6953 6956
6954 void AfterTest() override {} 6957 void AfterTest() override {}
6955 6958
6956 int num_swaps_ = 0; 6959 int num_swaps_ = 0;
6957 gfx::Rect drawn_viewport_; 6960 gfx::Rect drawn_viewport_;
6958 }; 6961 };
6959 6962
(...skipping 17 matching lines...) Expand all
6977 6980
6978 RenderPass* pass = AddRenderPass(&frame->render_passes, RenderPassId(1, 1), 6981 RenderPass* pass = AddRenderPass(&frame->render_passes, RenderPassId(1, 1),
6979 gfx::Rect(3, 3, 10, 10), gfx::Transform()); 6982 gfx::Rect(3, 3, 10, 10), gfx::Transform());
6980 AddOneOfEveryQuadType(pass, host_impl->resource_provider(), child_pass->id, 6983 AddOneOfEveryQuadType(pass, host_impl->resource_provider(), child_pass->id,
6981 &mailbox_sync_token); 6984 &mailbox_sync_token);
6982 return draw_result; 6985 return draw_result;
6983 } 6986 }
6984 6987
6985 void DisplayReceivedCompositorFrameOnThread( 6988 void DisplayReceivedCompositorFrameOnThread(
6986 const CompositorFrame& frame) override { 6989 const CompositorFrame& frame) override {
6987 EXPECT_EQ(2u, frame.render_pass_list.size()); 6990 ASSERT_TRUE(frame.delegated_frame_data);
6991
6992 EXPECT_EQ(2u, frame.delegated_frame_data->render_pass_list.size());
6988 // Each render pass has 10 resources in it. And the root render pass has a 6993 // Each render pass has 10 resources in it. And the root render pass has a
6989 // mask resource used when drawing the child render pass. The number 10 may 6994 // mask resource used when drawing the child render pass. The number 10 may
6990 // change if AppendOneOfEveryQuadType() is updated, and the value here 6995 // change if AppendOneOfEveryQuadType() is updated, and the value here
6991 // should be updated accordingly. 6996 // should be updated accordingly.
6992 EXPECT_EQ(21u, frame.resource_list.size()); 6997 EXPECT_EQ(21u, frame.delegated_frame_data->resource_list.size());
6993 6998
6994 EndTest(); 6999 EndTest();
6995 } 7000 }
6996 7001
6997 void AfterTest() override {} 7002 void AfterTest() override {}
6998 }; 7003 };
6999 7004
7000 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestSubmitFrameResources); 7005 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestSubmitFrameResources);
7001 7006
7002 } // namespace 7007 } // namespace
7003 } // namespace cc 7008 } // 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