Rewrap comments to 80 columns in Source/platform/graphics/.

third_party/WebKit/Source/platform/graphics/GraphicsContext.cpp

 /*
  * Copyright (C) 2003, 2004, 2005, 2006, 2009 Apple Inc. All rights reserved.
  * Copyright (C) 2013 Google Inc. All rights reserved.
  *
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions
  * are met:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 56 matching lines @@
       m_inDrawingRecorder(false)
 #endif
       ,
       m_disabledState(disableContextOrPainting),
       m_deviceScaleFactor(1.0f),
       m_printing(false),
       m_hasMetaData(!!metaData) {
   if (metaData)
     m_metaData = *metaData;
 
-  // FIXME: Do some tests to determine how many states are typically used, and allocate
-  // several here.
+  // FIXME: Do some tests to determine how many states are typically used, and
+  // allocate several here.
   m_paintStateStack.append(GraphicsContextState::create());
   m_paintState = m_paintStateStack.last().get();
 
   if (contextDisabled()) {
     DEFINE_STATIC_LOCAL(SkCanvas*, nullCanvas, (SkCreateNullCanvas()));
     m_canvas = nullCanvas;
   }
 }
 
 GraphicsContext::~GraphicsContext() {
@@ skipping 116 matching lines @@
       drawLooperBuilder ? drawLooperBuilder->detachDrawLooper() : nullptr);
 }
 
 SkColorFilter* GraphicsContext::getColorFilter() const {
   return immutableState()->getColorFilter();
 }
 
 void GraphicsContext::setColorFilter(ColorFilter colorFilter) {
   GraphicsContextState* stateToSet = mutableState();
 
-  // We only support one active color filter at the moment. If (when) this becomes a problem,
-  // we should switch to using color filter chains (Skia work in progress).
+  // We only support one active color filter at the moment. If (when) this
+  // becomes a problem, we should switch to using color filter chains (Skia work
+  // in progress).
   DCHECK(!stateToSet->getColorFilter());
   stateToSet->setColorFilter(WebCoreColorFilterToSkiaColorFilter(colorFilter));
 }
 
 void GraphicsContext::concat(const SkMatrix& matrix) {
   if (contextDisabled())
     return;
 
   ASSERT(m_canvas);
 
@@ skipping 250 matching lines @@
 
   // We know these are vertical or horizontal lines, so the length will just
   // be the sum of the displacement component vectors give or take 1 -
   // probably worth the speed up of no square root, which also won't be exact.
   FloatSize disp = p2 - p1;
   int length = SkScalarRoundToInt(disp.width() + disp.height());
   SkPaint paint(immutableState()->strokePaint(length));
 
   if (getStrokeStyle() == DottedStroke || getStrokeStyle() == DashedStroke) {
     // Do a rect fill of our endpoints.  This ensures we always have the
-    // appearance of being a border.  We then draw the actual dotted/dashed line.
+    // appearance of being a border.  We then draw the actual dotted/dashed
+    // line.
     SkRect r1, r2;
     r1.set(p1.x(), p1.y(), p1.x() + width, p1.y() + width);
     r2.set(p2.x(), p2.y(), p2.x() + width, p2.y() + width);
 
     if (isVerticalLine) {
       r1.offset(-width / 2, 0);
       r2.offset(-width / 2, -width);
     } else {
       r1.offset(0, -width / 2);
       r2.offset(-width, -width / 2);
@@ skipping 536 matching lines @@
       !WebCoreFloatNearlyEqual(
           oRadii.bottomRight().height() - strokeSize.height(),
           iRadii.bottomRight().height()) ||
       !WebCoreFloatNearlyEqual(oRadii.bottomLeft().width() - strokeSize.width(),
                                iRadii.bottomLeft().width()) ||
       !WebCoreFloatNearlyEqual(
           oRadii.bottomLeft().height() - strokeSize.height(),
           iRadii.bottomLeft().height()))
     return false;
 
-  // We also ignore DRRects with a very thick relative stroke (shapes which are mostly filled by
-  // the stroke): Skia's stroke outline can diverge significantly from the outer/inner contours
-  // in some edge cases, so we fall back to drawDRRect instead.
+  // We also ignore DRRects with a very thick relative stroke (shapes which are
+  // mostly filled by the stroke): Skia's stroke outline can diverge
+  // significantly from the outer/inner contours in some edge cases, so we fall
+  // back to drawDRRect instead.

[Stephen White, 2016/10/05 14:45:37]

Nit: drawDRRect() (not new to this patch)

[Peter Kasting, 2016/10/05 18:46:01]

Done.

   const float kMaxStrokeToSizeRatio = 0.75f;
   if (2 * strokeSize.width() / outer.rect().width() > kMaxStrokeToSizeRatio ||
       2 * strokeSize.height() / outer.rect().height() > kMaxStrokeToSizeRatio)
     return false;
 
   return true;
 }
 
 }  // anonymous namespace
 
@@ skipping 88 matching lines @@
     return;
   }
 
   clipRRect(rrect, shouldAntialias, regionOp);
 }
 
 void GraphicsContext::clipOut(const Path& pathToClip) {
   if (contextDisabled())
     return;
 
-  // Use const_cast and temporarily toggle the inverse fill type instead of copying the path.
+  // Use const_cast and temporarily toggle the inverse fill type instead of
+  // copying the path.
   SkPath& path = const_cast<SkPath&>(pathToClip.getSkPath());
   path.toggleInverseFillType();
   clipPath(path, AntiAliased);
   path.toggleInverseFillType();
 }
 
 void GraphicsContext::clipOutRoundedRect(const FloatRoundedRect& rect) {
   if (contextDisabled())
     return;
 
@@ skipping 101 matching lines @@
 
   SkPaint paint(immutableState()->fillPaint());
   paint.setColor(color.rgb());
   m_canvas->drawDRRect(SkRRect::MakeRect(rect), roundedHoleRect, paint);
 }
 
 void GraphicsContext::adjustLineToPixelBoundaries(FloatPoint& p1,
                                                   FloatPoint& p2,
                                                   float strokeWidth,
                                                   StrokeStyle penStyle) {
-  // For odd widths, we add in 0.5 to the appropriate x/y so that the float arithmetic
-  // works out.  For example, with a border width of 3, WebKit will pass us (y1+y2)/2, e.g.,
-  // (50+53)/2 = 103/2 = 51 when we want 51.5.  It is always true that an even width gave
-  // us a perfect position, but an odd width gave us a position that is off by exactly 0.5.
+  // For odd widths, we add in 0.5 to the appropriate x/y so that the float
+  // arithmetic works out.  For example, with a border width of 3, WebKit will
+  // pass us (y1+y2)/2, e.g., (50+53)/2 = 103/2 = 51 when we want 51.5.  It is
+  // always true that an even width gave us a perfect position, but an odd width
+  // gave us a position that is off by exactly 0.5.
   if (penStyle == DottedStroke || penStyle == DashedStroke) {
     if (p1.x() == p2.x()) {
       p1.setY(p1.y() + strokeWidth);
       p2.setY(p2.y() - strokeWidth);
     } else {
       p1.setX(p1.x() + strokeWidth);
       p2.setX(p2.x() - strokeWidth);
     }
   }
 
-  if (static_cast<int>(strokeWidth) % 2) {  //odd
+  if (static_cast<int>(strokeWidth) % 2) {  // odd
     if (p1.x() == p2.x()) {
       // We're a vertical line.  Adjust our x.
       p1.setX(p1.x() + 0.5f);
       p2.setX(p2.x() + 0.5f);
     } else {
       // We're a horizontal line. Adjust our y.
       p1.setY(p1.y() + 0.5f);
       p2.setY(p2.y() + 0.5f);
     }
   }
@@ skipping 105 matching lines @@
   static const SkPMColor colors[] = {
       SkPreMultiplyARGB(0x60, 0xFF, 0x00, 0x00),  // More transparent red
       SkPreMultiplyARGB(0x60, 0xC0, 0xC0, 0xC0)   // More transparent gray
   };
 
   return colors[index];
 }
 #endif
 
 }  // namespace blink