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

Unified Diff: cc/layers/layer.h

Issue 19106007: cc: Allow the main thread to cancel commits (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Fix scheduler tests Created 7 years, 5 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 | « no previous file | cc/layers/layer.cc » ('j') | cc/layers/layer.cc » ('J')
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: cc/layers/layer.h
diff --git a/cc/layers/layer.h b/cc/layers/layer.h
index 077eb2c649af694078f53a3bd6941ed5a71bbf6a..72d88e145da41d13fa8e72c3a4c0e74c737fa14e 100644
--- a/cc/layers/layer.h
+++ b/cc/layers/layer.h
@@ -393,7 +393,17 @@ class CC_EXPORT Layer : public base::RefCounted<Layer>,
Layer();
+ // These SetNeeds functions are in order of severity of update:
+ //
+ // Called when this layer has been modified in some way, but isn't sure
+ // that it needs a commit yet. It needs CalcDrawProperties and UpdateLayers
+ // before it knows whether or not a commit is required.
+ void SetNeedsUpdate();
+ // Called when a property has been modified in a way that the layer
+ // knows immediately that a commit is required. This implies SetNeedsUpdate.
void SetNeedsCommit();
+ // Called when there's been a change in layer structure. Implies both
+ // SetNeedsUpdate and SetNeedsCommit.
danakj 2013/07/17 20:53:24 Kinda but not, there's no needs_push_properties()
enne (OOO) 2013/07/18 17:36:37 Done.
void SetNeedsFullTreeSync();
bool IsPropertyChangeAllowed() const;
« no previous file with comments | « no previous file | cc/layers/layer.cc » ('j') | cc/layers/layer.cc » ('J')

Powered by Google App Engine
This is Rietveld 408576698