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

Side by Side Diff: cc/damage_tracker_unittest.cc

Issue 11369071: A speculative Revert for r165872 - Remove static thread pointers from CC, attempt 2 (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src/
Patch Set: Created 8 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 | Annotate | Revision Log
« no previous file with comments | « cc/cc_tests.gyp ('k') | cc/delegated_renderer_layer_impl_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 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 "cc/damage_tracker.h" 7 #include "cc/damage_tracker.h"
8 8
9 #include "cc/layer_impl.h" 9 #include "cc/layer_impl.h"
10 #include "cc/layer_sorter.h" 10 #include "cc/layer_sorter.h"
(...skipping 147 matching lines...) Expand 10 before | Expand all | Expand 10 after
158 scoped_ptr<LayerImpl> root = createTestTreeWithTwoSurfaces(); 158 scoped_ptr<LayerImpl> root = createTestTreeWithTwoSurfaces();
159 159
160 // Setup includes going past the first frame which always damages everything , so 160 // Setup includes going past the first frame which always damages everything , so
161 // that we can actually perform specific tests. 161 // that we can actually perform specific tests.
162 emulateDrawingOneFrame(root.get()); 162 emulateDrawingOneFrame(root.get());
163 163
164 return root.Pass(); 164 return root.Pass();
165 } 165 }
166 166
167 class DamageTrackerTest : public testing::Test { 167 class DamageTrackerTest : public testing::Test {
168 private:
169 // For testing purposes, fake that we are on the impl thread.
170 DebugScopedSetImplThread setImplThread;
168 }; 171 };
169 172
170 TEST_F(DamageTrackerTest, sanityCheckTestTreeWithOneSurface) 173 TEST_F(DamageTrackerTest, sanityCheckTestTreeWithOneSurface)
171 { 174 {
172 // Sanity check that the simple test tree will actually produce the expected render 175 // Sanity check that the simple test tree will actually produce the expected render
173 // surfaces and layer lists. 176 // surfaces and layer lists.
174 177
175 scoped_ptr<LayerImpl> root = createAndSetUpTestTreeWithOneSurface(); 178 scoped_ptr<LayerImpl> root = createAndSetUpTestTreeWithOneSurface();
176 179
177 EXPECT_EQ(2u, root->renderSurface()->layerList().size()); 180 EXPECT_EQ(2u, root->renderSurface()->layerList().size());
(...skipping 973 matching lines...) Expand 10 before | Expand all | Expand 10 after
1151 rootDamageRect = root->renderSurface()->damageTracker()->currentDamageRect() ; 1154 rootDamageRect = root->renderSurface()->damageTracker()->currentDamageRect() ;
1152 EXPECT_TRUE(rootDamageRect.IsEmpty()); 1155 EXPECT_TRUE(rootDamageRect.IsEmpty());
1153 1156
1154 // Damage should remain empty even after one frame, since there's yet no new damage 1157 // Damage should remain empty even after one frame, since there's yet no new damage
1155 emulateDrawingOneFrame(root.get()); 1158 emulateDrawingOneFrame(root.get());
1156 rootDamageRect = root->renderSurface()->damageTracker()->currentDamageRect() ; 1159 rootDamageRect = root->renderSurface()->damageTracker()->currentDamageRect() ;
1157 EXPECT_TRUE(rootDamageRect.IsEmpty()); 1160 EXPECT_TRUE(rootDamageRect.IsEmpty());
1158 } 1161 }
1159 1162
1160 } // namespace 1163 } // namespace
OLDNEW
« no previous file with comments | « cc/cc_tests.gyp ('k') | cc/delegated_renderer_layer_impl_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698