reflow comments in web/

third_party/WebKit/Source/web/WebRemoteFrameImpl.cpp

 // Copyright 2014 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "web/WebRemoteFrameImpl.h"
 
 #include "core/dom/Fullscreen.h"
 #include "core/dom/RemoteSecurityContext.h"
 #include "core/dom/SecurityContext.h"
 #include "core/frame/FrameView.h"
@@ skipping 340 matching lines @@
     const WebString& uniqueName,
     WebSandboxFlags sandboxFlags,
     WebFrameClient* client,
     WebFrame* previousSibling,
     const WebFrameOwnerProperties& frameOwnerProperties,
     WebFrame* opener) {
   WebLocalFrameImpl* child = WebLocalFrameImpl::create(scope, client, opener);
   insertAfter(child, previousSibling);
   RemoteFrameOwner* owner = RemoteFrameOwner::create(
       static_cast<SandboxFlags>(sandboxFlags), frameOwnerProperties);
-  // FIXME: currently this calls LocalFrame::init() on the created LocalFrame, which may
-  // result in the browser observing two navigations to about:blank (one from the initial
-  // frame creation, and one from swapping it into the remote process). FrameLoader might
-  // need a special initialization function for this case to avoid that duplicate navigation.
+  // FIXME: currently this calls LocalFrame::init() on the created LocalFrame,
+  // which may result in the browser observing two navigations to about:blank
+  // (one from the initial frame creation, and one from swapping it into the
+  // remote process).  FrameLoader might need a special initialization function
+  // for this case to avoid that duplicate navigation.
   child->initializeCoreFrame(frame()->host(), owner, name, uniqueName);
-  // Partially related with the above FIXME--the init() call may trigger JS dispatch. However,
+  // Partially related with the above FIXME--the init() call may trigger JS
+  // dispatch. However,
   // if the parent is remote, it should never be detached synchronously...
   DCHECK(child->frame());
   return child;
 }
 
 void WebRemoteFrameImpl::initializeCoreFrame(FrameHost* host,
                                              FrameOwner* owner,
                                              const AtomicString& name,
                                              const AtomicString& uniqueName) {
   setCoreFrame(RemoteFrame::create(m_frameClient.get(), host, owner));
@@ skipping 24 matching lines @@
   if (!frame.client())
     return nullptr;
   return static_cast<RemoteFrameClientImpl*>(frame.client())->webFrame();
 }
 
 void WebRemoteFrameImpl::setReplicatedOrigin(
     const WebSecurityOrigin& origin) const {
   DCHECK(frame());
   frame()->securityContext()->setReplicatedOrigin(origin);
 
-  // If the origin of a remote frame changed, the accessibility object for the owner
-  // element now points to a different child.
+  // If the origin of a remote frame changed, the accessibility object for the
+  // owner element now points to a different child.
   //
   // TODO(dmazzoni, dcheng): there's probably a better way to solve this.
   // Run SitePerProcessAccessibilityBrowserTest.TwoCrossSiteNavigations to
   // ensure an alternate fix works.  http://crbug.com/566222
   FrameOwner* owner = frame()->owner();
   if (owner && owner->isLocal()) {
     HTMLElement* ownerElement = toHTMLFrameOwnerElement(owner);
     AXObjectCache* cache = ownerElement->document().existingAXObjectCache();
     if (cache)
       cache->childrenChanged(ownerElement);
@@ skipping 28 matching lines @@
 
 void WebRemoteFrameImpl::setReplicatedInsecureRequestPolicy(
     WebInsecureRequestPolicy policy) const {
   DCHECK(frame());
   frame()->securityContext()->setInsecureRequestPolicy(policy);
 }
 
 void WebRemoteFrameImpl::setReplicatedPotentiallyTrustworthyUniqueOrigin(
     bool isUniqueOriginPotentiallyTrustworthy) const {
   DCHECK(frame());
-  // If |isUniqueOriginPotentiallyTrustworthy| is true, then the origin must be unique.
+  // If |isUniqueOriginPotentiallyTrustworthy| is true, then the origin must be
+  // unique.
   DCHECK(!isUniqueOriginPotentiallyTrustworthy ||
          frame()->securityContext()->getSecurityOrigin()->isUnique());
   frame()
       ->securityContext()
       ->getSecurityOrigin()
       ->setUniqueOriginIsPotentiallyTrustworthy(
           isUniqueOriginPotentiallyTrustworthy);
 }
 
 void WebRemoteFrameImpl::DispatchLoadEventForFrameOwner() const {
@@ skipping 47 matching lines @@
 }
 
 WebRemoteFrameImpl::WebRemoteFrameImpl(WebTreeScopeType scope,
                                        WebRemoteFrameClient* client)
     : WebRemoteFrame(scope),
       m_frameClient(RemoteFrameClientImpl::create(this)),
       m_client(client),
       m_selfKeepAlive(this) {}
 
 }  // namespace blink