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

Unified Diff: cc/tiles/tile_task_manager.cc

Issue 1951193002: cc: Add mailbox support to ResourceProvider write locks. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@worker_context_stream
Patch Set: cleanup tasks if ordering barrier fails Created 4 years, 6 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/tiles/tile_task_manager.cc
diff --git a/cc/tiles/tile_task_manager.cc b/cc/tiles/tile_task_manager.cc
index aa6466ff35de1b30e8348e52564d905c9a00c901..b57b0572d1af978d4b265581b5473f228f351f6d 100644
--- a/cc/tiles/tile_task_manager.cc
+++ b/cc/tiles/tile_task_manager.cc
@@ -15,25 +15,19 @@ TileTaskManager::~TileTaskManager() {}
// static
std::unique_ptr<TileTaskManagerImpl> TileTaskManagerImpl::Create(
- std::unique_ptr<RasterBufferProvider> raster_buffer_provider,
TaskGraphRunner* task_graph_runner) {
- return base::WrapUnique<TileTaskManagerImpl>(new TileTaskManagerImpl(
- std::move(raster_buffer_provider), task_graph_runner));
+ return base::WrapUnique<TileTaskManagerImpl>(
+ new TileTaskManagerImpl(task_graph_runner));
}
-TileTaskManagerImpl::TileTaskManagerImpl(
- std::unique_ptr<RasterBufferProvider> raster_buffer_provider,
- TaskGraphRunner* task_graph_runner)
- : raster_buffer_provider_(std::move(raster_buffer_provider)),
- task_graph_runner_(task_graph_runner),
+TileTaskManagerImpl::TileTaskManagerImpl(TaskGraphRunner* task_graph_runner)
+ : task_graph_runner_(task_graph_runner),
namespace_token_(task_graph_runner->GetNamespaceToken()) {}
TileTaskManagerImpl::~TileTaskManagerImpl() {}
void TileTaskManagerImpl::ScheduleTasks(TaskGraph* graph) {
TRACE_EVENT0("cc", "TileTaskManagerImpl::ScheduleTasks");
-
- raster_buffer_provider_->OrderingBarrier();
task_graph_runner_->ScheduleTasks(namespace_token_, graph);
}
@@ -57,12 +51,6 @@ void TileTaskManagerImpl::Shutdown() {
TaskGraph empty;
task_graph_runner_->ScheduleTasks(namespace_token_, &empty);
task_graph_runner_->WaitForTasksToFinishRunning(namespace_token_);
-
- raster_buffer_provider_->Shutdown();
-}
-
-RasterBufferProvider* TileTaskManagerImpl::GetRasterBufferProvider() const {
- return raster_buffer_provider_.get();
}
} // namespace cc

Powered by Google App Engine
This is Rietveld 408576698