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

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

Issue 2854953006: Move disable_display_vsync out of RendererSettings (Closed)
Patch Set: Correct typo Created 3 years, 7 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 | « cc/trees/layer_tree_host_impl_unittest.cc ('k') | cc/trees/layer_tree_host_unittest.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 2012 The Chromium Authors. All rights reserved. 1 // Copyright 2012 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 <stdint.h> 7 #include <stdint.h>
8 8
9 #include <sstream> 9 #include <sstream>
10 10
11 #include "base/files/file_path.h" 11 #include "base/files/file_path.h"
12 #include "base/files/file_util.h" 12 #include "base/files/file_util.h"
13 #include "base/path_service.h" 13 #include "base/path_service.h"
14 #include "base/strings/string_piece.h" 14 #include "base/strings/string_piece.h"
15 #include "base/time/time.h" 15 #include "base/time/time.h"
16 #include "cc/base/lap_timer.h" 16 #include "cc/base/lap_timer.h"
17 #include "cc/layers/nine_patch_layer.h" 17 #include "cc/layers/nine_patch_layer.h"
18 #include "cc/layers/solid_color_layer.h" 18 #include "cc/layers/solid_color_layer.h"
19 #include "cc/layers/texture_layer.h" 19 #include "cc/layers/texture_layer.h"
20 #include "cc/resources/single_release_callback.h" 20 #include "cc/resources/single_release_callback.h"
21 #include "cc/resources/texture_mailbox.h" 21 #include "cc/resources/texture_mailbox.h"
22 #include "cc/test/fake_content_layer_client.h" 22 #include "cc/test/fake_content_layer_client.h"
23 #include "cc/test/layer_tree_json_parser.h" 23 #include "cc/test/layer_tree_json_parser.h"
24 #include "cc/test/layer_tree_test.h" 24 #include "cc/test/layer_tree_test.h"
25 #include "cc/test/paths.h" 25 #include "cc/test/paths.h"
26 #include "cc/test/test_compositor_frame_sink.h"
26 #include "cc/trees/layer_tree_impl.h" 27 #include "cc/trees/layer_tree_impl.h"
27 #include "testing/perf/perf_test.h" 28 #include "testing/perf/perf_test.h"
28 29
29 namespace cc { 30 namespace cc {
30 namespace { 31 namespace {
31 32
32 static const int kTimeLimitMillis = 2000; 33 static const int kTimeLimitMillis = 2000;
33 static const int kWarmupRuns = 5; 34 static const int kWarmupRuns = 5;
34 static const int kTimeCheckInterval = 10; 35 static const int kTimeCheckInterval = 10;
35 36
36 class LayerTreeHostPerfTest : public LayerTreeTest { 37 class LayerTreeHostPerfTest : public LayerTreeTest {
37 public: 38 public:
38 LayerTreeHostPerfTest() 39 LayerTreeHostPerfTest()
39 : draw_timer_(kWarmupRuns, 40 : draw_timer_(kWarmupRuns,
40 base::TimeDelta::FromMilliseconds(kTimeLimitMillis), 41 base::TimeDelta::FromMilliseconds(kTimeLimitMillis),
41 kTimeCheckInterval), 42 kTimeCheckInterval),
42 commit_timer_(0, base::TimeDelta(), 1), 43 commit_timer_(0, base::TimeDelta(), 1),
43 full_damage_each_frame_(false), 44 full_damage_each_frame_(false),
44 begin_frame_driven_drawing_(false), 45 begin_frame_driven_drawing_(false),
45 measure_commit_cost_(false) { 46 measure_commit_cost_(false) {
46 } 47 }
47 48
48 void InitializeSettings(LayerTreeSettings* settings) override { 49 std::unique_ptr<TestCompositorFrameSink> CreateCompositorFrameSink(
49 // LayerTreeTests give the Display's BeginFrameSource directly to the 50 scoped_refptr<ContextProvider> compositor_context_provider,
50 // LayerTreeHost like we do in the Browser process via 51 scoped_refptr<ContextProvider> worker_context_provider) override {
51 // TestDelegatingOutputSurface, so setting disable_display_vsync here 52 constexpr bool disable_display_vsync = true;
52 // unthrottles both the DisplayScheduler and the Scheduler. 53 bool synchronous_composite =
53 settings->renderer_settings.disable_display_vsync = true; 54 !HasImplThread() &&
55 !layer_tree_host()->GetSettings().single_thread_proxy_scheduler;
56 return base::MakeUnique<TestCompositorFrameSink>(
57 compositor_context_provider, std::move(worker_context_provider),
58 shared_bitmap_manager(), gpu_memory_buffer_manager(),
59 layer_tree_host()->GetSettings().renderer_settings,
60 ImplThreadTaskRunner(), synchronous_composite, disable_display_vsync);
54 } 61 }
55 62
56 void BeginTest() override { 63 void BeginTest() override {
57 BuildTree(); 64 BuildTree();
58 PostSetNeedsCommitToMainThread(); 65 PostSetNeedsCommitToMainThread();
59 } 66 }
60 67
61 void BeginMainFrame(const BeginFrameArgs& args) override { 68 void BeginMainFrame(const BeginFrameArgs& args) override {
62 if (begin_frame_driven_drawing_ && !TestEnded()) 69 if (begin_frame_driven_drawing_ && !TestEnded())
63 layer_tree_host()->SetNeedsCommitWithForcedRedraw(); 70 layer_tree_host()->SetNeedsCommitWithForcedRedraw();
(...skipping 282 matching lines...) Expand 10 before | Expand all | Expand 10 after
346 TEST_F(LayerTreeHostPerfTestJsonReader, HeavyPageThreaded) { 353 TEST_F(LayerTreeHostPerfTestJsonReader, HeavyPageThreaded) {
347 begin_frame_driven_drawing_ = true; 354 begin_frame_driven_drawing_ = true;
348 measure_commit_cost_ = true; 355 measure_commit_cost_ = true;
349 SetTestName("heavy_page"); 356 SetTestName("heavy_page");
350 ReadTestFile("heavy_layer_tree"); 357 ReadTestFile("heavy_layer_tree");
351 RunTest(CompositorMode::THREADED); 358 RunTest(CompositorMode::THREADED);
352 } 359 }
353 360
354 } // namespace 361 } // namespace
355 } // namespace cc 362 } // namespace cc
OLDNEW
« no previous file with comments | « cc/trees/layer_tree_host_impl_unittest.cc ('k') | cc/trees/layer_tree_host_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698