Prune CreateVisiblePositionDeprecated from Editor::firstRectForRange

third_party/WebKit/Source/web/WebFrameWidgetImpl.cpp

 /*
  * Copyright (C) 2014 Google 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:
  *
  *     * Redistributions of source code must retain the above copyright
  * notice, this list of conditions and the following disclaimer.
  *     * Redistributions in binary form must reproduce the above
@@ skipping 733 matching lines @@
 
 // TODO(ekaramad):This method is almost duplicated in WebViewImpl as well. This
 // code needs to be refactored  (http://crbug.com/629721).
 bool WebFrameWidgetImpl::selectionBounds(WebRect& anchor, WebRect& focus) const
 {
     const LocalFrame* localFrame = focusedLocalFrameInWidget();
     if (!localFrame)
         return false;
 
     FrameSelection& selection = localFrame->selection();
+    if (selection.isNone())
+        return false;
+
+    // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
+    // needs to be audited.  See http://crbug.com/590369 for more details.
+    localFrame->document()->updateStyleAndLayoutIgnorePendingStylesheets();
+
+    DocumentLifecycle::DisallowTransitionScope disallowTransition(localFrame->document()->lifecycle());
 
     if (selection.isCaret()) {
-        // TODO(xiaochengh): The use of updateStyleAndLayoutIgnorePendingStylesheets
-        // needs to be audited.  See http://crbug.com/590369 for more details.
-        localFrame->document()->updateStyleAndLayoutIgnorePendingStylesheets();
-
         anchor = focus = selection.absoluteCaretBounds();
     } else {
         const EphemeralRange selectedRange = selection.selection().toNormalizedEphemeralRange();
         if (selectedRange.isNull())
             return false;
         anchor = localFrame->editor().firstRectForRange(EphemeralRange(selectedRange.startPosition()));
         focus = localFrame->editor().firstRectForRange(EphemeralRange(selectedRange.endPosition()));
     }
 
     // FIXME: This doesn't apply page scale. This should probably be contents to viewport. crbug.com/459293.
@@ skipping 723 matching lines @@
 }
 
 LocalFrame* WebFrameWidgetImpl::focusedLocalFrameAvailableForIme() const
 {
     if (!m_imeAcceptEvents)
         return nullptr;
     return focusedLocalFrameInWidget();
 }
 
 } // namespace blink