Showing virtual keyboard when input type changed

components/arc/ime/arc_ime_service.cc

 // Copyright 2016 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 "components/arc/ime/arc_ime_service.h"
 
 #include "base/logging.h"
 #include "components/arc/ime/arc_ime_bridge_impl.h"
 #include "components/exo/shell_surface.h"
 #include "components/exo/surface.h"
@@ skipping 115 matching lines @@
 // Overridden from arc::ArcImeBridge::Delegate
 
 void ArcImeService::OnTextInputTypeChanged(ui::TextInputType type) {
   if (ime_type_ == type)
     return;
   ime_type_ = type;
 
   ui::InputMethod* const input_method = GetInputMethod();
   if (input_method) {
     input_method->OnTextInputTypeChanged(this);
+    // TODO(crbug.com/581282): Remove this piggyback call when
+    // ImeInstance::ShowImeIfNeeded is wired to ARC.
     if (input_method->GetTextInputClient() == this &&
         ime_type_ != ui::TEXT_INPUT_TYPE_NONE) {

[dcheng, 2016/06/14 08:39:52]

So the final intent is to remove this once all the IME state is synced, right?

[takaoka, 2016/06/14 09:04:25]

Yes.
This can be removed once the corresponding CL on Android side (ag/1113933) has
been merged, new ShowImeIfNeeded RPC will be called from Android side as needed.

-      // TODO(kinaba): crbug.com/581282. This is tentative short-term solution.
-      //
-      // For fully correct implementation, rather than to piggyback the "show"
-      // request with the input type change, we need dedicated IPCs to share the
-      // virtual keyboard show/hide states between Chromium and ARC.
       input_method->ShowImeIfNeeded();
     }
   }
 }
 
 void ArcImeService::OnCursorRectChanged(const gfx::Rect& rect) {
   if (cursor_rect_ == rect)
     return;
   cursor_rect_ = rect;
 
   ui::InputMethod* const input_method = GetInputMethod();
   if (input_method)
     input_method->OnCaretBoundsChanged(this);
 }
 
 void ArcImeService::OnCancelComposition() {
   ui::InputMethod* const input_method = GetInputMethod();
   if (input_method)
     input_method->CancelComposition(this);
 }
 
+void ArcImeService::ShowImeIfNeeded() {
+  ui::InputMethod* const input_method = GetInputMethod();
+  if (input_method && input_method->GetTextInputClient() == this) {
+    input_method->ShowImeIfNeeded();
+  }
+}
+
 ////////////////////////////////////////////////////////////////////////////////
 // Oberridden from ui::TextInputClient:
 
 void ArcImeService::SetCompositionText(
     const ui::CompositionText& composition) {
   has_composition_text_ = !composition.text.empty();
   ime_bridge_->SendSetCompositionText(composition);
 }
 
 void ArcImeService::ConfirmCompositionText() {
@@ skipping 105 matching lines @@
 bool ArcImeService::ChangeTextDirectionAndLayoutAlignment(
     base::i18n::TextDirection direction) {
   return false;
 }
 
 bool ArcImeService::IsEditCommandEnabled(int command_id) {
   return false;
 }
 
 }  // namespace arc