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

Unified Diff: cc/resources/tile.cc

Issue 603683006: cc: Remove low quality mode and cleanup tile versions. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: update Created 6 years, 3 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/resources/tile.cc
diff --git a/cc/resources/tile.cc b/cc/resources/tile.cc
index c0db9d4ac2c5d87123f6a5e5345560447cd060f5..8b4805a2fbfc3501659f1382006161bf2f36af5b 100644
--- a/cc/resources/tile.cc
+++ b/cc/resources/tile.cc
@@ -91,35 +91,11 @@ void Tile::AsValueInto(base::debug::TracedValue* res) const {
}
size_t Tile::GPUMemoryUsageInBytes() const {
- size_t total_size = 0;
- for (int mode = 0; mode < NUM_RASTER_MODES; ++mode)
- total_size += managed_state_.tile_versions[mode].GPUMemoryUsageInBytes();
- return total_size;
-}
-
-RasterMode Tile::DetermineRasterModeForTree(WhichTree tree) const {
- return DetermineRasterModeForResolution(priority(tree).resolution);
-}
-
-RasterMode Tile::DetermineOverallRasterMode() const {
- return DetermineRasterModeForResolution(managed_state_.resolution);
-}
-
-RasterMode Tile::DetermineRasterModeForResolution(
- TileResolution resolution) const {
- RasterMode current_mode = managed_state_.raster_mode;
- RasterMode raster_mode = resolution == LOW_RESOLUTION
- ? LOW_QUALITY_RASTER_MODE
- : HIGH_QUALITY_RASTER_MODE;
- return std::min(raster_mode, current_mode);
+ return managed_state_.draw_info_.GPUMemoryUsageInBytes();
}
bool Tile::HasRasterTask() const {
- for (int mode = 0; mode < NUM_RASTER_MODES; ++mode) {
- if (managed_state_.tile_versions[mode].raster_task_.get())
- return true;
- }
- return false;
+ return !!managed_state_.draw_info_.raster_task_.get();
}
} // namespace cc

Powered by Google App Engine
This is Rietveld 408576698