Put OOBE UI on touch display if no keyboard detected

chrome/browser/ui/webui/chromeos/login/oobe_display_chooser.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/ui/webui/chromeos/login/oobe_display_chooser.h"
+
+#include "ash/display/display_configuration_controller.h"
+#include "ash/shell.h"
+#include "ui/display/display.h"
+#include "ui/display/manager/display_manager.h"
+#include "ui/display/screen.h"
+
+namespace chromeos {
+
+namespace {
+
+bool TouchSupportAvailable(const display::Display& display) {
+  return display.touch_support() ==
+         display::Display::TouchSupport::TOUCH_SUPPORT_AVAILABLE;
+}
+
+}  // namespace
+
+OobeDisplayChooser::OobeDisplayChooser() {}
+
+OobeDisplayChooser::~OobeDisplayChooser() {}
+
+void OobeDisplayChooser::TryToPlaceUiOnTouchDisplay() {
+  display::Display primary_display =
+      display::Screen::GetScreen()->GetPrimaryDisplay();
+
+  if (primary_display.is_valid() && !TouchSupportAvailable(primary_display))
+    MoveToTouchDisplay();
+}
+
+void OobeDisplayChooser::MoveToTouchDisplay() {
+  auto& displays =

[jdufault, 2017/05/09 18:59:38]

nit: auto& type is not obvious

+      ash::Shell::Get()->display_manager()->active_only_display_list();
+
+  if (displays.size() <= 1)
+    return;
+
+  for (auto& display : displays) {
+    if (TouchSupportAvailable(display)) {
+      ash::Shell::Get()
+          ->display_configuration_controller()
+          ->SetPrimaryDisplayId(display.id());

[oshima, 2017/05/10 00:14:31]

IIRC that this change will be saved locally, so
this will become a default when it's rebooted even with keyboard connected.

Just FYI.

[Felix Ekblom, 2017/05/10 10:59:13]

Thanks for pointing it out. I believe this to be OK for our use cases. If some
problematic edge case turn up I'll revisit this in a bug fix.

+      break;
+    }
+  }
+}
+
+}  // namespace chromeos