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

Unified Diff: cc/trees/occlusion_tracker.h

Issue 576173003: cc: Move UnoccludedContentRect to Occlusion. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 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/trees/occlusion_tracker.h
diff --git a/cc/trees/occlusion_tracker.h b/cc/trees/occlusion_tracker.h
index 36adf0659b5781a16444611485e1b412b453a493..b96752ce476fadcc8376f36b50d49245608f225e 100644
--- a/cc/trees/occlusion_tracker.h
+++ b/cc/trees/occlusion_tracker.h
@@ -25,12 +25,11 @@ class RenderSurface;
// front-to-back order. As each layer is visited, one of the methods in this
// class is called to notify it about the current target surface. Then,
// occlusion in the content space of the current layer may be queried, via
-// methods such UnoccludedContentRect(), or using Occlusion from
-// GetCurrentOcclusionForLayer. If the current layer owns a RenderSurfaceImpl,
-// then occlusion on that RenderSurfaceImpl may also be queried via
-// surfaceOccluded() and surfaceUnoccludedContentRect(). Finally, once finished
-// with the layer, occlusion behind the layer should be marked by calling
-// MarkOccludedBehindLayer().
+// Occlusion from GetCurrentOcclusionForLayer(). If the current layer owns a
+// RenderSurfaceImpl, then occlusion on that RenderSurfaceImpl may also be
+// queried via surfaceOccluded() and surfaceUnoccludedContentRect(). Finally,
+// once finished with the layer, occlusion behind the layer should be marked by
+// calling MarkOccludedBehindLayer().
template <typename LayerType>
class CC_EXPORT OcclusionTracker {
public:
@@ -49,13 +48,6 @@ class CC_EXPORT OcclusionTracker {
// traversal.
void LeaveLayer(const LayerIteratorPosition<LayerType>& layer_iterator);
- // Gives an unoccluded sub-rect of |content_rect| in the content space of a
- // layer. Used when considering occlusion for a layer that paints/draws
- // something. |render_target| is the contributing layer's render target, and
- // |draw_transform| and |impl_draw_transform_is_unknown| are relative to that.
- gfx::Rect UnoccludedContentRect(const gfx::Rect& content_rect,
- const gfx::Transform& draw_transform) const;
-
// Gives an unoccluded sub-rect of |content_rect| in the content space of the
// render_target owned by the layer. Used when considering occlusion for a
// contributing surface that is rendering into another target.

Powered by Google App Engine
This is Rietveld 408576698