reflow comments in core/svg/

third_party/WebKit/Source/core/svg/graphics/SVGImage.cpp

 /*
  * Copyright (C) 2006 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008, 2009 Apple Inc. All rights reserved.
  * Copyright (C) Research In Motion Limited 2011. 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.
@@ skipping 48 matching lines @@
 #include "third_party/skia/include/core/SkPicture.h"
 #include "wtf/PassRefPtr.h"
 
 namespace blink {
 
 SVGImage::SVGImage(ImageObserver* observer)
     : Image(observer), m_hasPendingTimelineRewind(false) {}
 
 SVGImage::~SVGImage() {
   if (m_page) {
-    // Store m_page in a local variable, clearing m_page, so that SVGImageChromeClient knows we're destructed.
+    // Store m_page in a local variable, clearing m_page, so that
+    // SVGImageChromeClient knows we're destructed.
     Page* currentPage = m_page.release();
     // Break both the loader and view references to the frame
     currentPage->willBeDestroyed();
   }
 
   // Verify that page teardown destroyed the Chrome
   ASSERT(!m_chromeClient || !m_chromeClient->image());
 }
 
 bool SVGImage::isInSVGImage(const Node* node) {
@@ skipping 93 matching lines @@
   LayoutReplaced::IntrinsicSizingInfo intrinsicSizingInfo;
   layoutObject->computeIntrinsicSizingInfo(intrinsicSizingInfo);
 
   // https://www.w3.org/TR/css3-images/#default-sizing
 
   if (intrinsicSizingInfo.hasWidth && intrinsicSizingInfo.hasHeight)
     return intrinsicSizingInfo.size;
 
   if (svg->preserveAspectRatio()->currentValue()->align() ==
       SVGPreserveAspectRatio::kSvgPreserveaspectratioNone) {
-    // TODO(davve): The intrinsic aspect ratio is not used to resolve a missing intrinsic width
-    // or height when preserveAspectRatio is none. It's unclear whether this is correct. See
-    // crbug.com/584172.
+    // TODO(davve): The intrinsic aspect ratio is not used to resolve a missing
+    // intrinsic width or height when preserveAspectRatio is none. It's unclear
+    // whether this is correct. See crbug.com/584172.
     return defaultObjectSize;
   }
 
   if (intrinsicSizingInfo.hasWidth) {
     if (intrinsicSizingInfo.aspectRatio.isEmpty())
       return FloatSize(intrinsicSizingInfo.size.width(),
                        defaultObjectSize.height());
 
     return FloatSize(intrinsicSizingInfo.size.width(),
                      resolveHeightForRatio(intrinsicSizingInfo.size.width(),
                                            intrinsicSizingInfo.aspectRatio));
   }
 
   if (intrinsicSizingInfo.hasHeight) {
     if (intrinsicSizingInfo.aspectRatio.isEmpty())
       return FloatSize(defaultObjectSize.width(),
                        intrinsicSizingInfo.size.height());
 
     return FloatSize(resolveWidthForRatio(intrinsicSizingInfo.size.height(),
                                           intrinsicSizingInfo.aspectRatio),
                      intrinsicSizingInfo.size.height());
   }
 
   if (!intrinsicSizingInfo.aspectRatio.isEmpty()) {
-    // "A contain constraint is resolved by setting the concrete object size to the largest
-    //  rectangle that has the object's intrinsic aspect ratio and additionally has neither
-    //  width nor height larger than the constraint rectangle's width and height, respectively."
+    // "A contain constraint is resolved by setting the concrete object size to
+    //  the largest rectangle that has the object's intrinsic aspect ratio and
+    //  additionally has neither width nor height larger than the constraint
+    //  rectangle's width and height, respectively."
     float solutionWidth = resolveWidthForRatio(defaultObjectSize.height(),
                                                intrinsicSizingInfo.aspectRatio);
     if (solutionWidth <= defaultObjectSize.width())
       return FloatSize(solutionWidth, defaultObjectSize.height());
 
     float solutionHeight = resolveHeightForRatio(
         defaultObjectSize.width(), intrinsicSizingInfo.aspectRatio);
     return FloatSize(defaultObjectSize.width(), solutionHeight);
   }
 
   return defaultObjectSize;
 }
 
 void SVGImage::drawForContainer(SkCanvas* canvas,
                                 const SkPaint& paint,
                                 const FloatSize containerSize,
                                 float zoom,
                                 const FloatRect& dstRect,
                                 const FloatRect& srcRect,
                                 const KURL& url) {
   if (!m_page)
     return;
 
-  // Temporarily disable the image observer to prevent changeInRect() calls due re-laying out the image.
+  // Temporarily disable the image observer to prevent changeInRect() calls due
+  // re-laying out the image.
   ImageObserverDisabler imageObserverDisabler(this);
 
   IntSize roundedContainerSize = roundedIntSize(containerSize);
 
   if (SVGSVGElement* rootElement = svgRootElement(m_page.get())) {
     if (LayoutSVGRoot* layoutObject =
             toLayoutSVGRoot(rootElement->layoutObject()))
       layoutObject->setContainerSize(roundedContainerSize);
   }
 
@@ skipping 30 matching lines @@
 
   // Expand the tile to account for repeat spacing.
   FloatRect spacedTile(tile);
   spacedTile.expand(FloatSize(repeatSpacing));
 
   SkPictureBuilder patternPicture(spacedTile, nullptr, &context);
   {
     DrawingRecorder patternPictureRecorder(
         patternPicture.context(), patternPicture, DisplayItem::Type::kSVGImage,
         spacedTile);
-    // When generating an expanded tile, make sure we don't draw into the spacing area.
+    // When generating an expanded tile, make sure we don't draw into the
+    // spacing area.
     if (tile != spacedTile)
       patternPicture.context().clip(tile);
     SkPaint paint;
     drawForContainer(patternPicture.context().canvas(), paint, containerSize,
                      zoom, tile, srcRect, url);
   }
   sk_sp<SkPicture> tilePicture = patternPicture.endRecording();
 
   SkMatrix patternTransform;
   patternTransform.setTranslate(phase.x() + spacedTile.x(),
@@ skipping 74 matching lines @@
   // time=0.) The reason we do this here and not in resetAnimation() is to
   // avoid setting timers from the latter.
   flushPendingTimelineRewind();
 
   SkPictureBuilder imagePicture(dstRect);
   {
     ClipRecorder clipRecorder(imagePicture.context(), imagePicture,
                               DisplayItem::kClipNodeImage,
                               enclosingIntRect(dstRect));
 
-    // We can only draw the entire frame, clipped to the rect we want. So compute where the top left
-    // of the image would be if we were drawing without clipping, and translate accordingly.
+    // We can only draw the entire frame, clipped to the rect we want. So
+    // compute where the top left of the image would be if we were drawing
+    // without clipping, and translate accordingly.
     FloatSize scale(dstRect.width() / srcRect.width(),
                     dstRect.height() / srcRect.height());
     FloatSize topLeftOffset(srcRect.location().x() * scale.width(),
                             srcRect.location().y() * scale.height());
     FloatPoint destOffset = dstRect.location() - topLeftOffset;
     AffineTransform transform =
         AffineTransform::translation(destOffset.x(), destOffset.y());
     transform.scale(scale.width(), scale.height());
     TransformRecorder transformRecorder(imagePicture.context(), imagePicture,
                                         transform);
@@ skipping 196 matching lines @@
       frame =
           LocalFrame::create(&dummyFrameLoaderClient, &page->frameHost(), 0);
       frame->setView(FrameView::create(frame));
       frame->init();
     }
 
     FrameLoader& loader = frame->loader();
     loader.forceSandboxFlags(SandboxAll);
 
     frame->view()->setScrollbarsSuppressed(true);
-    frame->view()->setCanHaveScrollbars(
-        false);                           // SVG Images will always synthesize a viewBox, if it's not available, and thus never see scrollbars.
-    frame->view()->setTransparent(true);  // SVG Images are transparent.
+    // SVG Images will always synthesize a viewBox, if it's not available, and
+    // thus never see scrollbars.
+    frame->view()->setCanHaveScrollbars(false);
+    // SVG Images are transparent.
+    frame->view()->setTransparent(true);
 
     m_page = page;
 
     TRACE_EVENT0("blink", "SVGImage::dataChanged::load");
     loader.load(FrameLoadRequest(
         0, blankURL(),
         SubstituteData(data(), AtomicString("image/svg+xml"),
                        AtomicString("UTF-8"), KURL(), ForceSynchronousLoad)));
 
     // Set the concrete object size before a container size is available.
     m_intrinsicSize = roundedIntSize(concreteObjectSize(FloatSize(
         LayoutReplaced::defaultWidth, LayoutReplaced::defaultHeight)));
   }
 
   return m_page ? SizeAvailable : SizeUnavailable;
 }
 
 String SVGImage::filenameExtension() const {
   return "svg";
 }
 
 }  // namespace blink