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

Unified Diff: cc/layers/layer_iterator.h

Issue 250803013: Don't clear render surfaces unnecessarily. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Make the bots happy. Created 6 years, 8 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 side-by-side diff with in-line comments
Download patch
Index: cc/layers/layer_iterator.h
diff --git a/cc/layers/layer_iterator.h b/cc/layers/layer_iterator.h
index f929eb69bd0e87fd066583738a0cd5c9a49f355f..e88ced1a43418b975498d98faf1e83e7c421bb99 100644
--- a/cc/layers/layer_iterator.h
+++ b/cc/layers/layer_iterator.h
@@ -103,16 +103,19 @@ template <typename LayerType> struct LayerIteratorPosition {
template <typename LayerType>
class LayerIterator {
typedef LayerIterator<LayerType> LayerIteratorType;
- typedef typename LayerType::RenderSurfaceListType LayerList;
+ typedef typename LayerType::LayerListType LayerList;
+ typedef typename LayerType::RenderSurfaceListType RenderSurfaceLayerList;
typedef typename LayerType::RenderSurfaceType RenderSurfaceType;
public:
LayerIterator() : render_surface_layer_list_(NULL) {}
- static LayerIteratorType Begin(const LayerList* render_surface_layer_list) {
+ static LayerIteratorType Begin(
+ const RenderSurfaceLayerList* render_surface_layer_list) {
return LayerIteratorType(render_surface_layer_list, true);
}
- static LayerIteratorType End(const LayerList* render_surface_layer_list) {
+ static LayerIteratorType End(
+ const RenderSurfaceLayerList* render_surface_layer_list) {
return LayerIteratorType(render_surface_layer_list, false);
}
@@ -161,7 +164,8 @@ class LayerIterator {
}
private:
- LayerIterator(const LayerList* render_surface_layer_list, bool start)
+ LayerIterator(const RenderSurfaceLayerList* render_surface_layer_list,
+ bool start)
: render_surface_layer_list_(render_surface_layer_list),
target_render_surface_layer_index_(0) {
for (size_t i = 0; i < render_surface_layer_list->size(); ++i) {
@@ -246,8 +250,9 @@ class LayerIterator {
inline LayerType* current_layer() const {
return current_layer_represents_target_render_surface()
- ? target_render_surface_layer()
- : target_render_surface_children().at(current_layer_index_);
+ ? target_render_surface_layer()
+ : LayerTreeHostCommon::get_child_as_raw_ptr(
+ target_render_surface_children(), current_layer_index_);
}
inline bool current_layer_represents_contributing_render_surface() const {
@@ -266,7 +271,7 @@ class LayerIterator {
return target_render_surface()->layer_list();
}
- const LayerList* render_surface_layer_list_;
+ const RenderSurfaceLayerList* render_surface_layer_list_;
// The iterator's current position.

Powered by Google App Engine
This is Rietveld 408576698