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

Unified Diff: Source/core/paint/DeprecatedPaintLayerStackingNodeIterator.h

Issue 1327483003: Add documentation to 2 DeprecatedPaintLayer objects (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Created 5 years, 4 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: Source/core/paint/DeprecatedPaintLayerStackingNodeIterator.h
diff --git a/Source/core/paint/DeprecatedPaintLayerStackingNodeIterator.h b/Source/core/paint/DeprecatedPaintLayerStackingNodeIterator.h
index cc5d209e69a5dc6c507483c02ae64929e0f50276..6dde5f8918d0c2595b2c77d8648e337f5416ebd7 100644
--- a/Source/core/paint/DeprecatedPaintLayerStackingNodeIterator.h
+++ b/Source/core/paint/DeprecatedPaintLayerStackingNodeIterator.h
@@ -38,6 +38,10 @@ namespace blink {
enum ChildrenIteration {
NegativeZOrderChildren = 1,
+ // Normal flow children are not mandated by CSS 2.1 but are an artifact of
+ // our implementation: we allocate DeprecatedPaintLayers for elements that
+ // are not treated as stacking contexts and thus we need to walk them
mstensho (USE GERRIT) 2015/09/01 18:59:06 I don't think "stacking contexts" is the right ter
Julien - ping for review 2015/09/03 14:25:54 The CSS word for stacking context + positioned is
mstensho (USE GERRIT) 2015/09/04 13:22:32 OK. Lacking a better word for a true stacking cont
+ // during painting and hit-testing.
NormalFlowChildren = 1 << 1,
PositiveZOrderChildren = 1 << 2,
AllChildren = NegativeZOrderChildren | NormalFlowChildren | PositiveZOrderChildren

Powered by Google App Engine
This is Rietveld 408576698