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

Index: chrome/browser/extensions/api/input_ime/input_ime_api.cc
diff --git a/chrome/browser/extensions/api/input_ime/input_ime_api.cc b/chrome/browser/extensions/api/input_ime/input_ime_api.cc
index b35997b4cc6de409e4fbbcce37f2634897ca521a..dcd6ba4689de31ef4587e4a9eb670f65912ffee0 100644
--- a/chrome/browser/extensions/api/input_ime/input_ime_api.cc
+++ b/chrome/browser/extensions/api/input_ime/input_ime_api.cc
@@ -6,6 +6,8 @@
 
 #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"
@@ -144,6 +146,22 @@ class ImeObserver : public InputMethodEngineInterface::Observer {
 
     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());
   }