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

Unified Diff: cc/trees/layer_tree_host_impl.h

Issue 1134123005: cc: split UpdateGpuRasterizationStatus() into two parts. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 5 years, 7 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/trees/layer_tree_host_impl.h
diff --git a/cc/trees/layer_tree_host_impl.h b/cc/trees/layer_tree_host_impl.h
index 54776e77315b74a08c74c23351547aeb77b6bc73..26cc8ae8f0eabba5a43e8c52c35ccadec89deaf7 100644
--- a/cc/trees/layer_tree_host_impl.h
+++ b/cc/trees/layer_tree_host_impl.h
@@ -323,11 +323,13 @@ class CC_EXPORT LayerTreeHostImpl
void set_has_gpu_rasterization_trigger(bool flag) {
Stephen White 2015/05/13 19:44:17 This should probably be CamelCased, now that it's
danakj 2015/05/13 19:50:58 this isn't a simple setter anymore, no hacker case
Stephen White 2015/05/13 20:15:20 Done.
has_gpu_rasterization_trigger_ = flag;
+ UpdateGpuRasterizationStatus();
}
void set_content_is_suitable_for_gpu_rasterization(bool flag) {
Stephen White 2015/05/13 19:44:17 Same here.
danakj 2015/05/13 19:50:58 dittos
Stephen White 2015/05/13 20:15:20 Done.
content_is_suitable_for_gpu_rasterization_ = flag;
+ UpdateGpuRasterizationStatus();
}
- void UpdateGpuRasterizationStatus();
+ void UpdateTreeResourcesIfNeeded();
danakj 2015/05/13 19:50:58 UpdateTreeResourcesForGpuRasterizationIfNeeded?
Stephen White 2015/05/13 20:15:20 Done.
bool use_gpu_rasterization() const { return use_gpu_rasterization_; }
bool use_msaa() const { return use_msaa_; }
@@ -571,6 +573,8 @@ class CC_EXPORT LayerTreeHostImpl
void ReleaseTreeResources();
void RecreateTreeResources();
+ void UpdateGpuRasterizationStatus();
+
bool IsSynchronousSingleThreaded() const;
Viewport* viewport() { return viewport_.get(); }
@@ -646,6 +650,7 @@ class CC_EXPORT LayerTreeHostImpl
bool use_gpu_rasterization_;
bool use_msaa_;
GpuRasterizationStatus gpu_rasterization_status_;
+ bool tree_resources_dirty_;
danakj 2015/05/13 19:50:58 _for_gpu_rasterization_
Stephen White 2015/05/13 20:15:20 Done.
scoped_ptr<TileTaskWorkerPool> tile_task_worker_pool_;
scoped_ptr<ResourcePool> resource_pool_;
scoped_ptr<ResourcePool> staging_resource_pool_;

Powered by Google App Engine
This is Rietveld 408576698