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

Unified Diff: cc/resources/image_copy_raster_worker_pool.cc

Issue 375303002: cc: Refactor ResourceProvider. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: review comments addressed Created 6 years, 5 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/image_copy_raster_worker_pool.cc
diff --git a/cc/resources/image_copy_raster_worker_pool.cc b/cc/resources/image_copy_raster_worker_pool.cc
index 0d4f7441e5222be3b78646df74e05dd6e98dbbae..fb0e31cf6442b80c4dd54ec799f96eeb76718b9d 100644
--- a/cc/resources/image_copy_raster_worker_pool.cc
+++ b/cc/resources/image_copy_raster_worker_pool.cc
@@ -10,6 +10,9 @@
#include "cc/debug/traced_value.h"
#include "cc/resources/resource_pool.h"
#include "cc/resources/scoped_resource.h"
+#include "gpu/command_buffer/client/gles2_interface.h"
+
+using gpu::gles2::GLES2Interface;
danakj 2014/07/14 15:29:11 not needed?
sohanjg 2014/07/14 15:59:28 Done.
namespace cc {
@@ -17,26 +20,34 @@ namespace cc {
scoped_ptr<RasterWorkerPool> ImageCopyRasterWorkerPool::Create(
base::SequencedTaskRunner* task_runner,
TaskGraphRunner* task_graph_runner,
+ ContextProvider* context_provider,
ResourceProvider* resource_provider,
ResourcePool* resource_pool) {
- return make_scoped_ptr<RasterWorkerPool>(new ImageCopyRasterWorkerPool(
- task_runner, task_graph_runner, resource_provider, resource_pool));
+ return make_scoped_ptr<RasterWorkerPool>(
+ new ImageCopyRasterWorkerPool(task_runner,
+ task_graph_runner,
+ context_provider,
+ resource_provider,
+ resource_pool));
}
ImageCopyRasterWorkerPool::ImageCopyRasterWorkerPool(
base::SequencedTaskRunner* task_runner,
TaskGraphRunner* task_graph_runner,
+ ContextProvider* context_provider,
ResourceProvider* resource_provider,
ResourcePool* resource_pool)
: task_runner_(task_runner),
task_graph_runner_(task_graph_runner),
namespace_token_(task_graph_runner->GetNamespaceToken()),
+ context_provider_(context_provider),
resource_provider_(resource_provider),
resource_pool_(resource_pool),
has_performed_copy_since_last_flush_(false),
raster_tasks_pending_(false),
raster_tasks_required_for_activation_pending_(false),
- raster_finished_weak_ptr_factory_(this) {}
+ raster_finished_weak_ptr_factory_(this) {
+}
ImageCopyRasterWorkerPool::~ImageCopyRasterWorkerPool() {
DCHECK_EQ(0u, raster_task_states_.size());
@@ -226,7 +237,8 @@ void ImageCopyRasterWorkerPool::FlushCopies() {
if (!has_performed_copy_since_last_flush_)
return;
- resource_provider_->ShallowFlushIfSupported();
+ if (context_provider_)
+ context_provider_->ContextGL()->ShallowFlushCHROMIUM();
has_performed_copy_since_last_flush_ = false;
}

Powered by Google App Engine
This is Rietveld 408576698