| 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();
|
| }
|
|
|
|
|