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

Side by Side Diff: cc/CCDamageTrackerTest.cpp

Issue 11079012: Refactor CCLayerTreeHostCommon - merge visible rect computation into calc draw transforms. (Closed) Base URL: http://git.chromium.org/chromium/src.git@master
Patch Set: Same patch without renaming calculateVisibleRect() Created 8 years, 2 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/CCLayerIteratorTest.cpp » ('j') | cc/CCLayerTreeHostCommon.cpp » ('J')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 "config.h" 5 #include "config.h"
6 6
7 #include "CCDamageTracker.h" 7 #include "CCDamageTracker.h"
8 8
9 #include "CCGeometryTestUtils.h" 9 #include "CCGeometryTestUtils.h"
10 #include "CCLayerImpl.h" 10 #include "CCLayerImpl.h"
(...skipping 17 matching lines...) Expand all
28 CCLayerSorter layerSorter; 28 CCLayerSorter layerSorter;
29 int dummyMaxTextureSize = 512; 29 int dummyMaxTextureSize = 512;
30 30
31 // Sanity check: The test itself should create the root layer's render surfa ce, so 31 // Sanity check: The test itself should create the root layer's render surfa ce, so
32 // that the surface (and its damage tracker) can persist acros s multiple 32 // that the surface (and its damage tracker) can persist acros s multiple
33 // calls to this function. 33 // calls to this function.
34 ASSERT_TRUE(root->renderSurface()); 34 ASSERT_TRUE(root->renderSurface());
35 ASSERT_FALSE(renderSurfaceLayerList.size()); 35 ASSERT_FALSE(renderSurfaceLayerList.size());
36 36
37 CCLayerTreeHostCommon::calculateDrawTransforms(root, root->bounds(), 1, &lay erSorter, dummyMaxTextureSize, renderSurfaceLayerList); 37 CCLayerTreeHostCommon::calculateDrawTransforms(root, root->bounds(), 1, &lay erSorter, dummyMaxTextureSize, renderSurfaceLayerList);
38 CCLayerTreeHostCommon::calculateVisibleRects(renderSurfaceLayerList);
39 } 38 }
40 39
41 void clearDamageForAllSurfaces(CCLayerImpl* layer) 40 void clearDamageForAllSurfaces(CCLayerImpl* layer)
42 { 41 {
43 if (layer->renderSurface()) 42 if (layer->renderSurface())
44 layer->renderSurface()->damageTracker()->didDrawDamagedArea(); 43 layer->renderSurface()->damageTracker()->didDrawDamagedArea();
45 44
46 // Recursively clear damage for any existing surface. 45 // Recursively clear damage for any existing surface.
47 for (size_t i = 0; i < layer->children().size(); ++i) 46 for (size_t i = 0; i < layer->children().size(); ++i)
48 clearDamageForAllSurfaces(layer->children()[i]); 47 clearDamageForAllSurfaces(layer->children()[i]);
(...skipping 1080 matching lines...) Expand 10 before | Expand all | Expand 10 after
1129 rootDamageRect = root->renderSurface()->damageTracker()->currentDamageRect() ; 1128 rootDamageRect = root->renderSurface()->damageTracker()->currentDamageRect() ;
1130 EXPECT_TRUE(rootDamageRect.isEmpty()); 1129 EXPECT_TRUE(rootDamageRect.isEmpty());
1131 1130
1132 // Damage should remain empty even after one frame, since there's yet no new damage 1131 // Damage should remain empty even after one frame, since there's yet no new damage
1133 emulateDrawingOneFrame(root.get()); 1132 emulateDrawingOneFrame(root.get());
1134 rootDamageRect = root->renderSurface()->damageTracker()->currentDamageRect() ; 1133 rootDamageRect = root->renderSurface()->damageTracker()->currentDamageRect() ;
1135 EXPECT_TRUE(rootDamageRect.isEmpty()); 1134 EXPECT_TRUE(rootDamageRect.isEmpty());
1136 } 1135 }
1137 1136
1138 } // namespace 1137 } // namespace
OLDNEW
« no previous file with comments | « no previous file | cc/CCLayerIteratorTest.cpp » ('j') | cc/CCLayerTreeHostCommon.cpp » ('J')

Powered by Google App Engine
This is Rietveld 408576698