reflow comments in core/frame

third_party/WebKit/Source/core/frame/LocalFrame.cpp

 /*
  * Copyright (C) 1998, 1999 Torben Weis <weis@kde.org>
  *                     1999 Lars Knoll <knoll@kde.org>
  *                     1999 Antti Koivisto <koivisto@kde.org>
  *                     2000 Simon Hausmann <hausmann@kde.org>
  *                     2000 Stefan Schimanski <1Stein@gmx.de>
  *                     2001 George Staikos <staikos@kde.org>
- * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All rights reserved.
+ * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011 Apple Inc. All
+ * rights reserved.
  * Copyright (C) 2005 Alexey Proskuryakov <ap@nypop.com>
  * Copyright (C) 2008 Nokia Corporation and/or its subsidiary(-ies)
  * Copyright (C) 2008 Eric Seidel <eric@webkit.org>
  * Copyright (C) 2008 Google Inc.
  *
  * This library is free software; you can redistribute it and/or
  * modify it under the terms of the GNU Library General Public
  * License as published by the Free Software Foundation; either
  * version 2 of the License, or (at your option) any later version.
  *
@@ skipping 72 matching lines @@
 #include "wtf/PtrUtil.h"
 #include "wtf/StdLibExtras.h"
 #include <memory>
 
 namespace blink {
 
 using namespace HTMLNames;
 
 namespace {
 
-// Convenience class for initializing a GraphicsContext to build a DragImage from a specific
-// region specified by |bounds|. After painting the using context(), the DragImage returned from
-// createImage() will only contain the content in |bounds| with the appropriate device scale
-// factor included.
+// Convenience class for initializing a GraphicsContext to build a DragImage
+// from a specific region specified by |bounds|. After painting the using
+// context(), the DragImage returned from createImage() will only contain the
+// content in |bounds| with the appropriate device scale factor included.
 class DragImageBuilder {
   STACK_ALLOCATED();
 
  public:
   DragImageBuilder(const LocalFrame& localFrame, const FloatRect& bounds)
       : m_localFrame(&localFrame), m_bounds(bounds) {
-    // TODO(oshima): Remove this when all platforms are migrated to use-zoom-for-dsf.
+    // TODO(oshima): Remove this when all platforms are migrated to
+    // use-zoom-for-dsf.
     float deviceScaleFactor =
         m_localFrame->host()->deviceScaleFactorDeprecated();
     float pageScaleFactor = m_localFrame->host()->visualViewport().scale();
     m_bounds.setWidth(m_bounds.width() * deviceScaleFactor * pageScaleFactor);
     m_bounds.setHeight(m_bounds.height() * deviceScaleFactor * pageScaleFactor);
     m_pictureBuilder = wrapUnique(new SkPictureBuilder(
         SkRect::MakeIWH(m_bounds.width(), m_bounds.height())));
 
     AffineTransform transform;
     transform.scale(deviceScaleFactor * pageScaleFactor,
@@ skipping 257 matching lines @@
 void LocalFrame::detach(FrameDetachType type) {
   PluginScriptForbiddenScope forbidPluginDestructorScripting;
   m_loader.stopAllLoaders();
   // Don't allow any new child frames to load in this frame: attaching a new
   // child frame during or after detaching children results in an attached
   // frame on a detached DOM tree, which is bad.
   SubframeLoadingDisabler disabler(*document());
   m_loader.dispatchUnloadEvent();
   detachChildren();
 
-  // All done if detaching the subframes brought about a detach of this frame also.
+  // All done if detaching the subframes brought about a detach of this frame
+  // also.
   if (!client())
     return;
 
-  // stopAllLoaders() needs to be called after detachChildren(), because detachChildren()
-  // will trigger the unload event handlers of any child frames, and those event
-  // handlers might start a new subresource load in this frame.
+  // stopAllLoaders() needs to be called after detachChildren(), because
+  // detachChildren() will trigger the unload event handlers of any child
+  // frames, and those event handlers might start a new subresource load in this
+  // frame.
   m_loader.stopAllLoaders();
   m_loader.detach();
   document()->shutdown();
   // This is the earliest that scripting can be disabled:
   // - FrameLoader::detach() can fire XHR abort events
   // - Document::shutdown()'s deferred widget updates can run script.
   ScriptForbiddenScope forbidScript;
   m_loader.clear();
   if (!client())
     return;
 
   client()->willBeDetached();
-  // Notify ScriptController that the frame is closing, since its cleanup ends up calling
-  // back to FrameLoaderClient via WindowProxy.
+  // Notify ScriptController that the frame is closing, since its cleanup ends
+  // up calling back to FrameLoaderClient via WindowProxy.
   script().clearForClose();
   setView(nullptr);
 
   m_host->eventHandlerRegistry().didRemoveAllEventHandlers(*localDOMWindow());
 
   localDOMWindow()->frameDestroyed();
 
   // TODO: Page should take care of updating focus/scrolling instead of Frame.
   // TODO: It's unclear as to why this is called more than once, but it is,
   // so page() could be null.
@@ skipping 61 matching lines @@
   // is acceptable as its actions are either not needed or handled
   // by other means --
   //
   //  - LocalFrameLifecycleObserver::willDetachFrameHost() will have
   //    signalled the Inspector frameWindowDiscarded() notifications.
   //    We assume that all LocalFrames are detached, where that notification
   //    will have been done.
   //
   //  - Calling LocalDOMWindow::reset() is not needed (called from
   //    Frame::setDOMWindow().) The Member references it clears will now
-  //    die with the window. And the registered DOMWindowProperty instances that don't,
-  //    only keep a weak reference to this frame, so there's no need to be
-  //    explicitly notified that this frame is going away.
+  //    die with the window. And the registered DOMWindowProperty instances that
+  //    don't, only keep a weak reference to this frame, so there's no need to
+  //    be explicitly notified that this frame is going away.
   if (domWindow)
     script().clearWindowProxy();
 
   if (m_domWindow)
     m_domWindow->reset();
   m_domWindow = domWindow;
 }
 
 Document* LocalFrame::document() const {
   return m_domWindow ? m_domWindow->document() : nullptr;
@@ skipping 32 matching lines @@
   Frame* top = tree().top();
   return top &&
          !securityOrigin->canAccess(
              top->securityContext()->getSecurityOrigin());
 }
 
 void LocalFrame::setPrinting(bool printing,
                              const FloatSize& pageSize,
                              const FloatSize& originalPageSize,
                              float maximumShrinkRatio) {
-  // In setting printing, we should not validate resources already cached for the document.
-  // See https://bugs.webkit.org/show_bug.cgi?id=43704
+  // In setting printing, we should not validate resources already cached for
+  // the document.  See https://bugs.webkit.org/show_bug.cgi?id=43704
   ResourceCacheValidationSuppressor validationSuppressor(document()->fetcher());
 
   document()->setPrinting(printing);
   view()->adjustMediaTypeForPrinting(printing);
 
   if (shouldUsePrintingLayout()) {
     view()->forceLayoutForPagination(pageSize, originalPageSize,
                                      maximumShrinkRatio);
   } else {
     if (LayoutView* layoutView = view()->layoutView()) {
@@ skipping 57 matching lines @@
 
   Page* page = this->page();
   if (!page)
     return;
 
   Document* document = this->document();
   if (!document)
     return;
 
   // Respect SVGs zoomAndPan="disabled" property in standalone SVG documents.
-  // FIXME: How to handle compound documents + zoomAndPan="disabled"? Needs SVG WG clarification.
+  // FIXME: How to handle compound documents + zoomAndPan="disabled"? Needs SVG
+  // WG clarification.
   if (document->isSVGDocument()) {
     if (!document->accessSVGExtensions().zoomAndPanEnabled())
       return;
   }
 
   if (m_pageZoomFactor != pageZoomFactor) {
     if (FrameView* view = this->view()) {
-      // Update the scroll position when doing a full page zoom, so the content stays in relatively the same position.
+      // Update the scroll position when doing a full page zoom, so the content
+      // stays in relatively the same position.
       LayoutPoint scrollPosition = view->scrollPosition();
       float percentDifference = (pageZoomFactor / m_pageZoomFactor);
       view->setScrollPosition(
           DoublePoint(scrollPosition.x() * percentDifference,
                       scrollPosition.y() * percentDifference),
           ProgrammaticScroll);
     }
   }
 
   m_pageZoomFactor = pageZoomFactor;
@@ skipping 248 matching lines @@
   if (m_frame && m_frame->client() && m_frame->client()->frameBlameContext())
     m_frame->client()->frameBlameContext()->Enter();
 }
 
 ScopedFrameBlamer::~ScopedFrameBlamer() {
   if (m_frame && m_frame->client() && m_frame->client()->frameBlameContext())
     m_frame->client()->frameBlameContext()->Leave();
 }
 
 }  // namespace blink