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

Unified Diff: cc/quads/render_pass.h

Issue 2873593002: Force use of and cache render surface. (Closed)
Patch Set: Calculate damage of |force_render_surface|. Created 3 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/quads/render_pass.h
diff --git a/cc/quads/render_pass.h b/cc/quads/render_pass.h
index 0ad8debc8928f0c5d07506d39a0dbd11aa6e070c..84677ee5609236ec69bd3a681ac1bfc8b084263f 100644
--- a/cc/quads/render_pass.h
+++ b/cc/quads/render_pass.h
@@ -85,7 +85,10 @@ class CC_EXPORT RenderPass {
const FilterOperations& filters,
const FilterOperations& background_filters,
const gfx::ColorSpace& color_space,
- bool has_transparent_background);
+ bool has_transparent_background,
+ bool force_render_surface,
+ bool has_property_change_on_contributing_render_surface,
+ bool has_damage_on_surface_quad);
void AsValueInto(base::trace_event::TracedValue* dict) const;
@@ -127,6 +130,14 @@ class CC_EXPORT RenderPass {
// If false, the pixels in the render pass' texture are all opaque.
bool has_transparent_background = true;
+ // If true we might reuse the texture if there is no damage.
+ bool force_render_surface = false;
+ // A cumulated property changes from the contributing render surface. Not
+ // including property changes on itself.
+ bool has_property_change_on_contributing_render_surface = false;
weiliangc 2017/06/07 21:19:00 We should be able to just combine this and has_dam
wutao 2017/06/09 02:31:32 Nice!
+ // If true, indicates containing at least one surface quad with damage.
+ bool has_damage_on_surface_quad = false;
+
// If non-empty, the renderer should produce a copy of the render pass'
// contents as a bitmap, and give a copy of the bitmap to each callback in
// this list. This property should not be serialized between compositors, as

Powered by Google App Engine
This is Rietveld 408576698