Index: cc/trees/layer_tree_host_impl.cc |
diff --git a/cc/trees/layer_tree_host_impl.cc b/cc/trees/layer_tree_host_impl.cc |
index 5d5d750fd9e2dcfda7af9b9ce90401a251de20a9..40fcfd0cf10890ebc8bbd4a96cd1ce2e4c81f524 100644 |
--- a/cc/trees/layer_tree_host_impl.cc |
+++ b/cc/trees/layer_tree_host_impl.cc |
@@ -2011,62 +2011,86 @@ void LayerTreeHostImpl::CreateAndSetTileManager() { |
: proxy_->MainThreadTaskRunner(); |
DCHECK(task_runner); |
+ CreateResourceAndRasterWorkerPool( |
+ &raster_worker_pool_, &resource_pool_, &staging_resource_pool_); |
+ DCHECK(raster_worker_pool_); |
+ DCHECK(resource_pool_); |
+ |
+ tile_manager_ = TileManager::Create(this, |
+ task_runner, |
+ resource_pool_.get(), |
+ raster_worker_pool_->AsRasterizer(), |
+ rendering_stats_instrumentation_); |
+ |
+ UpdateTileManagerMemoryPolicy(ActualManagedMemoryPolicy()); |
+ need_to_update_visible_tiles_before_draw_ = false; |
+} |
+ |
+void LayerTreeHostImpl::CreateResourceAndRasterWorkerPool( |
+ scoped_ptr<RasterWorkerPool>* raster_worker_pool, |
+ scoped_ptr<ResourcePool>* resource_pool, |
+ scoped_ptr<ResourcePool>* staging_resource_pool) { |
+ base::SingleThreadTaskRunner* task_runner = |
+ proxy_->HasImplThread() ? proxy_->ImplThreadTaskRunner() |
+ : proxy_->MainThreadTaskRunner(); |
+ DCHECK(task_runner); |
+ |
ContextProvider* context_provider = output_surface_->context_provider(); |
if (!context_provider) { |
- resource_pool_ = |
+ *resource_pool = |
ResourcePool::Create(resource_provider_.get(), |
GL_TEXTURE_2D, |
resource_provider_->best_texture_format()); |
- raster_worker_pool_ = |
+ *raster_worker_pool = |
BitmapRasterWorkerPool::Create(proxy_->ImplThreadTaskRunner(), |
RasterWorkerPool::GetTaskGraphRunner(), |
resource_provider_.get()); |
} else if (use_gpu_rasterization_) { |
- resource_pool_ = |
+ *resource_pool = |
ResourcePool::Create(resource_provider_.get(), |
GL_TEXTURE_2D, |
resource_provider_->best_texture_format()); |
- raster_worker_pool_ = |
+ *raster_worker_pool = |
GpuRasterWorkerPool::Create(task_runner, |
context_provider, |
resource_provider_.get(), |
settings_.use_distance_field_text); |
- } else if (UseZeroCopyRasterizer()) { |
- resource_pool_ = ResourcePool::Create( |
+ } else if (settings_.use_zero_copy && CanUseZeroCopyRasterizer()) { |
+ *resource_pool = ResourcePool::Create( |
resource_provider_.get(), |
GetMapImageTextureTarget(context_provider->ContextCapabilities()), |
resource_provider_->best_texture_format()); |
- raster_worker_pool_ = |
+ *raster_worker_pool = |
ZeroCopyRasterWorkerPool::Create(proxy_->ImplThreadTaskRunner(), |
RasterWorkerPool::GetTaskGraphRunner(), |
resource_provider_.get()); |
- } else if (UseOneCopyRasterizer()) { |
+ } else if (settings_.use_one_copy && CanUseOneCopyRasterizer()) { |
// We need to create a staging resource pool when using copy rasterizer. |
- staging_resource_pool_ = ResourcePool::Create( |
+ *staging_resource_pool = ResourcePool::Create( |
resource_provider_.get(), |
GetMapImageTextureTarget(context_provider->ContextCapabilities()), |
resource_provider_->best_texture_format()); |
- resource_pool_ = |
+ *resource_pool = |
ResourcePool::Create(resource_provider_.get(), |
GL_TEXTURE_2D, |
resource_provider_->best_texture_format()); |
- raster_worker_pool_ = |
+ *raster_worker_pool = |
OneCopyRasterWorkerPool::Create(task_runner, |
RasterWorkerPool::GetTaskGraphRunner(), |
context_provider, |
resource_provider_.get(), |
staging_resource_pool_.get()); |
} else { |
- resource_pool_ = ResourcePool::Create( |
+ *resource_pool = ResourcePool::Create( |
resource_provider_.get(), |
GL_TEXTURE_2D, |
resource_provider_->memory_efficient_texture_format()); |
- raster_worker_pool_ = PixelBufferRasterWorkerPool::Create( |
+ *raster_worker_pool = PixelBufferRasterWorkerPool::Create( |
task_runner, |
RasterWorkerPool::GetTaskGraphRunner(), |
context_provider, |
@@ -2074,15 +2098,6 @@ void LayerTreeHostImpl::CreateAndSetTileManager() { |
GetMaxTransferBufferUsageBytes(context_provider->ContextCapabilities(), |
settings_.refresh_rate)); |
} |
- |
- tile_manager_ = TileManager::Create(this, |
- task_runner, |
- resource_pool_.get(), |
- raster_worker_pool_->AsRasterizer(), |
- rendering_stats_instrumentation_); |
- |
- UpdateTileManagerMemoryPolicy(ActualManagedMemoryPolicy()); |
- need_to_update_visible_tiles_before_draw_ = false; |
} |
void LayerTreeHostImpl::DestroyTileManager() { |
@@ -2098,13 +2113,13 @@ bool LayerTreeHostImpl::UsePendingTreeForSync() const { |
return settings_.impl_side_painting; |
} |
-bool LayerTreeHostImpl::UseZeroCopyRasterizer() const { |
- return settings_.use_zero_copy && GetRendererCapabilities().using_image; |
+bool LayerTreeHostImpl::CanUseZeroCopyRasterizer() const { |
+ return GetRendererCapabilities().using_image; |
} |
-bool LayerTreeHostImpl::UseOneCopyRasterizer() const { |
+bool LayerTreeHostImpl::CanUseOneCopyRasterizer() const { |
// Sync query support is required by one-copy rasterizer. |
- return settings_.use_one_copy && GetRendererCapabilities().using_image && |
+ return GetRendererCapabilities().using_image && |
resource_provider_->use_sync_query(); |
} |