Re-land Reformat comments in core/layout up until LayoutBox

third_party/WebKit/Source/core/layout/HitTestResult.cpp

Index: third_party/WebKit/Source/core/layout/HitTestResult.cpp
diff --git a/third_party/WebKit/Source/core/layout/HitTestResult.cpp b/third_party/WebKit/Source/core/layout/HitTestResult.cpp
index 3b4b0fee06af912cef36010664578094b17850a5..51e0fd88c3ff825195f8cd7064d38816ed04619d 100644
--- a/third_party/WebKit/Source/core/layout/HitTestResult.cpp
+++ b/third_party/WebKit/Source/core/layout/HitTestResult.cpp
@@ -238,7 +238,8 @@ bool HitTestResult::isSelected() const {
 String HitTestResult::title(TextDirection& dir) const {
   dir = LTR;
   // Find the title in the nearest enclosing DOM node.
-  // For <area> tags in image maps, walk the tree for the <area>, not the <img> using it.
+  // For <area> tags in image maps, walk the tree for the <area>, not the <img>
+  // using it.
   if (m_innerNode.get())
     m_innerNode->updateDistribution();
   for (Node* titleNode = m_innerNode.get(); titleNode;
@@ -303,9 +304,10 @@ KURL HitTestResult::absoluteImageURL() const {
     return KURL();
 
   AtomicString urlString;
-  // Always return a url for image elements and input elements with type=image, even if they
-  // don't have a LayoutImage (e.g. because the image didn't load and we are using an alt container).
-  // For other elements we don't create alt containers so ensure they contain a loaded image.
+  // Always return a url for image elements and input elements with type=image,
+  // even if they don't have a LayoutImage (e.g. because the image didn't load
+  // and we are using an alt container). For other elements we don't create alt
+  // containers so ensure they contain a loaded image.
   if (isHTMLImageElement(*innerNodeOrImageMapImage) ||
       (isHTMLInputElement(*innerNodeOrImageMapImage) &&
        toHTMLInputElement(innerNodeOrImageMapImage)->type() ==
@@ -362,10 +364,11 @@ String HitTestResult::textContent() const {
   return m_innerURLElement->textContent();
 }
 
-// FIXME: This function needs a better name and may belong in a different class. It's not
-// really isContentEditable(); it's more like needsEditingContextMenu(). In many ways, this
-// function would make more sense in the ContextMenu class, except that WebElementDictionary
-// hooks into it. Anyway, we should architect this better.
+// FIXME: This function needs a better name and may belong in a different class.
+// It's not really isContentEditable(); it's more like needsEditingContextMenu.
+// In many ways, this function would make more sense in the ContextMenu class,
+// except that WebElementDictionary hooks into it. Anyway, we should architect
+// this better.
 bool HitTestResult::isContentEditable() const {
   if (!m_innerNode)
     return false;
@@ -469,9 +472,10 @@ void HitTestResult::resolveRectBasedTest(
   m_innerPossiblyPseudoNode = nullptr;
   m_listBasedTestResult = nullptr;
 
-  // Update the HitTestResult as if the supplied node had been hit in normal point-based hit-test.
-  // Note that we don't know the local point after a rect-based hit-test, but we never use
-  // it so shouldn't bother with the cost of computing it.
+  // Update the HitTestResult as if the supplied node had been hit in normal
+  // point-based hit-test.
+  // Note that we don't know the local point after a rect-based hit-test, but we
+  // never use it so shouldn't bother with the cost of computing it.
   resolvedInnerNode->layoutObject()->updateHitTestResult(*this, LayoutPoint());
   ASSERT(!isRectBasedTest());
 }