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

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

Issue 12259027: cc: Simplify the logic for deciding to update tile priorities. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Add early-out and unit test Created 7 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 | Annotate | Revision Log
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/damage_tracker.h" 5 #include "cc/trees/damage_tracker.h"
6 6
7 #include "cc/base/math_util.h" 7 #include "cc/base/math_util.h"
8 #include "cc/layers/layer_impl.h" 8 #include "cc/layers/layer_impl.h"
9 #include "cc/test/fake_impl_proxy.h" 9 #include "cc/test/fake_impl_proxy.h"
10 #include "cc/test/fake_layer_tree_host_impl.h" 10 #include "cc/test/fake_layer_tree_host_impl.h"
(...skipping 23 matching lines...) Expand all
34 ASSERT_TRUE(root->render_surface()); 34 ASSERT_TRUE(root->render_surface());
35 ASSERT_FALSE(render_surface_layer_list.size()); 35 ASSERT_FALSE(render_surface_layer_list.size());
36 36
37 LayerTreeHostCommon::CalculateDrawProperties(root, 37 LayerTreeHostCommon::CalculateDrawProperties(root,
38 root->bounds(), 38 root->bounds(),
39 1.f, 39 1.f,
40 1.f, 40 1.f,
41 NULL, 41 NULL,
42 dummy_max_texture_size, 42 dummy_max_texture_size,
43 false, 43 false,
44 &render_surface_layer_list, 44 &render_surface_layer_list);
45 false);
46 } 45 }
47 46
48 void ClearDamageForAllSurfaces(LayerImpl* layer) { 47 void ClearDamageForAllSurfaces(LayerImpl* layer) {
49 if (layer->render_surface()) 48 if (layer->render_surface())
50 layer->render_surface()->damage_tracker()->DidDrawDamagedArea(); 49 layer->render_surface()->damage_tracker()->DidDrawDamagedArea();
51 50
52 // Recursively clear damage for any existing surface. 51 // Recursively clear damage for any existing surface.
53 for (size_t i = 0; i < layer->children().size(); ++i) 52 for (size_t i = 0; i < layer->children().size(); ++i)
54 ClearDamageForAllSurfaces(layer->children()[i]); 53 ClearDamageForAllSurfaces(layer->children()[i]);
55 } 54 }
(...skipping 1295 matching lines...) Expand 10 before | Expand all | Expand 10 after
1351 // Damage should remain empty even after one frame, since there's yet no new 1350 // Damage should remain empty even after one frame, since there's yet no new
1352 // damage. 1351 // damage.
1353 EmulateDrawingOneFrame(root.get()); 1352 EmulateDrawingOneFrame(root.get());
1354 root_damage_rect = 1353 root_damage_rect =
1355 root->render_surface()->damage_tracker()->current_damage_rect(); 1354 root->render_surface()->damage_tracker()->current_damage_rect();
1356 EXPECT_TRUE(root_damage_rect.IsEmpty()); 1355 EXPECT_TRUE(root_damage_rect.IsEmpty());
1357 } 1356 }
1358 1357
1359 } // namespace 1358 } // namespace
1360 } // namespace cc 1359 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698