reflow comments in web/

third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp

 /*
  * Copyright (C) 2014 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 56 matching lines @@
 #include "web/WebLocalFrameImpl.h"
 #include "web/WebPluginContainerImpl.h"
 #include "web/WebRemoteFrameImpl.h"
 #include "web/WebViewFrameWidget.h"
 #include "wtf/AutoReset.h"
 #include "wtf/PtrUtil.h"
 #include <memory>
 
 namespace blink {
 
-// WebFrameWidget ----------------------------------------------------------------
+// WebFrameWidget ------------------------------------------------------------
 
 WebFrameWidget* WebFrameWidget::create(WebWidgetClient* client,
                                        WebLocalFrame* localRoot) {
   // Pass the WebFrameWidget's self-reference to the caller.
   return WebFrameWidgetImpl::create(client, localRoot);
 }
 
 WebFrameWidget* WebFrameWidget::create(WebWidgetClient* client,
                                        WebView* webView,
                                        WebLocalFrame* mainFrame) {
@@ skipping 77 matching lines @@
   FrameView* view = m_localRoot->frameView();
   if (!view)
     return;
 
   m_size = newSize;
 
   updateMainFrameLayoutSize();
 
   view->resize(m_size);
 
-  // FIXME: In WebViewImpl this layout was a precursor to setting the minimum scale limit.
-  // It is not clear if this is necessary for frame-level widget resize.
+  // FIXME: In WebViewImpl this layout was a precursor to setting the minimum
+  // scale limit.  It is not clear if this is necessary for frame-level widget
+  // resize.
   if (view->needsLayout())
     view->layout();
 
-  // FIXME: Investigate whether this is needed; comment from eseidel suggests that this function
-  // is flawed.
+  // FIXME: Investigate whether this is needed; comment from eseidel suggests
+  // that this function is flawed.
   sendResizeEventAndRepaint();
 }
 
 void WebFrameWidgetImpl::sendResizeEventAndRepaint() {
   // FIXME: This is wrong. The FrameView is responsible sending a resizeEvent
   // as part of layout. Layout is also responsible for sending invalidations
   // to the embedder. This method and all callers may be wrong. -- eseidel.
   if (m_localRoot->frameView()) {
     // Enqueues the resize event.
     m_localRoot->frame()->document()->enqueueResizeEvent();
@@ skipping 120 matching lines @@
 
 WebInputEventResult WebFrameWidgetImpl::handleInputEvent(
     const WebInputEvent& inputEvent) {
   TRACE_EVENT1("input", "WebFrameWidgetImpl::handleInputEvent", "type",
                WebInputEvent::GetName(inputEvent.type));
 
   // Don't handle events once we've started shutting down.
   if (!page())
     return WebInputEventResult::NotHandled;
 
-  // Report the event to be NOT processed by WebKit, so that the browser can handle it appropriately.
+  // Report the event to be NOT processed by WebKit, so that the browser can
+  // handle it appropriately.
   if (m_ignoreInputEvents)
     return WebInputEventResult::NotHandled;
 
   // FIXME: pass event to m_localRoot's WebDevToolsAgentImpl once available.
 
   AutoReset<const WebInputEvent*> currentEventChange(&m_currentInputEvent,
                                                      &inputEvent);
 
   if (m_mouseCaptureNode && WebInputEvent::isMouseEventType(inputEvent.type)) {
     TRACE_EVENT1("input", "captured mouse event", "type", inputEvent.type);
@@ skipping 39 matching lines @@
 
   return PageWidgetDelegate::handleInputEvent(*this, inputEvent,
                                               m_localRoot->frame());
 }
 
 void WebFrameWidgetImpl::setCursorVisibilityState(bool isVisible) {
   page()->setIsCursorVisible(isVisible);
 }
 
 bool WebFrameWidgetImpl::hasTouchEventHandlersAt(const WebPoint& point) {
-  // FIXME: Implement this. Note that the point must be divided by pageScaleFactor.
+  // FIXME: Implement this. Note that the point must be divided by
+  // pageScaleFactor.
   return true;
 }
 
 void WebFrameWidgetImpl::setBaseBackgroundColor(WebColor color) {
   if (m_baseBackgroundColor == color)
     return;
 
   m_baseBackgroundColor = color;
 
   m_localRoot->frameView()->setBaseBackgroundColor(color);
@@ skipping 61 matching lines @@
     LocalFrame* focusedFrame = focusedLocalFrameInWidget();
     if (focusedFrame) {
       // Finish an ongoing composition to delete the composition node.
       if (focusedFrame->inputMethodController().hasComposition()) {
         WebAutofillClient* autofillClient =
             WebLocalFrameImpl::fromFrame(focusedFrame)->autofillClient();
 
         if (autofillClient)
           autofillClient->setIgnoreTextChanges(true);
 
-        // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
-        // needs to be audited.  See http://crbug.com/590369 for more details.
+        // TODO(xiaochengh): The use of
+        // updateStyleAndLayoutIgnorePendingStylesheets needs to be audited.
+        // See http://crbug.com/590369 for more details.
         focusedFrame->document()
             ->updateStyleAndLayoutIgnorePendingStylesheets();
 
         focusedFrame->inputMethodController().finishComposingText(
             InputMethodController::KeepSelection);
 
         if (autofillClient)
           autofillClient->setIgnoreTextChanges(false);
       }
       m_imeAcceptEvents = false;
@@ skipping 137 matching lines @@
   info.inputMode = inputModeOfFocusedElement();
 
   info.type = textInputType();
   info.flags = textInputFlags();
   if (info.type == WebTextInputTypeNone)
     return info;
 
   if (!focused->editor().canEdit())
     return info;
 
-  // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets needs to be audited.
-  // see http://crbug.com/590369 for more details.
+  // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
+  // needs to be audited.  see http://crbug.com/590369 for more details.
   focused->document()->updateStyleAndLayoutIgnorePendingStylesheets();
 
   DocumentLifecycle::DisallowTransitionScope disallowTransition(
       focused->document()->lifecycle());
 
   // Emits an object replacement character for each replaced element so that
   // it is exposed to IME and thus could be deleted by IME on android.
   info.value = plainText(EphemeralRange::rangeOfContents(*element),
                          TextIteratorEmitsObjectReplacementCharacter);
 
@@ skipping 27 matching lines @@
 WebTextInputType WebFrameWidgetImpl::textInputType() {
   LocalFrame* focusedFrame = focusedLocalFrameInWidget();
   if (!focusedFrame)
     return WebTextInputTypeNone;
 
   if (!focusedFrame->selection().isAvailable()) {
     // "mouse-capture-inside-shadow.html" reaches here.
     return WebTextInputTypeNone;
   }
 
-  // It's important to preserve the equivalence of textInputInfo().type and textInputType(),
-  // so perform the same rootEditableElement() existence check here for consistency.
+  // It's important to preserve the equivalence of textInputInfo().type and
+  // textInputType(), so perform the same rootEditableElement() existence check
+  // here for consistency.
   if (!focusedFrame->selection().selection().rootEditableElement())
     return WebTextInputTypeNone;
 
   Document* document = focusedFrame->document();
   if (!document)
     return WebTextInputTypeNone;
 
   Element* element = document->focusedElement();
   if (!element)
     return WebTextInputTypeNone;
@@ skipping 75 matching lines @@
     const EphemeralRange selectedRange =
         selection.selection().toNormalizedEphemeralRange();
     if (selectedRange.isNull())
       return false;
     anchor = localFrame->editor().firstRectForRange(
         EphemeralRange(selectedRange.startPosition()));
     focus = localFrame->editor().firstRectForRange(
         EphemeralRange(selectedRange.endPosition()));
   }
 
-  // FIXME: This doesn't apply page scale. This should probably be contents to viewport. crbug.com/459293.
+  // FIXME: This doesn't apply page scale. This should probably be contents to
+  // viewport. crbug.com/459293.
   IntRect scaledAnchor(localFrame->view()->contentsToRootFrame(anchor));
   IntRect scaledFocus(localFrame->view()->contentsToRootFrame(focus));
 
   anchor = scaledAnchor;
   focus = scaledFocus;
 
   if (!selection.selection().isBaseFirst())
     std::swap(anchor, focus);
   return true;
 }
@@ skipping 293 matching lines @@
   }
 
   if (!focusedFrame || !focusedFrame->isLocalFrame())
     return WebInputEventResult::NotHandled;
 
   LocalFrame* frame = toLocalFrame(focusedFrame);
 
   WebInputEventResult result = frame->eventHandler().keyEvent(event);
   if (result != WebInputEventResult::NotHandled) {
     if (WebInputEvent::RawKeyDown == event.type) {
-      // Suppress the next keypress event unless the focused node is a plugin node.
-      // (Flash needs these keypress events to handle non-US keyboards.)
+      // Suppress the next keypress event unless the focused node is a plugin
+      // node.  (Flash needs these keypress events to handle non-US keyboards.)
       Element* element = focusedElement();
       if (!element || !element->layoutObject() ||
           !element->layoutObject()->isEmbeddedObject())
         m_suppressNextKeypressEvent = true;
     }
     return result;
   }
 
 #if !OS(MACOSX)
   const WebInputEvent::Type contextMenuKeyTriggeringEventType =
@@ skipping 207 matching lines @@
     m_layerTreeView = m_client->layerTreeView();
   }
 
   if (WebDevToolsAgentImpl* devTools = m_localRoot->devToolsAgentImpl())
     devTools->layerTreeViewChanged(m_layerTreeView);
 
   page()->settings().setAcceleratedCompositingEnabled(m_layerTreeView);
   if (m_layerTreeView)
     page()->layerTreeViewInitialized(*m_layerTreeView);
 
-  // FIXME: only unittests, click to play, Android priting, and printing (for headers and footers)
-  // make this assert necessary. We should make them not hit this code and then delete allowsBrokenNullLayerTreeView.
+  // FIXME: only unittests, click to play, Android priting, and printing (for
+  // headers and footers) make this assert necessary. We should make them not
+  // hit this code and then delete allowsBrokenNullLayerTreeView.
   DCHECK(m_layerTreeView || !m_client ||
          m_client->allowsBrokenNullLayerTreeView());
 }
 
 void WebFrameWidgetImpl::setIsAcceleratedCompositingActive(bool active) {
   // In the middle of shutting down; don't try to spin back up a compositor.
   // FIXME: compositing startup/shutdown should be refactored so that it
   // turns on explicitly rather than lazily, which causes this awkwardness.
   if (m_layerTreeViewClosed)
     return;
@@ skipping 190 matching lines @@
   return flags;
 }
 
 LocalFrame* WebFrameWidgetImpl::focusedLocalFrameAvailableForIme() const {
   if (!m_imeAcceptEvents)
     return nullptr;
   return focusedLocalFrameInWidget();
 }
 
 }  // namespace blink