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

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

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

Powered by Google App Engine
This is Rietveld 408576698