Leverage OnFocus event to let component IME extension know about the current screen type (e.g. logi…

chrome/browser/extensions/api/input_ime/input_ime_api.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/extensions/api/input_ime/input_ime_api.h"
 
 #include "base/strings/string_number_conversions.h"
 #include "base/values.h"
+#include "chrome/browser/chromeos/login/lock/screen_locker.h"
+#include "chrome/browser/chromeos/login/users/user_manager.h"
 #include "chrome/browser/chromeos/profiles/profile_helper.h"
 #include "chrome/browser/extensions/extension_service.h"
 #include "chrome/browser/profiles/profile_manager.h"
 #include "chrome/common/extensions/api/input_ime.h"
 #include "chrome/common/extensions/api/input_ime/input_components_handler.h"
 #include "extensions/browser/event_router.h"
 #include "extensions/browser/extension_function_registry.h"
 #include "extensions/browser/extension_registry.h"
 #include "extensions/browser/extension_system.h"
 #include "extensions/common/manifest_handlers/background_info.h"
@@ skipping 118 matching lines @@
       const InputMethodEngineInterface::InputContext& context) OVERRIDE {
     if (profile_ == NULL || extension_id_.empty())
       return;
 
     input_ime::InputContext context_value;
     context_value.context_id = context.id;
     context_value.type = input_ime::InputContext::ParseType(context.type);
 
     scoped_ptr<base::ListValue> args(input_ime::OnFocus::Create(context_value));
 
+    // The component IME extensions need to know the current screen type (e.g.
+    // lock screen, login screen, etc.) so that its on-screen keyboard page
+    // won't open new windows/pages. See crbug.com/395621.
+    base::DictionaryValue* val = NULL;
+    if (args->GetDictionary(0, &val)) {
+      std::string screen_type;
+      if (!UserManager::Get()->IsUserLoggedIn()) {
+        screen_type = "login";
+      } else if (chromeos::ScreenLocker::default_screen_locker() &&
+               chromeos::ScreenLocker::default_screen_locker()->locked()) {
+        screen_type = "lock";
+      }

[Nikita (slow), 2014/07/25 09:23:24]

You should add another type here "multi-profiles login".

chromeos::UserAddingScreen::Get()->IsRunning();

[Shu Chen, 2014/07/25 09:41:06]

Thanks, I will create another cl to support it.

+      if (!screen_type.empty())
+        val->SetStringWithoutPathExpansion("screen", screen_type);
+    }
+
     DispatchEventToExtension(profile_, extension_id_,
                              input_ime::OnFocus::kEventName, args.Pass());
   }
 
   virtual void OnBlur(int context_id) OVERRIDE {
     if (profile_ == NULL || extension_id_.empty())
       return;
 
     scoped_ptr<base::ListValue> args(input_ime::OnBlur::Create(context_id));
 
@@ skipping 704 matching lines @@
       input_ime_event_router()->GetActiveEngine(details.extension_id);
   if (engine)
     engine->NotifyImeReady();
 }
 
 InputImeEventRouter* InputImeAPI::input_ime_event_router() {
   return InputImeEventRouter::GetInstance();
 }
 
 }  // namespace extensions