Change |EnsureCaretInRect| to |EnsureCaretNotInRect|.

content/browser/renderer_host/render_widget_host_view_aura.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "content/browser/renderer_host/render_widget_host_view_aura.h"
 
 #include <set>
 #include <utility>
 
 #include "base/auto_reset.h"
@@ skipping 1406 matching lines @@
   return true;
 }
 
 void RenderWidgetHostViewAura::ExtendSelectionAndDelete(
     size_t before, size_t after) {
   RenderFrameHostImpl* rfh = GetFocusedFrame();
   if (rfh)
     rfh->ExtendSelectionAndDelete(before, after);
 }
 
-void RenderWidgetHostViewAura::EnsureCaretInRect(const gfx::Rect& rect) {
-  gfx::Rect intersected_rect(
+void RenderWidgetHostViewAura::EnsureCaretNotInRect(const gfx::Rect& rect) {
+  gfx::Rect hiding_area_in_this_window(
       gfx::IntersectRects(rect, window_->GetBoundsInScreen()));
 
-  if (intersected_rect.IsEmpty())
+  if (hiding_area_in_this_window.IsEmpty())
     return;
 
   host_->ScrollFocusedEditableNodeIntoRect(
-      ConvertRectFromScreen(intersected_rect));
+      ConvertRectFromScreen(gfx::SubtractRects(window_->GetBoundsInScreen(),
+                                               hiding_area_in_this_window)));

[sadrul, 2016/12/08 00:57:29]

If you change |rect| from screen-space to local-space at the beginning of this
function, then you don't need to use GetBoundsInScreen(), or convert it back to
local-space here again using ConvertRectFromScreen(). I think doing that would
make this simpler.

[yhanada, 2016/12/08 03:31:36]

Done.

 }
 
 bool RenderWidgetHostViewAura::IsTextEditCommandEnabled(
     ui::TextEditCommand command) const {
   return false;
 }
 
 void RenderWidgetHostViewAura::SetTextEditCommandForNextKeyEvent(
     ui::TextEditCommand command) {}
 
@@ skipping 942 matching lines @@
 
 void RenderWidgetHostViewAura::SetPopupChild(
     RenderWidgetHostViewAura* popup_child_host_view) {
   popup_child_host_view_ = popup_child_host_view;
   event_handler_->SetPopupChild(
       popup_child_host_view,
       popup_child_host_view ? popup_child_host_view->event_handler() : nullptr);
 }
 
 }  // namespace content