Prune createVisiblePositionDeprecated from EditingUtilities

third_party/WebKit/Source/core/editing/commands/InsertListCommand.cpp

Index: third_party/WebKit/Source/core/editing/commands/InsertListCommand.cpp
diff --git a/third_party/WebKit/Source/core/editing/commands/InsertListCommand.cpp b/third_party/WebKit/Source/core/editing/commands/InsertListCommand.cpp
index 039777856d1be5fd214f58712ed40f8c55afe50e..3e407fe0f933d310ca1cdcf9097bdfbf48ed6820 100644
--- a/third_party/WebKit/Source/core/editing/commands/InsertListCommand.cpp
+++ b/third_party/WebKit/Source/core/editing/commands/InsertListCommand.cpp
@@ -68,12 +68,15 @@ HTMLUListElement* InsertListCommand::fixOrphanedListChild(
 HTMLElement* InsertListCommand::mergeWithNeighboringLists(
     HTMLElement* passedList,
     EditingState* editingState) {
+  document().updateStyleAndLayoutIgnorePendingStylesheets();

[yosin_UTC9, 2016/10/12 07:32:00]

It is better to call updateSAL() just before |canMergeLists()|.

[Xiaocheng, 2016/10/12 07:45:56]

Done.

+
   HTMLElement* list = passedList;
   Element* previousList = ElementTraversal::previousSibling(*list);
   if (canMergeLists(previousList, list)) {
     mergeIdenticalElements(previousList, list, editingState);
     if (editingState->isAborted())
       return nullptr;
+    document().updateStyleAndLayoutIgnorePendingStylesheets();

[yosin_UTC9, 2016/10/12 07:32:00]

Please move this updateSAL() just before |canMergeLists()| at L90.

Two calls of updateSAL() should not be slow.

[Xiaocheng, 2016/10/12 07:45:57]

Done.

   }
 
   if (!list)
@@ -530,6 +533,7 @@ void InsertListCommand::listifyParagraph(const VisiblePosition& originalStart,
     if (editingState->isAborted())
       return;
 
+    document().updateStyleAndLayoutIgnorePendingStylesheets();
     if (canMergeLists(previousList, nextList))
       mergeIdenticalElements(previousList, nextList, editingState);