Make IME extensions decide when to show the keyboard.

ash/common/system/chromeos/ime_menu/ime_menu_tray.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 "ash/common/system/chromeos/ime_menu/ime_menu_tray.h"
 
 #include "ash/common/accessibility_delegate.h"
 #include "ash/common/ash_constants.h"
 #include "ash/common/material_design/material_design_controller.h"
 #include "ash/common/session/session_state_delegate.h"
@@ skipping 415 matching lines @@
   // Overrides the keyboard url ref to make it shown with the given keyset.
   if (InputMethodManager::Get())
     InputMethodManager::Get()->OverrideKeyboardUrlRef(keyset);
 
   // If onscreen keyboard has been enabled, shows the keyboard directly.
   keyboard::KeyboardController* keyboard_controller =
       keyboard::KeyboardController::GetInstance();
   show_keyboard_ = true;
   if (keyboard_controller) {
     keyboard_controller->AddObserver(this);
-    keyboard_controller->ShowKeyboard(false);
+    // If the keyboard hasn't been loaded yet, it means it's the first time to
+    // show keyboard, OS need to trigger it. Otherwise, make IME extensions

[Shu Chen, 2016/12/26 02:16:39]

// If the keyboard window hasn't been created yet, it means the extension cannot
receive anything to show the keyboard. Therefore, instead of relying the
extension to show the keyboard, forcibly show the keyboard window here (which
will cause the keyboard window created).

[Azure Wei, 2016/12/26 06:53:30]

Done.

+    // decide when to show the keyboard.
+    if (!keyboard_controller->KeyboardHasBeenLoaded()) {

[Shu Chen, 2016/12/26 02:16:39]

s/KeyboardHasBeenLoaded/IsKeyboardWindowCreated/g

[Azure Wei, 2016/12/26 06:53:30]

Done.

+      keyboard_controller->ShowKeyboard(false);
+    }
     return;
   }
 
   AccessibilityDelegate* accessibility_delegate =
       WmShell::Get()->accessibility_delegate();
   // Fails to show the keyboard.
   if (accessibility_delegate->IsVirtualKeyboardEnabled())
     return;
 
   // Onscreen keyboard has not been enabled yet, forces to bring out the
@@ skipping 140 matching lines @@
   WmShell::Get()->system_tray_delegate()->GetCurrentIME(&current_ime_);
 
   // Updates the tray label based on the current input method.
   if (current_ime_.third_party)
     label_->SetText(current_ime_.short_name + base::UTF8ToUTF16("*"));
   else
     label_->SetText(current_ime_.short_name);
 }
 
 }  // namespace ash