Revert of Credential Manager: Return the first valid item from the PasswordStore.

components/password_manager/content/browser/content_credential_manager_dispatcher.cc

 // Copyright 2014 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.
 
 #include "components/password_manager/content/browser/content_credential_manager_dispatcher.h"
 
 #include "base/strings/string16.h"
 #include "base/strings/utf_string_conversions.h"
 #include "components/autofill/core/common/password_form.h"
 #include "components/password_manager/content/common/credential_manager_messages.h"
@@ skipping 28 matching lines @@
     IPC_MESSAGE_HANDLER(CredentialManagerHostMsg_NotifySignedOut,
                         OnNotifySignedOut);
     IPC_MESSAGE_HANDLER(CredentialManagerHostMsg_RequestCredential,
                         OnRequestCredential);
     IPC_MESSAGE_UNHANDLED(handled = false)
   IPC_END_MESSAGE_MAP()
   return handled;
 }
 
 void ContentCredentialManagerDispatcher::OnNotifyFailedSignIn(
-    int request_id, const CredentialInfo&) {
+    int request_id,
+    const password_manager::CredentialInfo&) {
   // TODO(mkwst): This is a stub.
   web_contents()->GetRenderViewHost()->Send(
       new CredentialManagerMsg_AcknowledgeFailedSignIn(
           web_contents()->GetRenderViewHost()->GetRoutingID(), request_id));
 }
 
 void ContentCredentialManagerDispatcher::OnNotifySignedIn(
-    int request_id, const CredentialInfo&) {
+    int request_id,
+    const password_manager::CredentialInfo&) {
   // TODO(mkwst): This is a stub.
   web_contents()->GetRenderViewHost()->Send(
       new CredentialManagerMsg_AcknowledgeSignedIn(
           web_contents()->GetRenderViewHost()->GetRoutingID(), request_id));
 }
 
 void ContentCredentialManagerDispatcher::OnNotifySignedOut(int request_id) {
   // TODO(mkwst): This is a stub.
   web_contents()->GetRenderViewHost()->Send(
       new CredentialManagerMsg_AcknowledgeSignedOut(
@@ skipping 20 matching lines @@
   form.origin = web_contents()->GetLastCommittedURL().GetOrigin();
   form.signon_realm = form.origin.spec();
 
   store->GetLogins(form, PasswordStore::DISALLOW_PROMPT, this);
 }
 
 void ContentCredentialManagerDispatcher::OnGetPasswordStoreResults(
     const std::vector<autofill::PasswordForm*>& results) {
   DCHECK(pending_request_id_);
 
-  if (results.empty()) {
-    // TODO(mkwst): This should be a separate message from above in
-    // OnRequestCredential. Waiting on a Blink-side change to make that
-    // possible.
-    web_contents()->GetRenderViewHost()->Send(
-        new CredentialManagerMsg_RejectCredentialRequest(
-            web_contents()->GetRenderViewHost()->GetRoutingID(),
-            pending_request_id_));
-    return;
-  }
-
-  // TODO(mkwst): This is a stub. We're just grabbing the first result and
-  // piping it down into Blink. Really, we should be kicking off some sort
-  // of UI full of magic moments and delight. Also, we should deal with
-  // federated login types.
-  CredentialInfo info(*results[0]);
+  // TODO(mkwst): This is a stub. We should be looking at |results| here. Baby
+  // steps.
+  password_manager::CredentialInfo info(base::ASCIIToUTF16("id"),
+                                        base::ASCIIToUTF16("name"),
+                                        GURL("https://example.com/image.png"));
   web_contents()->GetRenderViewHost()->Send(
       new CredentialManagerMsg_SendCredential(
           web_contents()->GetRenderViewHost()->GetRoutingID(),
           pending_request_id_,
           info));
   pending_request_id_ = 0;
 }
 
 PasswordStore* ContentCredentialManagerDispatcher::GetPasswordStore() {
   return client_ ? client_->GetPasswordStore() : nullptr;
 }
 
 }  // namespace password_manager