Ensure clean layout for all (but deprecated) functions creating VisiblePositions

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

 /*
  * Copyright (C) 2004, 2005, 2006, 2007, 2008, 2009 Apple Inc. All rights
  * reserved.
  * Portions Copyright (c) 2011 Motorola Mobility, 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.
@@ skipping 79 matching lines @@
   const PositionWithAffinityTemplate<Strategy> upstreamPosition(
       deepPosition, TextAffinity::Upstream);
   if (inSameLine(downstreamPosition, upstreamPosition))
     return VisiblePositionTemplate<Strategy>(downstreamPosition);
   return VisiblePositionTemplate<Strategy>(upstreamPosition);
 }
 
 template <typename Strategy>
 VisiblePositionTemplate<Strategy> VisiblePositionTemplate<Strategy>::afterNode(
     Node* node) {
-  // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
-  // needs to be audited.  See http://crbug.com/590369 for more details.
-  if (node)
-    node->document().updateStyleAndLayoutIgnorePendingStylesheets();
-
   return create(PositionWithAffinityTemplate<Strategy>(
       PositionTemplate<Strategy>::afterNode(node)));
 }
 
 template <typename Strategy>
 VisiblePositionTemplate<Strategy> VisiblePositionTemplate<Strategy>::beforeNode(
     Node* node) {
-  // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
-  // needs to be audited.  See http://crbug.com/590369 for more details.
-  if (node)
-    node->document().updateStyleAndLayoutIgnorePendingStylesheets();
-
   return create(PositionWithAffinityTemplate<Strategy>(
       PositionTemplate<Strategy>::beforeNode(node)));
 }
 
 template <typename Strategy>
 VisiblePositionTemplate<Strategy>
 VisiblePositionTemplate<Strategy>::firstPositionInNode(Node* node) {
-  // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
-  // needs to be audited.  See http://crbug.com/590369 for more details.
-  if (node)
-    node->document().updateStyleAndLayoutIgnorePendingStylesheets();
-
   return create(PositionWithAffinityTemplate<Strategy>(
       PositionTemplate<Strategy>::firstPositionInNode(node)));
 }
 
 template <typename Strategy>
 VisiblePositionTemplate<Strategy>
 VisiblePositionTemplate<Strategy>::inParentAfterNode(const Node& node) {
-  // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
-  // needs to be audited.  See http://crbug.com/590369 for more details.
-  node.document().updateStyleAndLayoutIgnorePendingStylesheets();
-
   return create(PositionWithAffinityTemplate<Strategy>(
       PositionTemplate<Strategy>::inParentAfterNode(node)));
 }
 
 template <typename Strategy>
 VisiblePositionTemplate<Strategy>
 VisiblePositionTemplate<Strategy>::inParentBeforeNode(const Node& node) {
-  // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
-  // needs to be audited.  See http://crbug.com/590369 for more details.
-  node.document().updateStyleAndLayoutIgnorePendingStylesheets();
-
   return create(PositionWithAffinityTemplate<Strategy>(
       PositionTemplate<Strategy>::inParentBeforeNode(node)));
 }
 
 template <typename Strategy>
 VisiblePositionTemplate<Strategy>
 VisiblePositionTemplate<Strategy>::lastPositionInNode(Node* node) {
-  // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
-  // needs to be audited.  See http://crbug.com/590369 for more details.
-  if (node)
-    node->document().updateStyleAndLayoutIgnorePendingStylesheets();
-
   return create(PositionWithAffinityTemplate<Strategy>(
       PositionTemplate<Strategy>::lastPositionInNode(node)));
 }
 
 VisiblePosition createVisiblePositionDeprecated(const Position& position,
                                                 TextAffinity affinity) {
   // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
   // needs to be audited.  See http://crbug.com/590369 for more details.
   if (position.isNotNull())
     position.document()->updateStyleAndLayoutIgnorePendingStylesheets();
@@ skipping 106 matching lines @@
     return;
   }
   DVLOG(0) << "Cannot showTree for (nil) VisiblePosition.";
 }
 
 void showTree(const blink::VisiblePosition& vpos) {
   vpos.showTreeForThis();
 }
 
 #endif