[Easy signin] Wire up userClick auth attempt to easy unlock app and back

chrome/browser/signin/screenlock_bridge.h

Index: chrome/browser/signin/screenlock_bridge.h
diff --git a/chrome/browser/signin/screenlock_bridge.h b/chrome/browser/signin/screenlock_bridge.h
index d732c0e147937944ac6a3556a2dffbfa9f13f48a..a8c7aafe155b273c0119ece80ecd346a86e6d305 100644
--- a/chrome/browser/signin/screenlock_bridge.h
+++ b/chrome/browser/signin/screenlock_bridge.h
@@ -21,6 +21,8 @@ class Profile;
 // ScreenlockBridge brings together the screenLockPrivate API and underlying
 // support. On ChromeOS, it delegates calls to the ScreenLocker. On other
 // platforms, it delegates calls to UserManagerUI (and friends).
+// TODO(tbarzic): Rename ScreenlockBridge to SignInScreenBridge, as this is not
+// used solely for the lock screen anymore.
 class ScreenlockBridge {
  public:
   class Observer {
@@ -133,6 +135,11 @@ class ScreenlockBridge {
     // Unlock from easy unlock app for a user.
     virtual void Unlock(const std::string& user_email) = 0;
 
+    // Attempts to login the user using an easy unlock key.
+    virtual void AttemptUserClickLogin(const std::string& user_email,

[Tim Song, 2014/09/23 19:37:59]

I was a bit confused by this name because the actual click is handled in
SignInScreenHandler. Maybe rename it to "AttemptEasyLogin" for clarity?

[tbarzic, 2014/09/23 19:58:35]

I named it "AttemptUserClickLogin" because the AuthType it's supposed to handle
is called USER_CLICK, but I see your point. I'll rename it (especially since
auth flow was named EASY_UNLOCK)..

[tbarzic, 2014/09/23 22:09:08]

Done.

+                                       const std::string& secret,
+                                       const std::string& key_label) = 0;
+
    protected:
     virtual ~LockHandler() {}
   };