Put OOBE UI on touch display if no keyboard detected

chrome/browser/ui/webui/chromeos/login/oobe_display_chooser.h

+// Copyright (c) 2017 The Chromium Authors. All rights reserved.

[oshima, 2017/05/09 05:58:45]

nit: nuke "(c)"

It's old style and obsolete. Same for the other files.

+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_OOBE_DISPLAY_CHOOSER_H_
+#define CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_OOBE_DISPLAY_CHOOSER_H_
+
+#include <memory>
+
+#include "base/macros.h"
+
+namespace ash {
+class DisplayConfigurationController;
+}  // namespace ash
+
+namespace display {
+class Display;
+class DisplayManager;
+class Screen;
+}  // namespace display
+
+namespace chromeos {
+
+// Tries to put the OOBE UI on a connected touch display (if available).
+class OobeDisplayChooser {
+ public:
+  static std::unique_ptr<OobeDisplayChooser> CreateDefault();

[oshima, 2017/05/09 05:58:45]

Since this is the only usage, you can just call it "Create()"

+
+  OobeDisplayChooser(ash::DisplayConfigurationController* dcc,

[jdufault, 2017/05/08 17:44:58]

Add comment

+                     display::DisplayManager* display_manager,
+                     display::Screen* screen);

[oshima, 2017/05/09 05:58:45]

does this have to be public?

+  ~OobeDisplayChooser();
+
+  void TryToPlaceUIOnTouchDisplay();
+  void OnDisplayAdded(const display::Display& new_display);
+
+ private:
+  void MoveToTouchDisplay();
+
+  ash::DisplayConfigurationController* configuration_controller_;
+  display::DisplayManager* display_manager_;
+  display::Screen* screen_;
+
+  DISALLOW_COPY_AND_ASSIGN(OobeDisplayChooser);
+};
+
+}  // namespace chromeos
+
+#endif  // CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_OOBE_DISPLAY_CHOOSER_H_