Move chromeos::EventRewriter to //ui/chromeos/events

chrome/browser/chromeos/events/event_rewriter_delegate_impl.cc

+// Copyright 2017 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/chromeos/events/event_rewriter_delegate_impl.h"
+
+#include "chrome/browser/chromeos/login/ui/login_display_host.h"
+#include "chrome/browser/extensions/extension_commands_global_registry.h"
+#include "chrome/browser/profiles/profile_manager.h"
+#include "chrome/browser/ui/webui/chromeos/login/oobe_ui.h"
+#include "chrome/browser/ui/webui/chromeos/login/signin_screen_handler.h"
+#include "chrome/common/pref_names.h"
+#include "components/prefs/pref_service.h"
+#include "components/user_manager/user_manager.h"
+
+namespace chromeos {
+
+EventRewriterDelegateImpl::EventRewriterDelegateImpl()
+    : pref_service_for_testing_(nullptr) {}
+
+EventRewriterDelegateImpl::~EventRewriterDelegateImpl() {}
+
+bool EventRewriterDelegateImpl::RewriteModifierKeys() {
+  // Do nothing if we have just logged in as guest but have not restarted chrome
+  // process yet (so we are still on the login screen). In this situations we
+  // have no user profile so can not do anything useful.
+  // Note that currently, unlike other accounts, when user logs in as guest, we
+  // restart chrome process. In future this is to be changed.
+  // TODO(glotov): remove the following condition when we do not restart chrome
+  // when user logs in as guest.
+  // TODO(kpschoedel): check whether this is still necessary.
+  if (user_manager::UserManager::Get()->IsLoggedInAsGuest() &&
+      LoginDisplayHost::default_host())
+    return false;
+  return true;
+}
+
+bool EventRewriterDelegateImpl::GetKeyboardRemappedPrefValue(
+    const std::string& pref_name,
+    int* value) const {
+  DCHECK(value);
+  // If we're at the login screen, try to get the pref from the global prefs
+  // dictionary.
+  if (LoginDisplayHost::default_host() &&
+      LoginDisplayHost::default_host()
+          ->GetOobeUI()
+          ->signin_screen_handler()
+          ->GetKeyboardRemappedPrefValue(pref_name, value))
+    return true;
+  const PrefService* pref_service = GetPrefService();
+  if (!pref_service)
+    return false;
+  const PrefService::Preference* preference =
+      pref_service->FindPreference(pref_name);
+  if (!preference)
+    return false;
+
+  DCHECK_EQ(preference->GetType(), base::Value::Type::INTEGER);
+  *value = preference->GetValue()->GetInt();
+  return true;
+}
+
+bool EventRewriterDelegateImpl::TopRowKeysAreFunctionKeys() const {
+  const PrefService* pref_service = GetPrefService();
+  if (!pref_service)
+    return false;
+
+  const PrefService::Preference* preference =
+      pref_service->FindPreference(prefs::kLanguageSendFunctionKeys);
+  if (!preference)
+    return false;
+  DCHECK_EQ(preference->GetType(), base::Value::Type::BOOLEAN);
+  return preference->GetValue()->GetBool();

[sadrul, 2017/03/14 01:15:41]

Just use preference->GetBoolean(...), instead of DCHECK GetType() and
GetValue()->GetBool()?

[Peng, 2017/03/14 16:06:52]

I checked the code and found the preference object doesn't have GetBoolean.

[sadrul, 2017/03/16 05:02:43]

Sorry, yeah, I was referring to using PrefService::GetBoolean() [1], which is
what the current code uses.

1:
https://cs.chromium.org/chromium/src/components/prefs/pref_service.h?type=cs&...

[Peng, 2017/03/16 15:44:58]

Done.

+}
+
+bool EventRewriterDelegateImpl::IsExtensionCommandRegistered(
+    ui::KeyboardCode key_code,
+    int flags) const {
+  // Some keyboard events for ChromeOS get rewritten, such as:
+  // Search+Shift+Left gets converted to Shift+Home (BeginDocument).
+  // This doesn't make sense if the user has assigned that shortcut
+  // to an extension. Because:
+  // 1) The extension would, upon seeing a request for Ctrl+Shift+Home have
+  //    to register for Shift+Home, instead.
+  // 2) The conversion is unnecessary, because Shift+Home (BeginDocument) isn't
+  //    going to be executed.
+  // Therefore, we skip converting the accelerator if an extension has
+  // registered for this shortcut.
+  Profile* profile = ProfileManager::GetActiveUserProfile();
+  if (!profile || !extensions::ExtensionCommandsGlobalRegistry::Get(profile))
+    return false;
+
+  const int modifiers = flags & (ui::EF_SHIFT_DOWN | ui::EF_CONTROL_DOWN |
+                                 ui::EF_ALT_DOWN | ui::EF_COMMAND_DOWN);
+  ui::Accelerator accelerator(key_code, modifiers);
+  return extensions::ExtensionCommandsGlobalRegistry::Get(profile)
+      ->IsRegistered(accelerator);
+}
+
+const PrefService* EventRewriterDelegateImpl::GetPrefService() const {
+  if (pref_service_for_testing_)
+    return pref_service_for_testing_;
+  Profile* profile = ProfileManager::GetActiveUserProfile();
+  return profile ? profile->GetPrefs() : nullptr;
+}
+
+}  // namespace chromeos