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

Side by Side Diff: Source/core/rendering/compositing/CompositedLayerMapping.h

Issue 266473009: Fix & clarify some comments in compositing. Rename clippedByAncestor to more accurately (Closed) Base URL: svn://svn.chromium.org/blink/trunk
Patch Set: Make conditionals clearer in shouldClipCompositedBounds Created 6 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 unified diff | Download patch | Annotate | Revision Log
OLDNEW
1 /* 1 /*
2 * Copyright (C) 2009, 2010, 2011 Apple Inc. All rights reserved. 2 * Copyright (C) 2009, 2010, 2011 Apple Inc. All rights reserved.
3 * 3 *
4 * Redistribution and use in source and binary forms, with or without 4 * Redistribution and use in source and binary forms, with or without
5 * modification, are permitted provided that the following conditions 5 * modification, are permitted provided that the following conditions
6 * are met: 6 * are met:
7 * 1. Redistributions of source code must retain the above copyright 7 * 1. Redistributions of source code must retain the above copyright
8 * notice, this list of conditions and the following disclaimer. 8 * notice, this list of conditions and the following disclaimer.
9 * 2. Redistributions in binary form must reproduce the above copyright 9 * 2. Redistributions in binary form must reproduce the above copyright
10 * notice, this list of conditions and the following disclaimer in the 10 * notice, this list of conditions and the following disclaimer in the
(...skipping 267 matching lines...) Expand 10 before | Expand all | Expand 10 after
278 278
279 void doPaintTask(GraphicsLayerPaintInfo&, GraphicsContext*, const IntRect& c lip); 279 void doPaintTask(GraphicsLayerPaintInfo&, GraphicsContext*, const IntRect& c lip);
280 280
281 RenderLayer& m_owningLayer; 281 RenderLayer& m_owningLayer;
282 282
283 // The hierarchy of layers that is maintained by the CompositedLayerMapping looks like this: 283 // The hierarchy of layers that is maintained by the CompositedLayerMapping looks like this:
284 // 284 //
285 // + m_ancestorClippingLayer [OPTIONAL] 285 // + m_ancestorClippingLayer [OPTIONAL]
286 // + m_graphicsLayer 286 // + m_graphicsLayer
287 // + m_childContainmentLayer [OPTIONAL] <-OR-> m_scrollingLayer [OPTI ONAL] <-OR-> m_childTransformLayer 287 // + m_childContainmentLayer [OPTIONAL] <-OR-> m_scrollingLayer [OPTI ONAL] <-OR-> m_childTransformLayer
288 // + m_scrollingContents Layer [OPTIONAL] 288 // + m_scrollingContents Layer [Present iff m_scrollingLayer is present]
289 // 289 //
290 // We need an ancestor clipping layer if our clipping ancestor is not our an cestor in the 290 // We need an ancestor clipping layer if our clipping ancestor is not our an cestor in the
291 // clipping tree. Here's what that might look like. 291 // clipping tree. Here's what that might look like.
292 // 292 //
293 // Let A = the clipping ancestor, 293 // Let A = the clipping ancestor,
294 // B = the clip descendant, and 294 // B = the clip descendant, and
295 // SC = the stacking context that is the ancestor of A and B in the stac king tree. 295 // SC = the stacking context that is the ancestor of A and B in the stac king tree.
296 // 296 //
297 // SC 297 // SC
298 // + A = m_graphicsLayer 298 // + A = m_graphicsLayer
(...skipping 45 matching lines...) Expand 10 before | Expand all | Expand 10 after
344 // the background layer (or repainting). 344 // the background layer (or repainting).
345 OwnPtr<GraphicsLayer> m_foregroundLayer; // Only used in cases where we need to draw the foreground separately. 345 OwnPtr<GraphicsLayer> m_foregroundLayer; // Only used in cases where we need to draw the foreground separately.
346 OwnPtr<GraphicsLayer> m_backgroundLayer; // Only used in cases where we need to draw the background separately. 346 OwnPtr<GraphicsLayer> m_backgroundLayer; // Only used in cases where we need to draw the background separately.
347 347
348 OwnPtr<GraphicsLayer> m_layerForHorizontalScrollbar; 348 OwnPtr<GraphicsLayer> m_layerForHorizontalScrollbar;
349 OwnPtr<GraphicsLayer> m_layerForVerticalScrollbar; 349 OwnPtr<GraphicsLayer> m_layerForVerticalScrollbar;
350 OwnPtr<GraphicsLayer> m_layerForScrollCorner; 350 OwnPtr<GraphicsLayer> m_layerForScrollCorner;
351 351
352 // A squashing CLM has two possible squashing-related structures. 352 // A squashing CLM has two possible squashing-related structures.
353 // 353 //
354 // If m_clippingAncestorLayer is present: 354 // If m_ancestorClippingLayer is present:
355 // 355 //
356 // m_ancestorClippingLayer 356 // m_ancestorClippingLayer
357 // + m_graphicsLayer 357 // + m_graphicsLayer
358 // + m_squashingLayer 358 // + m_squashingLayer
359 // 359 //
360 // If not: 360 // If not:
361 // 361 //
362 // m_squashingContainmentLayer 362 // m_squashingContainmentLayer
363 // + m_graphicsLayer 363 // + m_graphicsLayer
364 // + m_squashingLayer 364 // + m_squashingLayer
(...skipping 10 matching lines...) Expand all
375 bool m_requiresOwnBackingStoreForAncestorReasons : 1; 375 bool m_requiresOwnBackingStoreForAncestorReasons : 1;
376 bool m_canCompositeFilters : 1; 376 bool m_canCompositeFilters : 1;
377 bool m_backgroundLayerPaintsFixedRootBackground : 1; 377 bool m_backgroundLayerPaintsFixedRootBackground : 1;
378 bool m_needToUpdateGraphicsLayer : 1; 378 bool m_needToUpdateGraphicsLayer : 1;
379 bool m_needToUpdateGraphicsLayerOfAllDecendants : 1; 379 bool m_needToUpdateGraphicsLayerOfAllDecendants : 1;
380 }; 380 };
381 381
382 } // namespace WebCore 382 } // namespace WebCore
383 383
384 #endif // CompositedLayerMapping_h 384 #endif // CompositedLayerMapping_h
OLDNEW
« no previous file with comments | « Source/core/rendering/RenderLayer.cpp ('k') | Source/core/rendering/compositing/CompositedLayerMapping.cpp » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698