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

Unified Diff: cc/tile_manager.h

Issue 12194015: cc: Rasterize cheap tiles immediately (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Fix picture pile reference management. Post a task for running cheap tasks. Created 7 years, 10 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/tile_manager.h
diff --git a/cc/tile_manager.h b/cc/tile_manager.h
index e05217b5b527ab0ee2edfd31d6fd25658a9b1015..dfd2cfdceb9297ff5d174c6e790ccd1207e95424 100644
--- a/cc/tile_manager.h
+++ b/cc/tile_manager.h
@@ -120,6 +120,7 @@ class CC_EXPORT TileManager : public WorkerPoolClient {
void ManageTiles();
void CheckForCompletedTileUploads();
void AbortPendingTileUploads();
+ void DidCompleteFrame();
scoped_ptr<base::Value> BasicStateAsValue() const;
scoped_ptr<base::Value> AllTilesAsValue() const;
@@ -167,10 +168,9 @@ class CC_EXPORT TileManager : public WorkerPoolClient {
void OnImageDecodeTaskCompleted(
scoped_refptr<Tile> tile,
uint32_t pixel_ref_id);
- bool CanDispatchRasterTask(Tile* tile);
+ bool CanDispatchRasterTask(Tile* tile) const;
scoped_ptr<ResourcePool::Resource> PrepareTileForRaster(Tile* tile);
void DispatchOneRasterTask(scoped_refptr<Tile> tile);
- void PerformOneRaster(Tile* tile);
void OnRasterCompleted(
reveman 2013/02/19 18:05:23 we don't need OnRasterCompleted anymore, right? pl
Sami 2013/02/19 22:24:38 Done.
scoped_refptr<Tile> tile,
scoped_ptr<ResourcePool::Resource> resource,
@@ -193,7 +193,7 @@ class CC_EXPORT TileManager : public WorkerPoolClient {
const RasterTaskMetadata& raster_task_metadata,
PicturePileImpl* picture_pile,
RenderingStats* stats);
- static void RunImageDecodeTask(skia::LazyPixelRef* pixel_ref,
+ static void PerformImageDecode(skia::LazyPixelRef* pixel_ref,
reveman 2013/02/19 18:05:23 Lets change PerformRaster back to RunRasterTask in
Sami 2013/02/19 22:24:38 Done.
RenderingStats* stats);
static void RecordCheapnessPredictorResults(bool is_predicted_cheap,
@@ -232,6 +232,8 @@ class CC_EXPORT TileManager : public WorkerPoolClient {
RenderingStats rendering_stats_;
bool use_cheapness_estimator_;
+ bool did_schedule_cheap_tasks_;
+ bool allow_cheap_tasks_;
int raster_state_count_[NUM_STATES][NUM_TREES][NUM_BINS];
DISALLOW_COPY_AND_ASSIGN(TileManager);

Powered by Google App Engine
This is Rietveld 408576698