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

Side by Side Diff: cc/tiles/image_decode_controller.h

Issue 1866043006: cc: Remove ScheduleOnOriginThread() and CompleteOnOriginThread(). (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: wip - fixed few unit tests Created 4 years, 8 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 unified diff | Download patch
OLDNEW
1 // Copyright 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CC_TILES_IMAGE_DECODE_CONTROLLER_H_ 5 #ifndef CC_TILES_IMAGE_DECODE_CONTROLLER_H_
6 #define CC_TILES_IMAGE_DECODE_CONTROLLER_H_ 6 #define CC_TILES_IMAGE_DECODE_CONTROLLER_H_
7 7
8 #include "base/memory/ref_counted.h" 8 #include "base/memory/ref_counted.h"
9 #include "cc/playback/decoded_draw_image.h" 9 #include "cc/playback/decoded_draw_image.h"
10 #include "cc/playback/draw_image.h" 10 #include "cc/playback/draw_image.h"
11 11
12 namespace cc { 12 namespace cc {
13 13
14 class ImageDecodeTask; 14 class ImageDecodeTask;
15 class Task;
15 16
16 // ImageDecodeController is responsible for generating decode tasks, decoding 17 // ImageDecodeController is responsible for generating decode tasks, decoding
17 // images, storing images in cache, and being able to return the decoded images 18 // images, storing images in cache, and being able to return the decoded images
18 // when requested. 19 // when requested.
19 20
20 // ImageDecodeController is responsible for the following things: 21 // ImageDecodeController is responsible for the following things:
21 // 1. Given a DrawImage, it can return an ImageDecodeTask which when run will 22 // 1. Given a DrawImage, it can return an ImageDecodeTask which when run will
22 // decode and cache the resulting image. If the image does not need a task to 23 // decode and cache the resulting image. If the image does not need a task to
23 // be decoded, then nullptr will be returned. The return value of the 24 // be decoded, then nullptr will be returned. The return value of the
24 // function indicates whether the image was or is going to be locked, so an 25 // function indicates whether the image was or is going to be locked, so an
(...skipping 27 matching lines...) Expand all
52 // predecoded. 53 // predecoded.
53 // 54 //
54 // This is called by a raster task (on a worker thread) when an image is 55 // This is called by a raster task (on a worker thread) when an image is
55 // required. 56 // required.
56 virtual DecodedDrawImage GetDecodedImageForDraw(const DrawImage& image) = 0; 57 virtual DecodedDrawImage GetDecodedImageForDraw(const DrawImage& image) = 0;
57 // Unrefs an image. This should be called for every GetDecodedImageForDraw 58 // Unrefs an image. This should be called for every GetDecodedImageForDraw
58 // when the draw with the image is finished. 59 // when the draw with the image is finished.
59 virtual void DrawWithImageFinished(const DrawImage& image, 60 virtual void DrawWithImageFinished(const DrawImage& image,
60 const DecodedDrawImage& decoded_image) = 0; 61 const DecodedDrawImage& decoded_image) = 0;
61 62
63 virtual void CompleteTask(Task* task) = 0;
64
62 // This function informs the controller that now is a good time to clean up 65 // This function informs the controller that now is a good time to clean up
63 // memory. This is called periodically from the compositor thread. 66 // memory. This is called periodically from the compositor thread.
64 virtual void ReduceCacheUsage() = 0; 67 virtual void ReduceCacheUsage() = 0;
65 }; 68 };
66 69
67 } // namespace cc 70 } // namespace cc
68 71
69 #endif // CC_TILES_IMAGE_DECODE_CONTROLLER_H_ 72 #endif // CC_TILES_IMAGE_DECODE_CONTROLLER_H_
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698