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

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

Issue 156603005: cc: Clean up iterator template to only take 1 parameter. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: iteratorcleanup: Created 6 years, 10 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 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/occlusion_tracker.h" 5 #include "cc/trees/occlusion_tracker.h"
6 6
7 #include "base/time/time.h" 7 #include "base/time/time.h"
8 #include "cc/layers/layer_iterator.h" 8 #include "cc/layers/layer_iterator.h"
9 #include "cc/layers/solid_color_layer_impl.h" 9 #include "cc/layers/solid_color_layer_impl.h"
10 #include "cc/test/fake_layer_tree_host_impl_client.h" 10 #include "cc/test/fake_layer_tree_host_impl_client.h"
(...skipping 71 matching lines...) Expand 10 before | Expand all | Expand 10 after
82 opaque_layer->SetDrawsContent(true); 82 opaque_layer->SetDrawsContent(true);
83 opaque_layer->SetBounds(viewport_rect.size()); 83 opaque_layer->SetBounds(viewport_rect.size());
84 opaque_layer->SetContentBounds(viewport_rect.size()); 84 opaque_layer->SetContentBounds(viewport_rect.size());
85 active_tree()->root_layer()->AddChild(opaque_layer.PassAs<LayerImpl>()); 85 active_tree()->root_layer()->AddChild(opaque_layer.PassAs<LayerImpl>());
86 86
87 active_tree()->UpdateDrawProperties(); 87 active_tree()->UpdateDrawProperties();
88 const LayerImplList& rsll = active_tree()->RenderSurfaceLayerList(); 88 const LayerImplList& rsll = active_tree()->RenderSurfaceLayerList();
89 ASSERT_EQ(1u, rsll.size()); 89 ASSERT_EQ(1u, rsll.size());
90 EXPECT_EQ(1u, rsll[0]->render_surface()->layer_list().size()); 90 EXPECT_EQ(1u, rsll[0]->render_surface()->layer_list().size());
91 91
92 typedef LayerIterator<LayerImpl, 92 LayerIterator<LayerImpl> begin = LayerIterator<LayerImpl>::Begin(&rsll);
93 LayerImpl::LayerListType, 93 LayerIterator<LayerImpl> end = LayerIterator<LayerImpl>::End(&rsll);
94 LayerImpl::RenderSurfaceType,
95 LayerIteratorActions::FrontToBack> IteratorType;
96 IteratorType begin = IteratorType::Begin(&rsll);
97 IteratorType end = IteratorType::End(&rsll);
98 94
99 LayerIteratorPosition<LayerImpl> pos = begin; 95 LayerIteratorPosition<LayerImpl> pos = begin;
100 96
101 // The opaque_layer adds occlusion over the whole viewport. 97 // The opaque_layer adds occlusion over the whole viewport.
102 tracker.EnterLayer(pos); 98 tracker.EnterLayer(pos);
103 tracker.LeaveLayer(pos); 99 tracker.LeaveLayer(pos);
104 100
105 gfx::Transform transform_to_target; 101 gfx::Transform transform_to_target;
106 transform_to_target.Translate(0, 96); 102 transform_to_target.Translate(0, 96);
107 bool impl_draw_transform_is_unknown = false; 103 bool impl_draw_transform_is_unknown = false;
(...skipping 24 matching lines...) Expand all
132 EXPECT_EQ(active_tree()->root_layer(), next.current_layer); 128 EXPECT_EQ(active_tree()->root_layer(), next.current_layer);
133 129
134 ++begin; 130 ++begin;
135 EXPECT_EQ(end, begin); 131 EXPECT_EQ(end, begin);
136 132
137 PrintResults(); 133 PrintResults();
138 } 134 }
139 135
140 } // namespace 136 } // namespace
141 } // namespace cc 137 } // namespace cc
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698