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

Unified Diff: cc/playback/raster_source.h

Issue 1318733006: cc: Do the math for a tile's content rect in layer space once. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 4 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/playback/raster_source.h
diff --git a/cc/playback/raster_source.h b/cc/playback/raster_source.h
index 869ad93900ea664f9f1b1b737315b368fbafbe16..0ff0d546487281f614e46e22aa858a53a99d798b 100644
--- a/cc/playback/raster_source.h
+++ b/cc/playback/raster_source.h
@@ -71,16 +71,14 @@ class CC_EXPORT RasterSource : public base::RefCountedThreadSafe<RasterSource> {
virtual gfx::Size GetSize() const = 0;
// Populate the given list with all SkPixelRefs that may overlap the given
- // rect at given scale.
+ // rect in layer space.
virtual void GatherPixelRefs(
- const gfx::Rect& content_rect,
- float contents_scale,
+ const gfx::Rect& layer_rect,
std::vector<skia::PositionPixelRef>* pixel_refs) const = 0;
- // Return true iff this raster source can raster the given rect at given
- // scale.
- virtual bool CoversRect(const gfx::Rect& content_rect,
- float contents_scale) const = 0;
+ // Return true iff this raster source can raster the given rect in layer
+ // space.
+ virtual bool CoversRect(const gfx::Rect& layer_rect) const = 0;
// Returns true if this raster source has anything to rasterize.
virtual bool HasRecordings() const = 0;

Powered by Google App Engine
This is Rietveld 408576698