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

Unified Diff: cc/trees/occlusion_tracker.h

Issue 93663004: [#2] Pass gfx structs by const ref (gfx::Rect, gfx::RectF) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase to ToT, fix builds on non-linux platforms! Created 6 years, 11 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
« no previous file with comments | « cc/trees/layer_tree_host_unittest_delegated.cc ('k') | cc/trees/occlusion_tracker.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/trees/occlusion_tracker.h
diff --git a/cc/trees/occlusion_tracker.h b/cc/trees/occlusion_tracker.h
index 68418c7e43533436881f397ed8ffbe2ce9e09120..85334bca2c19ec304bd4268b2801046103f01e1c 100644
--- a/cc/trees/occlusion_tracker.h
+++ b/cc/trees/occlusion_tracker.h
@@ -32,7 +32,7 @@ class RenderSurface;
template <typename LayerType, typename RenderSurfaceType>
class CC_EXPORT OcclusionTrackerBase {
public:
- OcclusionTrackerBase(gfx::Rect screen_space_clip_rect,
+ OcclusionTrackerBase(const gfx::Rect& screen_space_clip_rect,
bool record_metrics_for_frame);
~OcclusionTrackerBase();
@@ -48,7 +48,7 @@ class CC_EXPORT OcclusionTrackerBase {
// |render_target| is the contributing layer's render target, and
// |draw_transform| and |impl_draw_transform_is_unknown| are relative to that.
bool Occluded(const LayerType* render_target,
- gfx::Rect content_rect,
+ const gfx::Rect& content_rect,
const gfx::Transform& draw_transform,
bool impl_draw_transform_is_unknown) const;
@@ -58,7 +58,7 @@ class CC_EXPORT OcclusionTrackerBase {
// |draw_transform| and |impl_draw_transform_is_unknown| are relative to that.
gfx::Rect UnoccludedContentRect(
const LayerType* render_target,
- gfx::Rect content_rect,
+ const gfx::Rect& content_rect,
const gfx::Transform& draw_transform,
bool impl_draw_transform_is_unknown) const;
@@ -68,7 +68,7 @@ class CC_EXPORT OcclusionTrackerBase {
gfx::Rect UnoccludedContributingSurfaceContentRect(
const LayerType* layer,
bool for_replica,
- gfx::Rect content_rect) const;
+ const gfx::Rect& content_rect) const;
// Report operations for recording overdraw metrics.
OverdrawMetrics* overdraw_metrics() const {
« no previous file with comments | « cc/trees/layer_tree_host_unittest_delegated.cc ('k') | cc/trees/occlusion_tracker.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698