MD Settings: First half of Device > Keyboard sub-page.

chrome/browser/ui/webui/settings/chromeos/keyboard_handler.cc

+// Copyright 2016 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/chromeos/keyboard_handler.h"
+
+#include "base/bind.h"
+#include "base/command_line.h"
+#include "base/values.h"
+#include "chrome/browser/profiles/profile.h"
+#include "chromeos/chromeos_switches.h"
+#include "content/public/browser/web_ui.h"
+#include "ui/events/devices/device_data_manager.h"
+#include "ui/events/devices/keyboard_device.h"
+
+
+namespace chromeos {
+namespace settings {
+namespace {

[dschuyler, 2016/03/02 01:08:06]

I've usually seen anonymous namespaces placed between
the includes and the named namespaces.  Should/could
this be moved up or does it need to be here (or I'm
I wrong about what is the desired layout)?

[michaelpg, 2016/03/02 02:42:06]

If I understand you correctly, I don't think it matters much, and I see the
pattern used here pretty often.

But in this case, HasExternalKeyboard doesn't actually use the chromeos or
settings namespace, so it's fine and maybe slightly better the other way.

+
+bool HasExternalKeyboard() {
+  for (const ui::KeyboardDevice& keyboard :
+       ui::DeviceDataManager::GetInstance()->keyboard_devices()) {
+    if (keyboard.type == ui::InputDeviceType::INPUT_DEVICE_EXTERNAL)
+      return true;
+  }
+
+  return false;
+}
+
+}  // namespace
+
+KeyboardHandler::KeyboardHandler(content::WebUI* webui)
+    : profile_(Profile::FromWebUI(webui)) {
+  ui::DeviceDataManager::GetInstance()->AddObserver(this);
+}
+
+KeyboardHandler::~KeyboardHandler() {
+  ui::DeviceDataManager::GetInstance()->RemoveObserver(this);
+}
+
+void KeyboardHandler::RegisterMessages() {
+  web_ui()->RegisterMessageCallback(
+      "initializeKeyboardSettings",
+      base::Bind(&KeyboardHandler::HandleInitialize,
+                 base::Unretained(this)));
+}
+
+void KeyboardHandler::OnKeyboardDeviceConfigurationChanged() {
+  UpdateShowKeys();
+}
+
+void KeyboardHandler::HandleInitialize(const base::ListValue* args) {
+  UpdateShowKeys();
+}
+
+void KeyboardHandler::UpdateShowKeys() const {
+  const base::FundamentalValue has_caps_lock(HasExternalKeyboard());
+  const base::FundamentalValue has_diamond_key(
+      base::CommandLine::ForCurrentProcess()->HasSwitch(
+          chromeos::switches::kHasChromeOSDiamondKey));
+  web_ui()->CallJavascriptFunction("cr.webUIListenerCallback",
+                                   base::StringValue("show-keys-changed"),
+                                   has_caps_lock,
+                                   has_diamond_key);
+}
+
+}  // namespace settings
+}  // namespace chromeos