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

Unified Diff: cc/resources/picture_layer_tiling_set.h

Issue 640063010: cc: Don't swap PictureLayerTilingSet on activate. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: noswap: perftest Created 6 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
Index: cc/resources/picture_layer_tiling_set.h
diff --git a/cc/resources/picture_layer_tiling_set.h b/cc/resources/picture_layer_tiling_set.h
index f0d0fd7890702ec8a0b4b149fa5216d2068ea004..15b9744a27be870e525bd180e15ab6d47598f37b 100644
--- a/cc/resources/picture_layer_tiling_set.h
+++ b/cc/resources/picture_layer_tiling_set.h
@@ -42,28 +42,44 @@ class CC_EXPORT PictureLayerTilingSet {
~PictureLayerTilingSet();
- void SetClient(PictureLayerTilingClient* client);
const PictureLayerTilingClient* client() const { return client_; }
- void RemoveTilesInRegion(const Region& region);
void CleanUpTilings(float min_acceptable_high_res_scale,
float max_acceptable_high_res_scale,
const std::vector<PictureLayerTiling*>& needed_tilings,
bool should_have_low_res,
PictureLayerTilingSet* twin_set,
PictureLayerTilingSet* recycled_twin_set);
-
+ void RemoveNonIdealTilings();
// Make this set of tilings match the same set of content scales from |other|.
// Delete any tilings that don't meet |minimum_contents_scale|. Recreate
// any tiles that intersect |layer_invalidation|. Update the size of all
// tilings to |new_layer_bounds|.
// Returns true if we had at least one high res tiling synced.
+ // TODO(danakj): Remove this !!!
bool SyncTilings(const PictureLayerTilingSet& other,
enne (OOO) 2014/12/01 22:08:17 SyncTilingsForTesting in the meantime?
danakj 2014/12/05 22:30:44 Done.
const gfx::Size& new_layer_bounds,
const Region& layer_invalidation,
float minimum_contents_scale,
RasterSource* raster_source);
+ void UpdateTilingsToCurrentRasterSource(
+ RasterSource* raster_source,
+ // TODO(danakj): DONT NEED LAYER BOUNDSS WE HAVE THE RASTER SOURCE
+ const gfx::Size& layer_bounds,
+ bool solid_color,
+ const Region& layer_invalidation,
+ float minimum_contents_scale);
+
+ void UpdateTilingsFromPending(
+ RasterSource* raster_source,
+ const PictureLayerTilingSet& other,
+ // TODO(danakj): DONT NEED LAYER BOUNDSS WE HAVE THE RASTER SOURCE
+ const gfx::Size& layer_bounds,
+ bool solid_color,
+ const Region& layer_invalidation,
+ float minimum_contents_scale);
+
PictureLayerTiling* AddTiling(float contents_scale,
const gfx::Size& layer_bounds);
size_t num_tilings() const { return tilings_.size(); }

Powered by Google App Engine
This is Rietveld 408576698