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

Unified Diff: cc/tiles/gpu_image_decode_cache.h

Issue 2631453002: Revert of cc: Add image decode queue functionality to image manager. (Closed)
Patch Set: Created 3 years, 11 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 | « cc/tiles/decoded_image_tracker_unittest.cc ('k') | cc/tiles/gpu_image_decode_cache.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/tiles/gpu_image_decode_cache.h
diff --git a/cc/tiles/gpu_image_decode_cache.h b/cc/tiles/gpu_image_decode_cache.h
index 08dd6602a8387d82aec4c503444550ef758aa50d..34f8dca4c394d1cf8d805d06d66904377ebf3630 100644
--- a/cc/tiles/gpu_image_decode_cache.h
+++ b/cc/tiles/gpu_image_decode_cache.h
@@ -99,8 +99,6 @@
public base::trace_event::MemoryDumpProvider,
public base::MemoryCoordinatorClient {
public:
- enum class DecodeTaskType { PART_OF_UPLOAD_TASK, STAND_ALONE_DECODE_TASK };
-
explicit GpuImageDecodeCache(ContextProvider* context,
ResourceFormat decode_format,
size_t max_gpu_image_bytes);
@@ -113,9 +111,6 @@
bool GetTaskForImageAndRef(const DrawImage& image,
const TracingInfo& tracing_info,
scoped_refptr<TileTask>* task) override;
- bool GetOutOfRasterDecodeTaskForImageAndRef(
- const DrawImage& image,
- scoped_refptr<TileTask>* task) override;
void UnrefImage(const DrawImage& image) override;
DecodedDrawImage GetDecodedImageForDraw(const DrawImage& draw_image) override;
void DrawWithImageFinished(const DrawImage& image,
@@ -136,8 +131,7 @@
void UploadImage(const DrawImage& image);
// Called by Decode / Upload tasks when tasks are finished.
- void OnImageDecodeTaskCompleted(const DrawImage& image,
- DecodeTaskType task_type);
+ void OnImageDecodeTaskCompleted(const DrawImage& image);
void OnImageUploadTaskCompleted(const DrawImage& image);
// For testing only.
@@ -173,9 +167,6 @@
bool decode_failure = false;
// If non-null, this is the pending decode task for this image.
scoped_refptr<TileTask> task;
- // Similar to above, but only is generated if there is no associated upload
- // generated for this task (ie, this is an out-of-raster request for decode.
- scoped_refptr<TileTask> stand_alone_task;
private:
struct UsageStats {
@@ -271,15 +262,7 @@
// rather than the upload task, if necessary.
scoped_refptr<TileTask> GetImageDecodeTaskAndRef(
const DrawImage& image,
- const TracingInfo& tracing_info,
- DecodeTaskType task_type);
-
- // Note that this function behaves as if it was public (all of the same locks
- // need to be acquired).
- bool GetTaskForImageAndRefInternal(const DrawImage& image,
- const TracingInfo& tracing_info,
- DecodeTaskType task_type,
- scoped_refptr<TileTask>* task);
+ const TracingInfo& tracing_info);
void RefImageDecode(const DrawImage& draw_image);
void UnrefImageDecode(const DrawImage& draw_image);
« no previous file with comments | « cc/tiles/decoded_image_tracker_unittest.cc ('k') | cc/tiles/gpu_image_decode_cache.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698