Add Phone as a security key sheet view

Add sheet view to show when prompting users to use their phone as a
security key to authenticate via the WebAuthN API.

Bug: 847985
Change-Id: I0bb605bf470456f5f29111834f69a71abfbe0430
Reviewed-on: https://chromium-review.googlesource.com/1177445
Commit-Queue: Jun Choi <hongjunchoi@chromium.org>
Reviewed-by: Balazs Engedy <engedy@chromium.org>
Cr-Commit-Position: refs/heads/master@{#583892}

chrome/app/generated_resources.grd

@@ -9177,6 +9177,12 @@ Please help our engineers fix this problem. Tell us what happened right before y
       <message name="IDS_WEBAUTHN_TRANSPORT_POPUP_CABLE" desc="Menu item text. The user selects this to verify their identity on a web site (i.e. sign in) using their phone. They wirelessly connect their phone to their computer over Bluetooth Low Energy, and then use the phone to sign in to web sites on their computer.">
         Verify via your phone
       </message>
+      <message name="IDS_WEBAUTHN_CABLE_ACTIVATE_TITLE" desc="Title of the dialog shown when the user tries to sign-in using a phone as a security key.">
+        Check your phone
+      </message>
+      <message name="IDS_WEBAUTHN_CABLE_ACTIVATE_DESCRIPTION" desc="Contents of the dialog shown when the user tries to sign-in using a phone as a security key.">
+        A notification was sent to your phone. Follow the prompts to confirm it's you.
+      </message>
    </if>
    <if expr="is_macosx">
      <message name="IDS_WEBAUTHN_TOUCH_ID_TITLE" desc="Title of the dialog shown when the user tries to sign in with Touch ID." meaning="'Touch ID' is the fingerprint recognition feature in macOS. Try to refer Apple support documentation in the target language for the appropriate product name translation.">

chrome/browser/ui/views/webauthn/sheet_view_factory.cc

@@ -88,6 +88,10 @@ std::unique_ptr<AuthenticatorRequestSheetView> CreateSheetViewForCurrentStepOf(
       sheet_view = std::make_unique<AuthenticatorRequestSheetView>(
           std::make_unique<AuthenticatorTouchIdSheetModel>(dialog_model));
       break;
+    case Step::kCableActivate:
+      sheet_view = std::make_unique<AuthenticatorRequestSheetView>(
+          std::make_unique<AuthenticatorPaaskSheetModel>(dialog_model));
+      break;
     case Step::kCompleted:
     case Step::kBlePowerOnAutomatic:
       sheet_view = std::make_unique<AuthenticatorRequestSheetView>(

chrome/browser/ui/webauthn/authenticator_dialog_browsertest.cc

@@ -63,6 +63,9 @@ class AuthenticatorDialogTest : public DialogBrowserTest {
           AuthenticatorRequestDialogModel::Step::kBleActivate);
     } else if (name == "touchid") {
       model->SetCurrentStep(AuthenticatorRequestDialogModel::Step::kTouchId);
+    } else if (name == "cable_activate") {
+      model->SetCurrentStep(
+          AuthenticatorRequestDialogModel::Step::kCableActivate);
     }
 
     ShowAuthenticatorRequestDialog(
@@ -128,3 +131,7 @@ IN_PROC_BROWSER_TEST_F(AuthenticatorDialogTest, InvokeUi_ble_activate) {
 IN_PROC_BROWSER_TEST_F(AuthenticatorDialogTest, InvokeUi_touchid) {
   ShowAndVerifyUi();
 }
+
+IN_PROC_BROWSER_TEST_F(AuthenticatorDialogTest, InvokeUi_cable_activate) {
+  ShowAndVerifyUi();
+}

chrome/browser/ui/webauthn/sheet_models.cc

@@ -368,3 +368,17 @@ base::string16 AuthenticatorTouchIdSheetModel::GetStepTitle() const {
 base::string16 AuthenticatorTouchIdSheetModel::GetStepDescription() const {
   return base::string16();
 }
+
+// AuthenticatorPaaskSheetModel -----------------------------------------
+
+gfx::ImageSkia* AuthenticatorPaaskSheetModel::GetStepIllustration() const {
+  return GetImage(IDR_WEBAUTHN_ILLUSTRATION_PHONE_1X);
+}
+
+base::string16 AuthenticatorPaaskSheetModel::GetStepTitle() const {
+  return l10n_util::GetStringUTF16(IDS_WEBAUTHN_CABLE_ACTIVATE_TITLE);
+}
+
+base::string16 AuthenticatorPaaskSheetModel::GetStepDescription() const {
+  return l10n_util::GetStringUTF16(IDS_WEBAUTHN_CABLE_ACTIVATE_DESCRIPTION);
+}

chrome/browser/ui/webauthn/sheet_models.h

@@ -207,4 +207,15 @@ class AuthenticatorTouchIdSheetModel : public AuthenticatorSheetModelBase {
   base::string16 GetStepDescription() const override;
 };
 
+class AuthenticatorPaaskSheetModel : public AuthenticatorSheetModelBase {
+ public:
+  using AuthenticatorSheetModelBase::AuthenticatorSheetModelBase;
+
+ private:
+  // AuthenticatorSheetModelBase:
+  gfx::ImageSkia* GetStepIllustration() const override;
+  base::string16 GetStepTitle() const override;
+  base::string16 GetStepDescription() const override;
+};
+
 #endif  // CHROME_BROWSER_UI_WEBAUTHN_SHEET_MODELS_H_

chrome/browser/webauthn/authenticator_request_dialog_model.cc

@@ -45,6 +45,9 @@ void AuthenticatorRequestDialogModel::StartGuidedFlowForTransport(
     case AuthenticatorTransport::kInternal:
       SetCurrentStep(Step::kTouchId);
       break;
+    case AuthenticatorTransport::kCloudAssistedBluetoothLowEnergy:
+      SetCurrentStep(Step::kCableActivate);
+      break;
     default:
       break;
   }

chrome/browser/webauthn/authenticator_request_dialog_model.h

@@ -46,6 +46,9 @@ class AuthenticatorRequestDialogModel {
 
     // Touch ID.
     kTouchId,
+
+    // Phone as a security key.
+    kCableActivate,
   };
 
   // Encapsulates information about authenticators that have been found but are