reflow comments in web/

third_party/WebKit/Source/web/TextFinder.cpp

 /*
  * Copyright (C) 2009 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 130 matching lines @@
       (options.forward ? 0 : Backwards) |
       (options.matchCase ? 0 : CaseInsensitive) |
       (wrapWithinFrame ? WrapAround : 0) |
       (options.wordStart ? AtWordStarts : 0) |
       (options.medialCapitalAsWordStart ? TreatMedialCapitalAsWordStart : 0) |
       (options.findNext ? 0 : StartInSelection);
   m_activeMatch = ownerFrame().frame()->editor().findStringAndScrollToVisible(
       searchText, m_activeMatch.get(), findOptions);
 
   if (!m_activeMatch) {
-    // If we're finding next the next active match might not be in the current frame.
-    // In this case we don't want to clear the matches cache.
+    // If we're finding next the next active match might not be in the current
+    // frame.  In this case we don't want to clear the matches cache.
     if (!options.findNext)
       clearFindMatchesCache();
 
     ownerFrame().frameView()->invalidatePaintForTickmarks();
     return false;
   }
 
   // If the user is browsing a page with autosizing, adjust the zoom to the
   // column where the next hit has been found. Doing this when autosizing is
   // not set will result in a zoom reset on small devices.
@@ skipping 100 matching lines @@
         identifier, m_activeMatchIndex + 1, WebAXObject(startObject),
         m_activeMatch->startOffset(), WebAXObject(endObject),
         m_activeMatch->endOffset());
   }
 }
 
 void TextFinder::scopeStringMatches(int identifier,
                                     const WebString& searchText,
                                     const WebFindOptions& options,
                                     bool reset) {
-  // TODO(dglazkov): The reset/continue cases need to be untangled into two separate functions. This collation of logic
-  // is unnecessary and adds to overall complexity of the code.
+  // TODO(dglazkov): The reset/continue cases need to be untangled into two
+  // separate functions. This collation of logic is unnecessary and adds to
+  // overall complexity of the code.
   if (reset) {
     // This is a brand new search, so we need to reset everything.
     // Scoping is just about to begin.
     m_scopingInProgress = true;
 
     // Need to keep the current identifier locally in order to finish the
     // request in case the frame is detached during the process.
     m_findRequestIdentifier = identifier;
 
     // Clear highlighting for this frame.
@@ skipping 33 matching lines @@
   if (m_resumeScopingFromRange) {
     // This is a continuation of a scoping operation that timed out and didn't
     // complete last time around, so we should start from where we left off.
     DCHECK(m_resumeScopingFromRange->collapsed());
     searchStart = fromPositionInDOMTree<EditingInFlatTreeStrategy>(
         m_resumeScopingFromRange->endPosition());
     if (searchStart.document() != searchEnd.document())
       return;
   }
 
-  // TODO(dglazkov): The use of updateStyleAndLayoutIgnorePendingStylesheets needs to be audited.
-  // see http://crbug.com/590369 for more details.
+  // TODO(dglazkov): The use of updateStyleAndLayoutIgnorePendingStylesheets
+  // needs to be audited.  see http://crbug.com/590369 for more details.
   searchStart.document()->updateStyleAndLayoutIgnorePendingStylesheets();
 
   // This timeout controls how long we scope before releasing control. This
   // value does not prevent us from running for longer than this, but it is
   // periodically checked to see if we have exceeded our allocated time.
   const double maxScopingDuration = 0.1;  // seconds
 
   int matchCount = 0;
   bool timedOut = false;
   double startTime = currentTime();
   do {
     // Find next occurrence of the search string.
     // FIXME: (http://crbug.com/6818) This WebKit operation may run for longer
     // than the timeout value, and is not interruptible as it is currently
     // written. We may need to rewrite it with interruptibility in mind, or
     // find an alternative.
     const EphemeralRangeInFlatTree result =
         findPlainText(EphemeralRangeInFlatTree(searchStart, searchEnd),
                       searchText, options.matchCase ? 0 : CaseInsensitive);
     if (result.isCollapsed()) {
       // Not found.
       break;
     }
     Range* resultRange = Range::create(
         result.document(), toPositionInDOMTree(result.startPosition()),
         toPositionInDOMTree(result.endPosition()));
     if (resultRange->collapsed()) {
-      // resultRange will be collapsed if the matched text spans over multiple TreeScopes.
-      // FIXME: Show such matches to users.
+      // resultRange will be collapsed if the matched text spans over multiple
+      // TreeScopes.  FIXME: Show such matches to users.
       searchStart = result.endPosition();
       continue;
     }
 
     ++matchCount;
 
     // Catch a special case where Find found something but doesn't know what
     // the bounding box for it is. In this case we set the first match we find
     // as the active rect.
     IntRect resultBounds = resultRange->boundingBox();
@@ skipping 168 matching lines @@
     filteredMatches.reserveCapacity(m_findMatchesCache.size() - deadMatches);
 
     for (const FindMatch& match : m_findMatchesCache) {
       if (!match.m_rect.isEmpty())
         filteredMatches.append(match);
     }
 
     m_findMatchesCache.swap(filteredMatches);
   }
 
-  // Invalidate the rects in child frames. Will be updated later during traversal.
+  // Invalidate the rects in child frames. Will be updated later during
+  // traversal.
   if (!m_findMatchRectsAreValid)
     for (WebFrame* child = ownerFrame().firstChild(); child;
          child = child->nextSibling())
       toWebLocalFrameImpl(child)->ensureTextFinder().m_findMatchRectsAreValid =
           false;
 
   m_findMatchRectsAreValid = true;
 }
 
 WebFloatRect TextFinder::activeFindMatchRect() {
@@ skipping 63 matching lines @@
 
     // Set this frame as the active frame (the one with the active highlight).
     m_currentActiveMatchFrame = true;
     ownerFrame().viewImpl()->setFocusedFrame(&ownerFrame());
 
     if (m_activeMatch)
       setMarkerActive(m_activeMatch.get(), false);
     m_activeMatch = range;
     setMarkerActive(m_activeMatch.get(), true);
 
-    // Clear any user selection, to make sure Find Next continues on from the match we just activated.
+    // Clear any user selection, to make sure Find Next continues on from the
+    // match we just activated.
     ownerFrame().frame()->selection().clear();
 
     // Make sure no node is focused. See http://crbug.com/38700.
     ownerFrame().frame()->document()->clearFocusedElement();
   }
 
   IntRect activeMatchRect;
   IntRect activeMatchBoundingBox = enclosingIntRect(
       LayoutObject::absoluteBoundingBoxRectForRange(m_activeMatch.get()));
 
@@ skipping 131 matching lines @@
 
 DEFINE_TRACE(TextFinder) {
   visitor->trace(m_ownerFrame);
   visitor->trace(m_activeMatch);
   visitor->trace(m_resumeScopingFromRange);
   visitor->trace(m_deferredScopingWork);
   visitor->trace(m_findMatchesCache);
 }
 
 }  // namespace blink