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

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

Issue 328753002: Clean up PointIsClippedBySurfaceOrClipRect (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 6 years, 6 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 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 "cc/trees/layer_tree_impl.h" 5 #include "cc/trees/layer_tree_impl.h"
6 6
7 #include <limits> 7 #include <limits>
8 #include <set> 8 #include <set>
9 9
10 #include "base/debug/trace_event.h" 10 #include "base/debug/trace_event.h"
(...skipping 1097 matching lines...) Expand 10 before | Expand all | Expand 10 after
1108 screen_space_point, 1108 screen_space_point,
1109 current_layer->render_surface()->screen_space_transform(), 1109 current_layer->render_surface()->screen_space_transform(),
1110 current_layer->render_surface()->content_rect(), 1110 current_layer->render_surface()->content_rect(),
1111 NULL)) 1111 NULL))
1112 return true; 1112 return true;
1113 1113
1114 // Note that drawable content rects are actually in target surface space, so 1114 // Note that drawable content rects are actually in target surface space, so
1115 // the transform we have to provide is the target surface's 1115 // the transform we have to provide is the target surface's
1116 // screen_space_transform. 1116 // screen_space_transform.
1117 LayerImpl* render_target = current_layer->render_target(); 1117 LayerImpl* render_target = current_layer->render_target();
1118 if (LayerClipsSubtree(current_layer) && 1118 if (render_target && LayerClipsSubtree(current_layer) &&
1119 !PointHitsRect( 1119 !PointHitsRect(
1120 screen_space_point, 1120 screen_space_point,
1121 render_target->render_surface()->screen_space_transform(), 1121 render_target->render_surface()->screen_space_transform(),
1122 current_layer->drawable_content_rect(), 1122 current_layer->drawable_content_rect(),
1123 NULL)) 1123 NULL))
1124 return true; 1124 return true;
1125 1125
1126 current_layer = current_layer->parent(); 1126 current_layer = current_layer->parent();
1127 } 1127 }
1128 1128
(...skipping 140 matching lines...) Expand 10 before | Expand all | Expand 10 after
1269 LayerImpl* LayerTreeImpl::FindLayerThatIsHitByPointInTouchHandlerRegion( 1269 LayerImpl* LayerTreeImpl::FindLayerThatIsHitByPointInTouchHandlerRegion(
1270 const gfx::PointF& screen_space_point) { 1270 const gfx::PointF& screen_space_point) {
1271 FindTouchEventLayerFunctor func = {screen_space_point}; 1271 FindTouchEventLayerFunctor func = {screen_space_point};
1272 FindClosestMatchingLayerDataForRecursion data_for_recursion; 1272 FindClosestMatchingLayerDataForRecursion data_for_recursion;
1273 FindClosestMatchingLayer( 1273 FindClosestMatchingLayer(
1274 screen_space_point, root_layer(), func, &data_for_recursion); 1274 screen_space_point, root_layer(), func, &data_for_recursion);
1275 return data_for_recursion.closest_match; 1275 return data_for_recursion.closest_match;
1276 } 1276 }
1277 1277
1278 } // namespace cc 1278 } // namespace cc
OLDNEW
« no previous file with comments | « no previous file | cc/trees/layer_tree_impl_unittest.cc » ('j') | cc/trees/layer_tree_impl_unittest.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698