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

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

Issue 2449853004: Getting rid of DelegatedFrameData (Closed)
Patch Set: IsEmpty + rebase 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
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 6960 matching lines...) Expand 10 before | Expand all | Expand 10 after
6971 EXPECT_EQ(DRAW_SUCCESS, draw_result); 6971 EXPECT_EQ(DRAW_SUCCESS, draw_result);
6972 EXPECT_EQ(0, num_swaps_); 6972 EXPECT_EQ(0, num_swaps_);
6973 drawn_viewport_ = host_impl->DeviceViewport(); 6973 drawn_viewport_ = host_impl->DeviceViewport();
6974 return draw_result; 6974 return draw_result;
6975 } 6975 }
6976 6976
6977 void DisplayReceivedCompositorFrameOnThread( 6977 void DisplayReceivedCompositorFrameOnThread(
6978 const CompositorFrame& frame) override { 6978 const CompositorFrame& frame) override {
6979 EXPECT_EQ(1, ++num_swaps_); 6979 EXPECT_EQ(1, ++num_swaps_);
6980 6980
6981 DelegatedFrameData* last_frame_data = frame.delegated_frame_data.get(); 6981 EXPECT_EQ(drawn_viewport_, frame.render_pass_list.back()->output_rect);
6982 ASSERT_TRUE(frame.delegated_frame_data);
6983 EXPECT_EQ(drawn_viewport_,
6984 last_frame_data->render_pass_list.back()->output_rect);
6985 EXPECT_EQ(0.5f, frame.metadata.min_page_scale_factor); 6982 EXPECT_EQ(0.5f, frame.metadata.min_page_scale_factor);
6986 EXPECT_EQ(4.f, frame.metadata.max_page_scale_factor); 6983 EXPECT_EQ(4.f, frame.metadata.max_page_scale_factor);
6987 6984
6988 EXPECT_EQ(0u, frame.delegated_frame_data->resource_list.size()); 6985 EXPECT_EQ(0u, frame.resource_list.size());
6989 EXPECT_EQ(1u, frame.delegated_frame_data->render_pass_list.size()); 6986 EXPECT_EQ(1u, frame.render_pass_list.size());
6990 6987
6991 EndTest(); 6988 EndTest();
6992 } 6989 }
6993 6990
6994 void AfterTest() override {} 6991 void AfterTest() override {}
6995 6992
6996 int num_swaps_ = 0; 6993 int num_swaps_ = 0;
6997 gfx::Rect drawn_viewport_; 6994 gfx::Rect drawn_viewport_;
6998 }; 6995 };
6999 6996
(...skipping 17 matching lines...) Expand all
7017 7014
7018 RenderPass* pass = AddRenderPass(&frame->render_passes, RenderPassId(1, 1), 7015 RenderPass* pass = AddRenderPass(&frame->render_passes, RenderPassId(1, 1),
7019 gfx::Rect(3, 3, 10, 10), gfx::Transform()); 7016 gfx::Rect(3, 3, 10, 10), gfx::Transform());
7020 AddOneOfEveryQuadType(pass, host_impl->resource_provider(), child_pass->id, 7017 AddOneOfEveryQuadType(pass, host_impl->resource_provider(), child_pass->id,
7021 &mailbox_sync_token); 7018 &mailbox_sync_token);
7022 return draw_result; 7019 return draw_result;
7023 } 7020 }
7024 7021
7025 void DisplayReceivedCompositorFrameOnThread( 7022 void DisplayReceivedCompositorFrameOnThread(
7026 const CompositorFrame& frame) override { 7023 const CompositorFrame& frame) override {
7027 ASSERT_TRUE(frame.delegated_frame_data); 7024 EXPECT_EQ(2u, frame.render_pass_list.size());
7028
7029 EXPECT_EQ(2u, frame.delegated_frame_data->render_pass_list.size());
7030 // Each render pass has 10 resources in it. And the root render pass has a 7025 // Each render pass has 10 resources in it. And the root render pass has a
7031 // mask resource used when drawing the child render pass. The number 10 may 7026 // mask resource used when drawing the child render pass. The number 10 may
7032 // change if AppendOneOfEveryQuadType() is updated, and the value here 7027 // change if AppendOneOfEveryQuadType() is updated, and the value here
7033 // should be updated accordingly. 7028 // should be updated accordingly.
7034 EXPECT_EQ(21u, frame.delegated_frame_data->resource_list.size()); 7029 EXPECT_EQ(21u, frame.resource_list.size());
7035 7030
7036 EndTest(); 7031 EndTest();
7037 } 7032 }
7038 7033
7039 void AfterTest() override {} 7034 void AfterTest() override {}
7040 }; 7035 };
7041 7036
7042 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestSubmitFrameResources); 7037 SINGLE_AND_MULTI_THREAD_TEST_F(LayerTreeHostTestSubmitFrameResources);
7043 7038
7044 } // namespace 7039 } // namespace
7045 } // namespace cc 7040 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698