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

Unified Diff: third_party/WebKit/Source/core/layout/compositing/CompositingInputsUpdater.cpp

Issue 2393673004: reflow comments in core/layout/compositing,core/observer (Closed)
Patch Set: comments (heh!) Created 4 years, 2 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: third_party/WebKit/Source/core/layout/compositing/CompositingInputsUpdater.cpp
diff --git a/third_party/WebKit/Source/core/layout/compositing/CompositingInputsUpdater.cpp b/third_party/WebKit/Source/core/layout/compositing/CompositingInputsUpdater.cpp
index 70f79074c18fb29355e7284464131fd288455ebd..4b2a832d770eea6f35d346f13a3579932cbdceb6 100644
--- a/third_party/WebKit/Source/core/layout/compositing/CompositingInputsUpdater.cpp
+++ b/third_party/WebKit/Source/core/layout/compositing/CompositingInputsUpdater.cpp
@@ -34,8 +34,8 @@ static const PaintLayer* findParentLayerOnClippingContainerChain(
for (current = current->parent();
current && !current->canContainFixedPositionObjects();
current = current->parent()) {
- // CSS clip applies to fixed position elements even for ancestors that are not what the
- // fixed element is positioned with respect to.
+ // CSS clip applies to fixed position elements even for ancestors that
+ // are not what the fixed element is positioned with respect to.
if (current->hasClip()) {
DCHECK(current->hasLayer());
return static_cast<const LayoutBoxModelObject*>(current)->layer();
@@ -47,7 +47,8 @@ static const PaintLayer* findParentLayerOnClippingContainerChain(
if (current->hasLayer())
return static_cast<const LayoutBoxModelObject*>(current)->layer();
- // Having clip or overflow clip forces the LayoutObject to become a layer, except for contains: paint, which may apply to SVG.
+ // Having clip or overflow clip forces the LayoutObject to become a layer,
+ // except for contains: paint, which may apply to SVG.
// SVG (other than LayoutSVGRoot) cannot have PaintLayers.
DCHECK(!current->hasClipRelatedProperty() ||
current->styleRef().containsPaint());
@@ -149,8 +150,8 @@ void CompositingInputsUpdater::updateRecursive(PaintLayer* layer,
properties.clippedAbsoluteBoundingBox =
enclosingIntRect(m_geometryMap.absoluteRect(
FloatRect(layer->boundingBoxForCompositingOverlapTest())));
- // FIXME: Setting the absBounds to 1x1 instead of 0x0 makes very little sense,
- // but removing this code will make JSGameBench sad.
+ // FIXME: Setting the absBounds to 1x1 instead of 0x0 makes very little
+ // sense, but removing this code will make JSGameBench sad.
// See https://codereview.chromium.org/13912020/
if (properties.clippedAbsoluteBoundingBox.isEmpty())
properties.clippedAbsoluteBoundingBox.setSize(IntSize(1, 1));

Powered by Google App Engine
This is Rietveld 408576698