Ensure clean layout before calling TextIterator::rangeLength in editing code

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

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2012 Apple Inc. All rights reserved.
  * Copyright (C) 2005 Alexey Proskuryakov.
  *
  * 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 158 matching lines @@
     // because of text fields and textareas. The DOM for those is not
     // directly in the document DOM, so ensure that the range does not cross a
     // boundary of one of those.
     Node* startContainer = range.startPosition().computeContainerNode();
     if (startContainer != &scope && !startContainer->isDescendantOf(&scope))
         return PlainTextRange();
     Node* endContainer = range.endPosition().computeContainerNode();
     if (endContainer != scope && !endContainer->isDescendantOf(&scope))
         return PlainTextRange();
 
+    // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
+    // needs to be audited.  See http://crbug.com/590369 for more details.
+    scope.document().updateStyleAndLayoutIgnorePendingStylesheets();

[yosin_UTC9, 2016/08/30 07:40:24]

I hope following patch will hoist this updateLayout to call sites of
PlainTextRange::create

[Xiaocheng, 2016/08/30 07:42:52]

Yeah, there will be such a patch.

+
     size_t start = TextIterator::rangeLength(Position(&const_cast<ContainerNode&>(scope), 0), range.startPosition());
     size_t end = TextIterator::rangeLength(Position(&const_cast<ContainerNode&>(scope), 0), range.endPosition());
 
     return PlainTextRange(start, end);
 }
 
 PlainTextRange PlainTextRange::create(const ContainerNode& scope, const Range& range)
 {
     return create(scope, EphemeralRange(&range));
 }
 
 } // namespace blink