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

Unified Diff: cc/resources/tile_manager.cc

Issue 69343005: Added preliminary support for tile rasterization with Ganesh (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fix cc_perftests compile error. Created 7 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/tile_manager.cc
diff --git a/cc/resources/tile_manager.cc b/cc/resources/tile_manager.cc
index b443516887c8e65c6c0ba464dec3820ef926042b..a1509dd65d983728744f38d8f848e5e16576658f 100644
--- a/cc/resources/tile_manager.cc
+++ b/cc/resources/tile_manager.cc
@@ -168,6 +168,7 @@ scoped_ptr<base::Value> RasterTaskCompletionStatsAsValue(
scoped_ptr<TileManager> TileManager::Create(
TileManagerClient* client,
ResourceProvider* resource_provider,
+ ContextProvider* context_provider,
size_t num_raster_threads,
RenderingStatsInstrumentation* rendering_stats_instrumentation,
bool use_map_image,
@@ -177,6 +178,7 @@ scoped_ptr<TileManager> TileManager::Create(
return make_scoped_ptr(
new TileManager(client,
resource_provider,
+ context_provider,
use_map_image ?
ImageRasterWorkerPool::Create(
resource_provider,
@@ -194,6 +196,7 @@ scoped_ptr<TileManager> TileManager::Create(
TileManager::TileManager(
TileManagerClient* client,
ResourceProvider* resource_provider,
+ ContextProvider* context_provider,
scoped_ptr<RasterWorkerPool> raster_worker_pool,
size_t num_raster_threads,
size_t max_raster_usage_bytes,
@@ -204,6 +207,8 @@ TileManager::TileManager(
raster_worker_pool->GetResourceTarget(),
raster_worker_pool->GetResourceFormat())),
raster_worker_pool_(raster_worker_pool.Pass()),
+ gpu_rasterizer_(GpuRasterizer::Create(
+ context_provider, resource_provider)),
prioritized_tiles_dirty_(false),
all_tiles_that_need_to_be_rasterized_have_memory_(true),
all_tiles_required_for_activation_have_memory_(true),
@@ -217,6 +222,7 @@ TileManager::TileManager(
did_initialize_visible_tile_(false),
did_check_for_completed_tasks_since_last_schedule_tasks_(true) {
raster_worker_pool_->SetClient(this);
+ gpu_rasterizer_->SetClient(this);
}
TileManager::~TileManager() {
@@ -816,10 +822,14 @@ void TileManager::ScheduleTasks(
DCHECK(tile_version.requires_resource());
DCHECK(!tile_version.resource_);
- if (tile_version.raster_task_.is_null())
- tile_version.raster_task_ = CreateRasterTask(tile);
+ if (tile->use_gpu_rasterizer()) {
+ CreateGpuRasterTask(tile);
+ } else {
+ if (tile_version.raster_task_.is_null())
+ tile_version.raster_task_ = CreateRasterTask(tile);
- tasks.Append(tile_version.raster_task_, tile->required_for_activation());
+ tasks.Append(tile_version.raster_task_, tile->required_for_activation());
+ }
}
// We must reduce the amount of unused resoruces before calling
@@ -832,6 +842,8 @@ void TileManager::ScheduleTasks(
raster_worker_pool_->ScheduleTasks(&tasks);
did_check_for_completed_tasks_since_last_schedule_tasks_ = false;
+
+ gpu_rasterizer_->FlushRasterTasks(rendering_stats_instrumentation_);
}
RasterWorkerPool::Task TileManager::CreateImageDecodeTask(
@@ -896,6 +908,12 @@ RasterWorkerPool::RasterTask TileManager::CreateRasterTask(Tile* tile) {
&decode_tasks);
}
+void TileManager::CreateGpuRasterTask(Tile* tile) {
+ scoped_ptr<ResourcePool::Resource> resource =
+ resource_pool_->AcquireResource(tile->tile_size_.size());
+ gpu_rasterizer_->PushRasterTask(tile, resource.Pass());
+}
+
void TileManager::OnImageDecodeTaskCompleted(
int layer_id,
skia::LazyPixelRef* pixel_ref,
@@ -964,6 +982,34 @@ void TileManager::OnRasterTaskCompleted(
did_initialize_visible_tile_ = true;
}
+void TileManager::OnGpuRasterTaskCompleted(
+ Tile* tile,
+ scoped_ptr<ResourcePool::Resource> resource,
+ bool was_canceled) {
+ ManagedTileState& mts = tile->managed_state();
+ ManagedTileState::TileVersion& tile_version =
+ mts.tile_versions[mts.raster_mode];
+ DCHECK(tile_version.raster_task_.is_null());
+
+ if (was_canceled) {
+ ++update_visible_tiles_stats_.canceled_count;
+ resource_pool_->ReleaseResource(resource.Pass());
+ return;
+ }
+
+ ++update_visible_tiles_stats_.completed_count;
+
+ tile_version.set_use_resource();
+ tile_version.resource_ = resource.Pass();
+
+ bytes_releasable_ += BytesConsumedIfAllocated(tile);
+ ++resources_releasable_;
+
+ FreeUnusedResourcesForTile(tile);
+ if (tile->priority(ACTIVE_TREE).distance_to_visible_in_pixels == 0)
+ did_initialize_visible_tile_ = true;
+}
+
scoped_refptr<Tile> TileManager::CreateTile(PicturePileImpl* picture_pile,
gfx::Size tile_size,
gfx::Rect content_rect,
@@ -971,7 +1017,8 @@ scoped_refptr<Tile> TileManager::CreateTile(PicturePileImpl* picture_pile,
float contents_scale,
int layer_id,
int source_frame_number,
- bool can_use_lcd_text) {
+ bool can_use_lcd_text,
+ bool use_gpu_rasterizer) {
scoped_refptr<Tile> tile = make_scoped_refptr(new Tile(this,
picture_pile,
tile_size,
@@ -980,7 +1027,8 @@ scoped_refptr<Tile> TileManager::CreateTile(PicturePileImpl* picture_pile,
contents_scale,
layer_id,
source_frame_number,
- can_use_lcd_text));
+ can_use_lcd_text,
+ use_gpu_rasterizer));
DCHECK(tiles_.find(tile->id()) == tiles_.end());
tiles_[tile->id()] = tile;

Powered by Google App Engine
This is Rietveld 408576698