Get rid of redundant functions updateIfNeeded() in FrameSelection and SelectionEditor

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

 /*
  * Copyright (C) 2004, 2008, 2009, 2010 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
  *    notice, this list of conditions and the following disclaimer in the
@@ skipping 1133 matching lines @@
           .setIsHandleVisible(isHandleVisible())
           .build();
 
   if (newSelection.isNone())
     return;
 
   setSelection(newSelection, CloseTyping | ClearTypingStyle,
                CursorAlignOnScroll::IfNeeded, granularity);
 }
 
-void FrameSelection::updateIfNeeded() {

[Xiaocheng, 2017/02/25 02:45:46]

We should also remove this function from FS.h

-  DCHECK(!m_frame->document()->needsLayoutTreeUpdate());
-  m_selectionEditor->updateIfNeeded();
-}
-
 void FrameSelection::setCaretVisible(bool caretIsVisible) {
   m_frameCaret->setCaretVisibility(caretIsVisible ? CaretVisibility::Visible
                                                   : CaretVisibility::Hidden);
 }
 
 void FrameSelection::setCaretBlinkingSuspended(bool suspended) {
   m_frameCaret->setCaretBlinkingSuspended(suspended);
 }
 
 bool FrameSelection::isCaretBlinkingSuspended() const {
@@ skipping 21 matching lines @@
 }
 
 void showTree(const blink::FrameSelection* sel) {
   if (sel)
     sel->showTreeForThis();
   else
     LOG(INFO) << "Cannot showTree for <null> FrameSelection.";
 }
 
 #endif