Refreshes the caret bounds on window move for IME UI.

ui/views/widget/widget.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 "ui/views/widget/widget.h"
 
 #include "base/debug/trace_event.h"
 #include "base/logging.h"
 #include "base/message_loop/message_loop.h"
 #include "base/strings/utf_string_conversions.h"
@@ skipping 1070 matching lines @@
 gfx::Size Widget::GetMinimumSize() {
   return non_client_view_ ? non_client_view_->GetMinimumSize() : gfx::Size();
 }
 
 gfx::Size Widget::GetMaximumSize() {
   return non_client_view_ ? non_client_view_->GetMaximumSize() : gfx::Size();
 }
 
 void Widget::OnNativeWidgetMove() {
   widget_delegate_->OnWidgetMove();
+  View* focused_view = GetRootView()->GetFocusManager()->GetFocusedView();
+  if (focused_view && focused_view->GetInputMethod())
+    focused_view->GetInputMethod()->OnCaretBoundsChanged(focused_view);
   FOR_EACH_OBSERVER(WidgetObserver, observers_, OnWidgetBoundsChanged(
     this,
     GetWindowBoundsInScreen()));
 }
 
 void Widget::OnNativeWidgetSizeChanged(const gfx::Size& new_size) {
   root_view_->SetSize(new_size);
 

[oshima, 2014/04/23 05:55:44]

You may also want to update when the size is changed, which may cause the
re-layout?

[Jun Mukai, 2014/04/23 22:23:04]

I don't think so, each view can update the boundary upon its re-layout. For
example, views::Textfield does this at OnBoundsChanged()

[oshima, 2014/04/23 22:54:15]

Are you sure it gets called when only parent bounds has changed?

[Jun Mukai, 2014/04/23 23:16:02]

Sorry, I got unsure about your point.
The widget size change causes RootView::SetSize (see above), and it causes root
view's bounds change (and Layout). Whether this causes
Textfield::OnBoundsChanged or not depends on how the views are structured.
However, if the widget's size change causes the change of the bounds of
textfield (or whatever) in the widget, OnBoundsChanged() has to be called.

[Jun Mukai, 2014/04/23 23:26:48]

chatted offline and I was wrong.
There could be a case that resize just moves a container parent which contains a
textfield. In that case, the container's bounds will be changed but this keeps
textfield in it.
Added.

   // Size changed notifications can fire prior to full initialization
   // i.e. during session restore.  Avoid saving session state during these
   // startup procedures.
   if (native_widget_initialized_)
     SaveWindowPlacement();
 
   FOR_EACH_OBSERVER(WidgetObserver, observers_, OnWidgetBoundsChanged(
     this,
     GetWindowBoundsInScreen()));
 }
@@ skipping 354 matching lines @@
 
 ////////////////////////////////////////////////////////////////////////////////
 // internal::NativeWidgetPrivate, NativeWidget implementation:
 
 internal::NativeWidgetPrivate* NativeWidgetPrivate::AsNativeWidgetPrivate() {
   return this;
 }
 
 }  // namespace internal
 }  // namespace views