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

Unified Diff: third_party/WebKit/Source/core/paint/ReplacedPainter.cpp

Issue 2392443009: reflow comments in core/paint (Closed)
Patch Set: 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/paint/ReplacedPainter.cpp
diff --git a/third_party/WebKit/Source/core/paint/ReplacedPainter.cpp b/third_party/WebKit/Source/core/paint/ReplacedPainter.cpp
index 2492b7e4188062c4b1e9672cf5cac1d7d816a6bd..a8bc45199a223135ee98db45581390d89166a5be 100644
--- a/third_party/WebKit/Source/core/paint/ReplacedPainter.cpp
+++ b/third_party/WebKit/Source/core/paint/ReplacedPainter.cpp
@@ -70,7 +70,8 @@ void ReplacedPainter::paint(const PaintInfo& paintInfo,
if (borderRect.isEmpty()) {
completelyClippedOut = true;
} else if (shouldApplyViewportClip(m_layoutReplaced)) {
- // Push a clip if we have a border radius, since we want to round the foreground content that gets painted.
+ // Push a clip if we have a border radius, since we want to round the
+ // foreground content that gets painted.
FloatRoundedRect roundedInnerRect =
m_layoutReplaced.style()->getRoundedInnerBorderFor(
borderRect,
@@ -99,8 +100,8 @@ void ReplacedPainter::paint(const PaintInfo& paintInfo,
}
}
- // The selection tint never gets clipped by border-radius rounding, since we want it to run right up to the edges of
- // surrounding content.
+ // The selection tint never gets clipped by border-radius rounding, since we
+ // want it to run right up to the edges of surrounding content.
bool drawSelectionTint =
paintInfo.phase == PaintPhaseForeground &&
m_layoutReplaced.getSelectionState() != SelectionNone &&
@@ -132,7 +133,8 @@ bool ReplacedPainter::shouldPaint(
return false;
// If we're invisible or haven't received a layout yet, just bail.
- // But if it's an SVG root, there can be children, so we'll check visibility later.
+ // But if it's an SVG root, there can be children, so we'll check visibility
+ // later.
if (!m_layoutReplaced.isSVGRoot() &&
m_layoutReplaced.style()->visibility() != EVisibility::Visible)
return false;

Powered by Google App Engine
This is Rietveld 408576698