cros: Allow pin keyboard to keep focus without popping up the pin keyboard.

chrome/browser/ui/webui/settings/people_handler.cc

 // Copyright 2015 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/ui/webui/settings/people_handler.h"
 
 #include <string>
 
 #include "base/bind.h"
 #include "base/bind_helpers.h"
@@ skipping 41 matching lines @@
 #include "content/public/browser/web_contents.h"
 #include "content/public/browser/web_contents_delegate.h"
 #include "google_apis/gaia/gaia_auth_util.h"
 #include "google_apis/gaia/gaia_constants.h"
 #include "net/base/url_util.h"
 #include "ui/base/l10n/l10n_util.h"
 #include "ui/base/webui/web_ui_util.h"
 
 #if defined(OS_CHROMEOS)
 #include "components/signin/core/browser/signin_manager_base.h"
+#include "ui/keyboard/keyboard_util.h"
 #else
 #include "components/signin/core/browser/signin_manager.h"
 #endif
 
 using browser_sync::ProfileSyncService;
 using content::WebContents;
 using l10n_util::GetStringFUTF16;
 using l10n_util::GetStringUTF16;
 
 namespace {
@@ skipping 137 matching lines @@
       base::Bind(&PeopleHandler::HandleGetSyncStatus, base::Unretained(this)));
   web_ui()->RegisterMessageCallback(
       "SyncSetupManageOtherPeople",
       base::Bind(&PeopleHandler::HandleManageOtherPeople,
                  base::Unretained(this)));
 #if defined(OS_CHROMEOS)
   web_ui()->RegisterMessageCallback(
       "SyncSetupDoSignOutOnAuthError",
       base::Bind(&PeopleHandler::HandleDoSignOutOnAuthError,
                  base::Unretained(this)));
+  web_ui()->RegisterMessageCallback(
+      "SetForceDisableVirtualKeyboard",
+      base::Bind(&PeopleHandler::HandleSetForceDisableVirtualKeyboard,
+                 base::Unretained(this)));
 #else
   web_ui()->RegisterMessageCallback(
       "SyncSetupStopSyncing",
       base::Bind(&PeopleHandler::HandleStopSyncing, base::Unretained(this)));
   web_ui()->RegisterMessageCallback(
       "SyncSetupStartSignIn",
       base::Bind(&PeopleHandler::HandleStartSignin, base::Unretained(this)));
 #endif
 }
 
@@ skipping 279 matching lines @@
   OpenSyncSetup(false /* creating_supervised_user */);
 }
 
 #if defined(OS_CHROMEOS)
 // On ChromeOS, we need to sign out the user session to fix an auth error, so
 // the user goes through the real signin flow to generate a new auth token.
 void PeopleHandler::HandleDoSignOutOnAuthError(const base::ListValue* args) {
   DVLOG(1) << "Signing out the user to fix a sync error.";
   chrome::AttemptUserExit();
 }
+
+void PeopleHandler::HandleSetForceDisableVirtualKeyboard(
+    const base::ListValue* args) {
+  bool disable_keyboard = false;
+  args->GetBoolean(0, &disable_keyboard);
+  keyboard::SetForceDisableVirtualKeyboardForWebUi(disable_keyboard);

[xiyuan, 2016/11/09 18:06:32]

Do we need to make sure this is reverted in PeopleHandler's dtor similar to
CoreOobeHandler? Doing it in js might be not be enough, e.g. user closes the
webui while the pin dialog is visible.

[sammiequon, 2016/11/10 21:06:01]

Decided to remove blocking keyboard in settings, since user may switch tabs with
the pin dialog on and not be able to auto bring up the keyboard, and the smaller
pin dialog looks ok with the keyboard and will not be used too often.

+}
 #endif
 
 #if !defined(OS_CHROMEOS)
 void PeopleHandler::HandleStartSignin(const base::ListValue* args) {
   AllowJavascript();
 
   // Should only be called if the user is not already signed in.
   DCHECK(!SigninManagerFactory::GetForProfile(profile_)->IsAuthenticated());
   bool creating_supervised_user = false;
   args->GetBoolean(0, &creating_supervised_user);
@@ skipping 385 matching lines @@
   base::FilePath profile_file_path = profile_->GetPath();
   ProfileMetrics::LogProfileSyncSignIn(profile_file_path);
 
   // We're done configuring, so notify ProfileSyncService that it is OK to
   // start syncing.
   sync_blocker_.reset();
   service->SetFirstSetupComplete();
 }
 
 }  // namespace settings