SetHostConfiguration() and ConfigureHost() are moved from ScreenObserver.

chrome/browser/chromeos/login/screens/host_pairing_screen.h

Index: chrome/browser/chromeos/login/screens/host_pairing_screen.h
diff --git a/chrome/browser/chromeos/login/screens/host_pairing_screen.h b/chrome/browser/chromeos/login/screens/host_pairing_screen.h
index d1f469d5a98b777c0ffb74cca736613580d77799..44059198aa9e0c22dd863b0e790d52a6b4c270f8 100644
--- a/chrome/browser/chromeos/login/screens/host_pairing_screen.h
+++ b/chrome/browser/chromeos/login/screens/host_pairing_screen.h
@@ -18,10 +18,22 @@ class HostPairingScreen
       public pairing_chromeos::HostPairingController::Observer,
       public HostPairingScreenActor::Delegate {
  public:
+  class Delegate {
+   public:
+    virtual ~Delegate() {}
+    virtual void ConfigureHost(bool accepted_eula,
+                               const std::string& lang,
+                               const std::string& timezone,
+                               bool send_reports,
+                               const std::string& keyboard_layout) = 0;
+  };
+
   HostPairingScreen(ScreenObserver* observer, HostPairingScreenActor* actor,
                     pairing_chromeos::HostPairingController* remora_controller);
   virtual ~HostPairingScreen();
 
+  void SetDelegate(Delegate* delegate);

[achuithb, 2014/10/27 18:50:57]

I think passing this into the ctor cuts a few unnecessary lines of code.

+
  private:
   typedef pairing_chromeos::HostPairingController::Stage Stage;
 
@@ -54,6 +66,8 @@ class HostPairingScreen
   // Controller performing pairing. Owned by the wizard controller.
   pairing_chromeos::HostPairingController* remora_controller_;
 
+  Delegate* delegate_;
+
   // Current stage of pairing process.
   Stage current_stage_;