Put OOBE UI on touch display if no keyboard detected

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

 // Copyright (c) 2011 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.
 
 #ifndef CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_OOBE_UI_H_
 #define CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_OOBE_UI_H_
 
 #include <map>
 #include <memory>
 #include <string>
@@ skipping 35 matching lines @@
 class HIDDetectionView;
 class HostPairingScreenView;
 class KioskAppMenuHandler;
 class KioskAutolaunchScreenView;
 class KioskEnableScreenView;
 class LoginScreenContext;
 class NativeWindowDelegate;
 class NetworkDropdownHandler;
 class NetworkStateInformer;
 class NetworkView;
+class OobeDisplayChooser;
 class SigninScreenHandler;
 class SigninScreenHandlerDelegate;
 class SupervisedUserCreationScreenHandler;
 class ResetView;
 class TermsOfServiceScreenView;
 class UserBoardView;
 class UserImageView;
 class UpdateView;
 class WrongHWIDScreenView;
 
@@ skipping 97 matching lines @@
   }
 
   NetworkStateInformer* network_state_informer_for_test() const {
     return network_state_informer_.get();
   }
 
   // Does ReloadContent() if needed (for example, if material design mode has
   // changed).
   void UpdateLocalizedStringsIfNeeded();
 
+  void OnDisplayAdded(const display::Display& new_display);

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

Add comment.

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

|new_display| is unused, remove it. Is OnDisplayConfigurationChanged a better
name? What happens when we remove a display?

[Felix Ekblom, 2017/05/09 08:54:50]

I've assumed only one touch display for the initial work on this task to
simplify as much as possible. Reacting to OnDisplayRemoved() could be
interesting for a follow up CL, as well as using a better display choosing
heuristic, if we assume multiple connected touch displays. 

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

I renamed the method according to your suggestion as a part of Patch set 3.

+
  private:
   // Lookup a view by its statically registered OobeScreen.
   template <typename TView>
   TView* GetView() {
     OobeScreen expected_screen = TView::kScreenId;
     for (BaseScreenHandler* handler : screen_handlers_) {
       if (expected_screen == handler->oobe_screen())
         return static_cast<TView*>(handler);
     }
 
@@ skipping 51 matching lines @@
   std::vector<base::Closure> ready_callbacks_;
 
   // List of registered observers.
   base::ObserverList<Observer> observer_list_;
 
   // Observer of CrosSettings watching the kRebootOnShutdown policy.
   std::unique_ptr<ShutdownPolicyHandler> shutdown_policy_handler_;
 
   std::unique_ptr<ash::ScreenDimmer> screen_dimmer_;
 
+  std::unique_ptr<OobeDisplayChooser> oobe_display_chooser_;
+
   // Store the deferred JS calls before the screen handler instance is
   // initialized.
   std::unique_ptr<JSCallsContainer> js_calls_container;
 
   DISALLOW_COPY_AND_ASSIGN(OobeUI);
 };
 
 }  // namespace chromeos
 
 #endif  // CHROME_BROWSER_UI_WEBUI_CHROMEOS_LOGIN_OOBE_UI_H_