Support multi display candidate window.

chrome/browser/chromeos/input_method/candidate_window.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 "chrome/browser/chromeos/input_method/candidate_window.h"
 
 #include <algorithm>
 #include <string>
 #include <vector>
 
@@ skipping 17 matching lines @@
 #include "ui/views/controls/textfield/textfield.h"
 #include "ui/views/layout/box_layout.h"
 #include "ui/views/layout/fill_layout.h"
 #include "ui/views/layout/grid_layout.h"
 #include "ui/views/widget/widget.h"
 #include "ui/views/window/non_client_view.h"
 
 #if defined(USE_ASH)
 #include "ash/shell.h"
 #include "ash/shell_window_ids.h"
+#include "ash/wm/coordinate_conversion.h"
 #include "ash/wm/window_animations.h"
 #endif  // USE_ASH
 
 namespace chromeos {
 namespace input_method {
 
 namespace {
 
 // Colors used in the candidate window UI.
 const SkColor kFrameColor = SkColorSetRGB(0x96, 0x96, 0x96);
@@ skipping 1614 matching lines @@
   const int kKeepPositionThreshold = 2;  // px
   const gfx::Rect& last_location =
       candidate_window_->cursor_location();
   const int delta_y = abs(last_location.y() - cursor_location.y());
   if ((last_location.x() == cursor_location.x()) &&
       (delta_y <= kKeepPositionThreshold)) {
     DVLOG(1) << "Ignored set_cursor_location signal to prevent window shake";
     return;
   }
 
+#if defined(USE_ASH)
+  aura::RootWindow* root_window =
+      ash::wm::GetRootWindowAt(cursor_location.origin());
+  if (root_window) {
+    gfx::NativeView current_container = ash::Shell::GetContainer(
+        root_window,
+        ash::internal::kShellWindowId_InputMethodContainer);
+    views::Widget::ReparentNativeView(frame_->GetNativeView(),

[mazda, 2012/09/12 16:49:45]

Is it necessary to do this here?
I think ScreenPositionController does reparenting in Widget::SetBounds if bounds
on another root window is set in screen coordinates.

[Seigo Nonaka, 2012/09/12 17:24:50]

Seems transient window like CandidateWindow does not move RootWindow by
ScreenPositionController::SetBounds.

http://code.google.com/searchframe#OAMlx_jo-ck/src/ash/display/screen_positio...

Add comment and moved to ResizeAndMoveParentFrame.

On 2012/09/12 16:49:45, mazda wrote:

+                                      current_container);
+  }
+#endif
+
   // Remember the cursor location.
   candidate_window_->set_cursor_location(cursor_location);
   candidate_window_->set_composition_head_location(composition_head);
   // Move the window per the cursor location.
   candidate_window_->ResizeAndMoveParentFrame();
   infolist_window_->ResizeAndMoveParentFrame();
 }
 
 void CandidateWindowControllerImpl::OnUpdateAuxiliaryText(
     const std::string& utf8_text,
@@ skipping 87 matching lines @@
 }
 
 // static
 CandidateWindowController*
 CandidateWindowController::CreateCandidateWindowController() {
   return new CandidateWindowControllerImpl;
 }
 
 }  // namespace input_method
 }  // namespace chromeos