Index: cc/trees/layer_tree_host_common.cc |
diff --git a/cc/trees/layer_tree_host_common.cc b/cc/trees/layer_tree_host_common.cc |
index 7a3b5adbab102d8c5c52b7b095cb4d760ce0b534..70c5037221d1979579ac149f2fe0190b205834a9 100644 |
--- a/cc/trees/layer_tree_host_common.cc |
+++ b/cc/trees/layer_tree_host_common.cc |
@@ -161,8 +161,8 @@ static gfx::Vector2dF ComputeChangeOfBasisTranslation( |
} |
enum TranslateRectDirection { |
- TRANSLATE_RECT_DIRECTION_TO_ANCESTOR, |
- TRANSLATE_RECT_DIRECTION_TO_DESCENDANT |
+ TranslateRectDirectionToAncestor, |
+ TranslateRectDirectionToDescendant |
}; |
template <typename LayerType> |
@@ -172,7 +172,7 @@ static gfx::Rect TranslateRectToTargetSpace(const LayerType& ancestor_layer, |
TranslateRectDirection direction) { |
gfx::Vector2dF translation = ComputeChangeOfBasisTranslation<LayerType>( |
ancestor_layer, descendant_layer); |
- if (direction == TRANSLATE_RECT_DIRECTION_TO_DESCENDANT) |
+ if (direction == TranslateRectDirectionToDescendant) |
translation.Scale(-1.f); |
return gfx::ToEnclosingRect( |
gfx::RectF(rect.origin() + translation, rect.size())); |
@@ -211,16 +211,20 @@ static void UpdateClipRectsForClipChild( |
// clip rects will want to be in its target space, not ours. |
if (clip_parent == layer->clip_parent()) { |
*clip_rect_in_parent_target_space = TranslateRectToTargetSpace<LayerType>( |
- *clip_parent, *layer->parent(), *clip_rect_in_parent_target_space, |
- TRANSLATE_RECT_DIRECTION_TO_DESCENDANT); |
+ *clip_parent, |
+ *layer->parent(), |
+ *clip_rect_in_parent_target_space, |
+ TranslateRectDirectionToDescendant); |
} else { |
// If we're being clipped by our scroll parent, we must translate through |
// our common ancestor. This happens to be our parent, so it is sufficent to |
// translate from our clip parent's space to the space of its ancestor (our |
// parent). |
- *clip_rect_in_parent_target_space = TranslateRectToTargetSpace<LayerType>( |
- *layer->parent(), *clip_parent, *clip_rect_in_parent_target_space, |
- TRANSLATE_RECT_DIRECTION_TO_ANCESTOR); |
+ *clip_rect_in_parent_target_space = |
+ TranslateRectToTargetSpace<LayerType>(*layer->parent(), |
+ *clip_parent, |
+ *clip_rect_in_parent_target_space, |
+ TranslateRectDirectionToAncestor); |
} |
} |
@@ -282,8 +286,10 @@ void UpdateAccumulatedSurfaceState( |
// so we'll need to transform it before it is applied. |
if (layer->clip_parent()) { |
clip_rect = TranslateRectToTargetSpace<LayerType>( |
- *layer->clip_parent(), *layer, clip_rect, |
- TRANSLATE_RECT_DIRECTION_TO_DESCENDANT); |
+ *layer->clip_parent(), |
+ *layer, |
+ clip_rect, |
+ TranslateRectDirectionToDescendant); |
} |
target_rect.Intersect(clip_rect); |
} |
@@ -1197,30 +1203,12 @@ struct PreCalculateMetaInformationRecursiveData { |
} |
}; |
-static bool ValidateRenderSurface(LayerImpl* layer) { |
- // This test verifies that there are no cases where a LayerImpl needs |
- // a render surface, but doesn't have one. |
- if (layer->render_surface()) |
- return true; |
- |
- return layer->filters().IsEmpty() && layer->background_filters().IsEmpty() && |
- !layer->mask_layer() && !layer->replica_layer() && |
- !IsRootLayer(layer) && !layer->is_root_for_isolated_group() && |
- !layer->HasCopyRequest(); |
-} |
- |
-static bool ValidateRenderSurface(Layer* layer) { |
- return true; |
-} |
- |
// Recursively walks the layer tree to compute any information that is needed |
// before doing the main recursion. |
template <typename LayerType> |
static void PreCalculateMetaInformation( |
LayerType* layer, |
PreCalculateMetaInformationRecursiveData* recursive_data) { |
- DCHECK(ValidateRenderSurface(layer)); |
- |
layer->draw_properties().sorted_for_recursion = false; |
layer->draw_properties().has_child_with_a_scroll_parent = false; |