Stop using CreateVisiblePosition in FrameCaret::AbsoluteCaretBounds

FrameCaret::CaretPosition() is already canonicalized. There's no need
to canonicalize it again.

Bug: 897983
Change-Id: I3d58572099fa17b7aada06fe799989f6f8fe67f5
Reviewed-on: https://chromium-review.googlesource.com/c/1297633Reviewed-by: Yoshifumi Inoue <yosin@chromium.org>
Reviewed-by: Yoichi Osato <yoichio@chromium.org>
Commit-Queue: Xiaocheng Hu <xiaochengh@chromium.org>
Cr-Commit-Position: refs/heads/master@{#602366}

third_party/blink/renderer/core/editing/frame_caret.cc

@@ -181,9 +181,7 @@ IntRect FrameCaret::AbsoluteCaretBounds() const {
   if (!IsActive())
     return AbsoluteBoundsForLocalRect(caret_node, LayoutRect());
   return AbsoluteBoundsForLocalRect(
-      caret_node,
-      CaretDisplayItemClient::ComputeCaretRect(
-          CreateVisiblePosition(CaretPosition()).ToPositionWithAffinity()));
+      caret_node, CaretDisplayItemClient::ComputeCaretRect(CaretPosition()));
 }
 
 void FrameCaret::SetShouldShowBlockCursor(bool should_show_block_cursor) {