Allow input.ime.activate() to be called from a non-user-action for the first time.

chrome/browser/extensions/api/input_ime/input_ime_api_nonchromeos.cc

Index: chrome/browser/extensions/api/input_ime/input_ime_api_nonchromeos.cc
diff --git a/chrome/browser/extensions/api/input_ime/input_ime_api_nonchromeos.cc b/chrome/browser/extensions/api/input_ime/input_ime_api_nonchromeos.cc
index f60069b547aade20d8ee9189f9f51828226ea8f2..0ba5432059bc071b8333d1cfa39d5d9e3c5e11e5 100644
--- a/chrome/browser/extensions/api/input_ime/input_ime_api_nonchromeos.cc
+++ b/chrome/browser/extensions/api/input_ime/input_ime_api_nonchromeos.cc
@@ -46,6 +46,10 @@ const char kErrorNotCalledFromUserAction[] =
 // A preference determining whether to hide the warning bubble next time.
 const char kPrefWarningBubbleNeverShow[] = "skip_ime_warning_bubble";
 
+// A preference to see whether it is the first time to call input.ime.activate
+// since the extension is loaded.
+const char kPrefImeActivatedCalledBefore[] = "ime_activate_called_before";
+
 bool IsInputImeEnabled() {
   return !base::CommandLine::ForCurrentProcess()->HasSwitch(
       switches::kDisableInputImeAPI);
@@ -123,6 +127,10 @@ void InputImeAPI::OnExtensionLoaded(content::BrowserContext* browser_context,
                                     const Extension* extension) {
   // No-op if called multiple times.
   ui::IMEBridge::Initialize();
+  Profile* profile = Profile::FromBrowserContext(browser_context);
+  ExtensionPrefs::Get(profile)->UpdateExtensionPref(
+      extension->id(), kPrefImeActivatedCalledBefore,
+      new base::FundamentalValue(false));
 }
 
 void InputImeAPI::OnExtensionUnloaded(content::BrowserContext* browser_context,
@@ -194,7 +202,31 @@ ExtensionFunction::ResponseAction InputImeActivateFunction::Run() {
   if (!event_router)
     return RespondNow(Error(kErrorNoActiveEngine));
 
-  // This API is only allowed to be called from a user action.
+  ExtensionPrefs* prefs = ExtensionPrefs::Get(profile);
+  bool allowed_to_activate = false;

[Shu Chen, 2016/04/01 08:10:52]

s/allowed_to_activate/warning_bubble_never_show/g

[Azure Wei, 2016/04/01 13:34:04]

Done.

+  bool ime_activated_called = false;
+
+  if (prefs->ReadPrefAsBoolean(extension_id(), kPrefImeActivatedCalledBefore,

[Shu Chen, 2016/04/01 08:10:52]

I think this block should also be after the disable_bubble_for_testing_ check?

[Azure Wei, 2016/04/01 13:34:04]

If put this after disable_bubble_for_testing_, it won't change anything as it
still needs to check user_gesture() every time.
p.s., tests use ScopedUserGestureForTests to make user_gesture() return true.

+                               &ime_activated_called) &&
+      prefs->ReadPrefAsBoolean(extension_id(), kPrefWarningBubbleNeverShow,
+                               &allowed_to_activate) &&
+      !ime_activated_called &&
+      allowed_to_activate) {
+    // If it the first time that the extension call input.ime.activate() since
+    // loaded, we allow it to be automatically activated from a non user action.
+    event_router->SetActiveEngine(extension_id());
+    ExtensionPrefs::Get(profile)->UpdateExtensionPref(
+        extension_id(), kPrefImeActivatedCalledBefore,
+        new base::FundamentalValue(true));
+    return RespondNow(NoArguments());
+  }
+
+  ExtensionPrefs::Get(profile)->UpdateExtensionPref(
+      extension_id(), kPrefImeActivatedCalledBefore,
+      new base::FundamentalValue(true));
+
+  // If not the first time, this API is only allowed to be called from a user
+  // action.
   if (!user_gesture())
     return RespondNow(Error(kErrorNotCalledFromUserAction));
 
@@ -204,11 +236,7 @@ ExtensionFunction::ResponseAction InputImeActivateFunction::Run() {
     return RespondNow(NoArguments());
   }
 
-  ExtensionPrefs* prefs = ExtensionPrefs::Get(profile);
-  bool allowed_to_activate = false;
-  if (prefs->ReadPrefAsBoolean(extension_id(), kPrefWarningBubbleNeverShow,
-                               &allowed_to_activate) &&
-      allowed_to_activate) {
+  if (allowed_to_activate) {
     // If user allows to activate the extension without showing the warning
     // bubble, sets the active engine directly.
     // Otherwise, the extension will be activated when the user presses the 'OK'