Adding mojo calls for easy unlock service

chrome/browser/ui/ash/lock_screen_client.h

 // 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.
 
 #ifndef CHROME_BROWSER_UI_ASH_LOCK_SCREEN_CLIENT_H_
 #define CHROME_BROWSER_UI_ASH_LOCK_SCREEN_CLIENT_H_
 
 #include "ash/public/interfaces/lock_screen.mojom.h"
 #include "base/macros.h"
+#include "base/memory/weak_ptr.h"
+#include "chrome/browser/chromeos/login/ui/views/user_board_view.h"
 #include "mojo/public/cpp/bindings/binding.h"
 
 // Handles method calls delegated back to chrome from ash. Also notifies ash of
 // relevant state changes in chrome.
-class LockScreenClient : public ash::mojom::LockScreenClient {
+class LockScreenClient : public ash::mojom::LockScreenClient,
+                         public chromeos::UserBoardView {

[xiyuan, 2017/05/31 17:51:19]

Can we have a separate UserBoardView impl/proxy instead of piggybacking on
LockScreenClient? It is fine to have ShowUserPodCustomIcon etc calls on
LockScreenClient but it feels strange to be a UserBoardView.

[xiaoyinh(OOO Sep 11-29), 2017/06/02 23:42:29]

Thanks! I added UserSelectionScreenProxy as a separate UserBoardView impl.
Please let me know if this makes sense.

  public:
   LockScreenClient();
   ~LockScreenClient() override;
 
+  // Handles method calls coming from ash into chrome.
+  class Delegate {
+   public:
+    Delegate();
+    virtual ~Delegate();
+    virtual void HandleAttemptUnlock(const AccountId& account_id) = 0;
+    virtual void HandleHardlockPod(const AccountId& account_id) = 0;
+    virtual void HandleRecordClickOnLockIcon(const AccountId& account_id) = 0;
+
+   private:
+    DISALLOW_COPY_AND_ASSIGN(Delegate);
+  };
+
   static LockScreenClient* Get();
 
   // ash::mojom::LockScreenClient:
   void AuthenticateUser(const AccountId& account_id,
                         const std::string& hashed_password,
                         bool authenticated_by_pin) override;
+  void AttemptUnlock(const AccountId& account_id) override;
+  void HardlockPod(const AccountId& account_id) override;
+  void RecordClickOnLockIcon(const AccountId& account_id) override;
+
+  // UserBoardView:
+  void SetPublicSessionDisplayName(const AccountId& account_id,
+                                   const std::string& display_name) override{};
+  void SetPublicSessionLocales(const AccountId& account_id,
+                               std::unique_ptr<base::ListValue> locales,
+                               const std::string& default_locale,
+                               bool multiple_recommended_locales) override{};
+  void ShowBannerMessage(const base::string16& message) override{};
+  void ShowUserPodCustomIcon(const AccountId& account_id,
+                             const base::DictionaryValue& icon) override;
+  void HideUserPodCustomIcon(const AccountId& account_id) override;
+  void SetAuthType(
+      const AccountId& account_id,
+      proximity_auth::ScreenlockBridge::LockHandler::AuthType auth_type,
+      const base::string16& initial_value) override;
+  void Bind(chromeos::UserSelectionScreen* screen) override{};
+  void Unbind() override{};
+  base::WeakPtr<UserBoardView> GetWeakPtr() override;
 
   // Wrappers around the mojom::SystemTray interface.

[xiyuan, 2017/05/31 17:51:19]

nit: This comment looks wrong.

[xiaoyinh(OOO Sep 11-29), 2017/06/02 23:42:29]

Thanks! Changed.

   void ShowErrorMessage(int32_t login_attempts,
                         const std::string& error_text,
                         const std::string& help_link_text,
                         int32_t help_topic_id);
   void ClearErrors();
+  void LoadUsers(const base::ListValue& users_list, bool show_guest);
+  void SetDelegate(Delegate* delegate);
 
  private:
   // Lock screen mojo service in ash.
   ash::mojom::LockScreenPtr lock_screen_;
 
   // Binds this object to the client interface.
   mojo::Binding<ash::mojom::LockScreenClient> binding_;
+  Delegate* delegate_ = nullptr;
 
   DISALLOW_COPY_AND_ASSIGN(LockScreenClient);
 };
 
 #endif  // CHROME_BROWSER_UI_ASH_LOCK_SCREEN_CLIENT_H_