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

Side by Side Diff: cc/output/delegating_renderer_unittest.cc

Issue 2317493003: cc: Toward merging DrawFrame and SwapBuffers in DelegatingRenderer. (Closed)
Patch Set: Created 4 years, 3 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 | « no previous file | cc/test/layer_tree_test.cc » ('j') | cc/test/layer_tree_test.cc » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 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/output/delegating_renderer.h" 5 #include "cc/output/delegating_renderer.h"
6 6
7 #include <stdint.h> 7 #include <stdint.h>
8 8
9 #include "cc/test/fake_output_surface.h" 9 #include "cc/test/fake_output_surface.h"
10 #include "cc/test/layer_tree_test.h" 10 #include "cc/test/layer_tree_test.h"
11 #include "cc/test/render_pass_test_utils.h" 11 #include "cc/test/render_pass_test_utils.h"
12 #include "testing/gtest/include/gtest/gtest.h" 12 #include "testing/gtest/include/gtest/gtest.h"
13 13
14 namespace cc { 14 namespace cc {
15 15
16 class DelegatingRendererTest : public LayerTreeTest {}; 16 class DelegatingRendererTest : public LayerTreeTest {};
17 17
18 class DelegatingRendererTestDraw : public DelegatingRendererTest { 18 class DelegatingRendererTestDraw : public DelegatingRendererTest {
19 public: 19 public:
20 void BeginTest() override { 20 void BeginTest() override {
21 layer_tree()->SetPageScaleFactorAndLimits(1.f, 0.5f, 4.f); 21 layer_tree()->SetPageScaleFactorAndLimits(1.f, 0.5f, 4.f);
22 PostSetNeedsCommitToMainThread(); 22 PostSetNeedsCommitToMainThread();
23 } 23 }
24 24
25 void AfterTest() override {} 25 void AfterTest() override {}
26 26
27 void DrawLayersOnThread(LayerTreeHostImpl* host_impl) override { 27 DrawResult PrepareToDrawOnThread(LayerTreeHostImpl* host_impl,
danakj 2016/09/07 00:25:35 Because DrawLayersOnThread now happens after Displ
28 LayerTreeHostImpl::FrameData* frame_data,
29 DrawResult draw_result) override {
28 EXPECT_EQ(0, num_swaps_); 30 EXPECT_EQ(0, num_swaps_);
29 drawn_viewport_ = host_impl->DeviceViewport(); 31 drawn_viewport_ = host_impl->DeviceViewport();
32 return draw_result;
30 } 33 }
31 34
32 void DisplayReceivedCompositorFrameOnThread( 35 void DisplayReceivedCompositorFrameOnThread(
33 const CompositorFrame& frame) override { 36 const CompositorFrame& frame) override {
34 EXPECT_EQ(1, ++num_swaps_); 37 EXPECT_EQ(1, ++num_swaps_);
35 38
36 DelegatedFrameData* last_frame_data = frame.delegated_frame_data.get(); 39 DelegatedFrameData* last_frame_data = frame.delegated_frame_data.get();
37 ASSERT_TRUE(frame.delegated_frame_data); 40 ASSERT_TRUE(frame.delegated_frame_data);
38 EXPECT_FALSE(frame.gl_frame_data); 41 EXPECT_FALSE(frame.gl_frame_data);
39 EXPECT_EQ(drawn_viewport_, 42 EXPECT_EQ(drawn_viewport_,
(...skipping 50 matching lines...) Expand 10 before | Expand all | Expand 10 after
90 // updated accordingly. 93 // updated accordingly.
91 EXPECT_EQ(22u, frame.delegated_frame_data->resource_list.size()); 94 EXPECT_EQ(22u, frame.delegated_frame_data->resource_list.size());
92 95
93 EndTest(); 96 EndTest();
94 } 97 }
95 }; 98 };
96 99
97 SINGLE_AND_MULTI_THREAD_TEST_F(DelegatingRendererTestResources); 100 SINGLE_AND_MULTI_THREAD_TEST_F(DelegatingRendererTestResources);
98 101
99 } // namespace cc 102 } // namespace cc
OLDNEW
« no previous file with comments | « no previous file | cc/test/layer_tree_test.cc » ('j') | cc/test/layer_tree_test.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698