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

Unified Diff: cc/resources/managed_tile_state.h

Issue 22831041: cc: Removed low priority bin from tile management. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebased Created 7 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
« no previous file with comments | « no previous file | cc/resources/managed_tile_state.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/resources/managed_tile_state.h
diff --git a/cc/resources/managed_tile_state.h b/cc/resources/managed_tile_state.h
index 3220add53dd93780958489ea5fddf70e12aa0c24..8b72917896eba086059938f937d0c9dce57316c0 100644
--- a/cc/resources/managed_tile_state.h
+++ b/cc/resources/managed_tile_state.h
@@ -22,9 +22,10 @@ enum ManagedTileBin {
SOON_BIN = 2, // Impl-side version of prepainting.
EVENTUALLY_AND_ACTIVE_BIN = 3, // Nice to have, and has a task or resource.
EVENTUALLY_BIN = 4, // Nice to have, if we've got memory and time.
- NEVER_AND_ACTIVE_BIN = 5, // Dont bother, but has a task or resource.
- NEVER_BIN = 6, // Dont bother.
- NUM_BINS = 7
+ AT_LAST_AND_ACTIVE_BIN = 5, // Only do this after all other bins.
+ AT_LAST_BIN = 6, // Only do this after all other bins.
+ NEVER_BIN = 7, // Dont bother.
+ NUM_BINS = 8
// NOTE: Be sure to update ManagedTileBinAsValue and kBinPolicyMap when adding
// or reordering fields.
};
@@ -131,15 +132,7 @@ class CC_EXPORT ManagedTileState {
TileVersion tile_versions[NUM_RASTER_MODES];
RasterMode raster_mode;
- // Ephemeral state, valid only during TileManager::ManageTiles.
- bool is_in_never_bin_on_both_trees() const {
- return (bin[HIGH_PRIORITY_BIN] == NEVER_BIN ||
- bin[HIGH_PRIORITY_BIN] == NEVER_AND_ACTIVE_BIN) &&
- (bin[LOW_PRIORITY_BIN] == NEVER_BIN ||
- bin[LOW_PRIORITY_BIN] == NEVER_AND_ACTIVE_BIN);
- }
-
- ManagedTileBin bin[NUM_BIN_PRIORITIES];
+ ManagedTileBin bin;
ManagedTileBin tree_bin[NUM_TREES];
// The bin that the tile would have if the GPU memory manager had
« no previous file with comments | « no previous file | cc/resources/managed_tile_state.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698