Introduce CanNotContinueOnNextLine() for DoesContinueOnNextLine()

third_party/WebKit/Source/core/editing/VisibleUnits.cpp

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple 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:
  * 1. Redistributions of source code must retain the above copyright
  *    notice, this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright
@@ skipping 1415 matching lines @@
     }
 
     if (CanBeBackwardCaretPosition(text_layout_object,
                                    current_pos.OffsetInLeafNode())) {
       return current_pos.ComputePosition();
     }
   }
   return last_visible.DeprecatedComputePosition();
 }
 
+// Returns true if |box| at |text_offset| can not continue on next line.
+static bool CanNotContinueOnNextLine(const LayoutText& text_layout_object,
+                                     InlineBox* box,
+                                     unsigned text_offset) {
+  InlineTextBox* const last_text_box = text_layout_object.LastTextBox();
+  if (box == last_text_box)
+    return true;
+  return LineLayoutAPIShim::LayoutObjectFrom(box->GetLineLayoutItem()) ==
+             text_layout_object &&
+         ToInlineTextBox(box)->Start() >= text_offset;
+}
+
 // The text continues on the next line only if the last text box is not on this
 // line and none of the boxes on this line have a larger start offset.
 static bool DoesContinueOnNextLine(const LayoutText& text_layout_object,
                                    InlineBox* box,
                                    unsigned text_offset) {
   InlineTextBox* const last_text_box = text_layout_object.LastTextBox();
   DCHECK_NE(box, last_text_box);
   for (InlineBox* runner = box->NextLeafChild(); runner;
        runner = runner->NextLeafChild()) {
-    if (runner == last_text_box)
-      return false;
-    if (LineLayoutAPIShim::LayoutObjectFrom(runner->GetLineLayoutItem()) ==
-            text_layout_object &&
-        ToInlineTextBox(runner)->Start() >= text_offset)
+    if (CanNotContinueOnNextLine(text_layout_object, runner, text_offset))
       return false;
   }
 
   for (InlineBox* runner = box->PrevLeafChild(); runner;
        runner = runner->PrevLeafChild()) {
-    if (runner == last_text_box)
-      return false;
-    if (LineLayoutAPIShim::LayoutObjectFrom(runner->GetLineLayoutItem()) ==
-            text_layout_object &&
-        ToInlineTextBox(runner)->Start() >= text_offset)
+    if (CanNotContinueOnNextLine(text_layout_object, runner, text_offset))
       return false;
   }
 
   return true;
 }
 
 // TODO(editing-dev): This function is just moved out from
 // |MostBackwardCaretPosition()|. We should study this function more and
 // name it appropriately. See https://trac.webkit.org/changeset/32438/
 // which introduce this.
@@ skipping 611 matching lines @@
 
 IntRect ComputeTextRect(const EphemeralRangeInFlatTree& range) {
   return EnclosingIntRect(ComputeTextRectTemplate(range));
 }
 
 FloatRect ComputeTextFloatRect(const EphemeralRange& range) {
   return ComputeTextRectTemplate(range);
 }
 
 }  // namespace blink