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

Unified Diff: cc/resources/resource_provider.cc

Issue 328263005: cc: Rename DirectRasterWorkerPool to GpuRasterWorkerPool (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: renaming DirectRasterBuffer and related apis Created 6 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/resources/resource_provider.cc
diff --git a/cc/resources/resource_provider.cc b/cc/resources/resource_provider.cc
index e3c21ff7c6b225ef2d207437a0748c465c37fe81..e26f9ed220bf5a7bdeb026368d4be9f6c8a8eaa5 100644
--- a/cc/resources/resource_provider.cc
+++ b/cc/resources/resource_provider.cc
@@ -429,31 +429,33 @@ bool ResourceProvider::RasterBuffer::UnlockForWrite() {
return DoUnlockForWrite();
}
-ResourceProvider::DirectRasterBuffer::DirectRasterBuffer(
+ResourceProvider::GpuRasterBuffer::GpuRasterBuffer(
const Resource* resource,
ResourceProvider* resource_provider,
- bool use_distance_field_text )
+ bool use_distance_field_text)
: RasterBuffer(resource, resource_provider),
surface_generation_id_(0u),
- use_distance_field_text_(use_distance_field_text) {}
+ use_distance_field_text_(use_distance_field_text) {
+}
-ResourceProvider::DirectRasterBuffer::~DirectRasterBuffer() {}
+ResourceProvider::GpuRasterBuffer::~GpuRasterBuffer() {
+}
-SkCanvas* ResourceProvider::DirectRasterBuffer::DoLockForWrite() {
+SkCanvas* ResourceProvider::GpuRasterBuffer::DoLockForWrite() {
if (!surface_)
surface_ = CreateSurface();
surface_generation_id_ = surface_ ? surface_->generationID() : 0u;
return surface_ ? surface_->getCanvas() : NULL;
}
-bool ResourceProvider::DirectRasterBuffer::DoUnlockForWrite() {
+bool ResourceProvider::GpuRasterBuffer::DoUnlockForWrite() {
// generationID returns a non-zero, unique value corresponding to the content
// of surface. Hence, a change since DoLockForWrite was called means the
// surface has changed.
return surface_ ? surface_generation_id_ != surface_->generationID() : false;
}
-skia::RefPtr<SkSurface> ResourceProvider::DirectRasterBuffer::CreateSurface() {
+skia::RefPtr<SkSurface> ResourceProvider::GpuRasterBuffer::CreateSurface() {
skia::RefPtr<SkSurface> surface;
switch (resource()->type) {
case GLTexture: {
@@ -842,7 +844,7 @@ void ResourceProvider::DeleteResourceInternal(ResourceMap::iterator it,
if (style == ForShutdown && resource->exported_count > 0)
lost_resource = true;
- resource->direct_raster_buffer.reset();
+ resource->gpu_raster_buffer.reset();
resource->image_raster_buffer.reset();
resource->pixel_raster_buffer.reset();
@@ -1753,22 +1755,22 @@ void ResourceProvider::DeleteAndReturnUnusedResourcesToChild(
}
}
-SkCanvas* ResourceProvider::MapDirectRasterBuffer(ResourceId id) {
- // Resource needs to be locked for write since DirectRasterBuffer writes
+SkCanvas* ResourceProvider::MapGpuRasterBuffer(ResourceId id) {
+ // Resource needs to be locked for write since GpuRasterBuffer writes
// directly to it.
LockForWrite(id);
Resource* resource = GetResource(id);
- if (!resource->direct_raster_buffer.get()) {
- resource->direct_raster_buffer.reset(
- new DirectRasterBuffer(resource, this, use_distance_field_text_));
+ if (!resource->gpu_raster_buffer.get()) {
+ resource->gpu_raster_buffer.reset(
+ new GpuRasterBuffer(resource, this, use_distance_field_text_));
}
- return resource->direct_raster_buffer->LockForWrite();
+ return resource->gpu_raster_buffer->LockForWrite();
}
-void ResourceProvider::UnmapDirectRasterBuffer(ResourceId id) {
+void ResourceProvider::UnmapGpuRasterBuffer(ResourceId id) {
Resource* resource = GetResource(id);
- DCHECK(resource->direct_raster_buffer.get());
- resource->direct_raster_buffer->UnlockForWrite();
+ DCHECK(resource->gpu_raster_buffer.get());
+ resource->gpu_raster_buffer->UnlockForWrite();
UnlockForWrite(id);
}

Powered by Google App Engine
This is Rietveld 408576698