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

Unified Diff: cc/raster/one_copy_tile_task_worker_pool.cc

Issue 1248253003: Add better support for RGBA_4444 textures. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 5 years, 4 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
« no previous file with comments | « no previous file | cc/raster/tile_task_worker_pool.cc » ('j') | cc/raster/tile_task_worker_pool.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/raster/one_copy_tile_task_worker_pool.cc
diff --git a/cc/raster/one_copy_tile_task_worker_pool.cc b/cc/raster/one_copy_tile_task_worker_pool.cc
index 51e451019ec28aa34c345b9ab224f17b4543f335..762ea68e880bb7b9df09a1d2ffb7762e1bb5e444 100644
--- a/cc/raster/one_copy_tile_task_worker_pool.cc
+++ b/cc/raster/one_copy_tile_task_worker_pool.cc
@@ -324,7 +324,7 @@ void OneCopyTileTaskWorkerPool::CheckForCompletedTasks() {
}
ResourceFormat OneCopyTileTaskWorkerPool::GetResourceFormat() const {
- return resource_provider_->best_texture_format();
+ return resource_provider_->memory_efficient_texture_format();
}
bool OneCopyTileTaskWorkerPool::GetResourceRequiresSwizzle() const {
@@ -337,10 +337,12 @@ scoped_ptr<RasterBuffer> OneCopyTileTaskWorkerPool::AcquireBufferForRaster(
uint64_t previous_content_id) {
// TODO(danakj): If resource_content_id != 0, we only need to copy/upload
// the dirty rect.
- DCHECK_EQ(resource->format(), resource_provider_->best_texture_format());
+ DCHECK_EQ(resource->format(),
+ resource_provider_->memory_efficient_texture_format());
return make_scoped_ptr<RasterBuffer>(new RasterBufferImpl(
- this, resource_provider_, resource_provider_->best_texture_format(),
- resource, previous_content_id));
+ this, resource_provider_,
+ resource_provider_->memory_efficient_texture_format(), resource,
+ previous_content_id));
}
void OneCopyTileTaskWorkerPool::ReleaseBufferForRaster(
« no previous file with comments | « no previous file | cc/raster/tile_task_worker_pool.cc » ('j') | cc/raster/tile_task_worker_pool.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698