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

Unified Diff: cc/trees/occlusion_tracker.cc

Issue 74923004: Remove unused variable OcclusionTracker::prevent_occlusion. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Created 7 years, 1 month 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
« no previous file with comments | « cc/trees/occlusion_tracker.h ('k') | cc/trees/occlusion_tracker_unittest.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/occlusion_tracker.cc
diff --git a/cc/trees/occlusion_tracker.cc b/cc/trees/occlusion_tracker.cc
index 0856c843d736ac571339d0aa7c0266d2ebec3f8e..09fc11492142255ab969a07fafce6124809df908 100644
--- a/cc/trees/occlusion_tracker.cc
+++ b/cc/trees/occlusion_tracker.cc
@@ -22,7 +22,6 @@ OcclusionTrackerBase<LayerType, RenderSurfaceType>::OcclusionTrackerBase(
gfx::Rect screen_space_clip_rect, bool record_metrics_for_frame)
: screen_space_clip_rect_(screen_space_clip_rect),
overdraw_metrics_(OverdrawMetrics::Create(record_metrics_for_frame)),
- prevent_occlusion_(false),
occluding_screen_space_rects_(NULL),
non_occluding_screen_space_rects_(NULL) {}
@@ -31,16 +30,13 @@ OcclusionTrackerBase<LayerType, RenderSurfaceType>::~OcclusionTrackerBase() {}
template <typename LayerType, typename RenderSurfaceType>
void OcclusionTrackerBase<LayerType, RenderSurfaceType>::EnterLayer(
- const LayerIteratorPosition<LayerType>& layer_iterator,
- bool prevent_occlusion) {
+ const LayerIteratorPosition<LayerType>& layer_iterator) {
LayerType* render_target = layer_iterator.target_render_surface_layer;
if (layer_iterator.represents_itself)
EnterRenderTarget(render_target);
else if (layer_iterator.represents_target_render_surface)
FinishedRenderTarget(render_target);
-
- prevent_occlusion_ = prevent_occlusion;
}
template <typename LayerType, typename RenderSurfaceType>
@@ -54,8 +50,6 @@ void OcclusionTrackerBase<LayerType, RenderSurfaceType>::LeaveLayer(
// but in a way that the surface's own occlusion won't occlude itself.
else if (layer_iterator.represents_contributing_render_surface)
LeaveToRenderTarget(render_target);
-
- prevent_occlusion_ = false;
}
template <typename RenderSurfaceType>
@@ -503,9 +497,6 @@ bool OcclusionTrackerBase<LayerType, RenderSurfaceType>::Occluded(
gfx::Rect content_rect,
const gfx::Transform& draw_transform,
bool impl_draw_transform_is_unknown) const {
- if (prevent_occlusion_)
- return false;
-
DCHECK(!stack_.empty());
if (stack_.empty())
return false;
@@ -555,9 +546,6 @@ gfx::Rect OcclusionTrackerBase<LayerType, RenderSurfaceType>::
gfx::Rect content_rect,
const gfx::Transform& draw_transform,
bool impl_draw_transform_is_unknown) const {
- if (prevent_occlusion_)
- return content_rect;
-
DCHECK(!stack_.empty());
if (stack_.empty())
return content_rect;
@@ -619,9 +607,6 @@ gfx::Rect OcclusionTrackerBase<LayerType, RenderSurfaceType>::
// target in the occlusion tracker.
DCHECK_EQ(layer, stack_.back().target);
- if (prevent_occlusion_)
- return content_rect;
-
if (content_rect.IsEmpty())
return content_rect;
« no previous file with comments | « cc/trees/occlusion_tracker.h ('k') | cc/trees/occlusion_tracker_unittest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698