Credential saving clank infobar for Smart Lock.

chrome/browser/password_manager/save_password_infobar_delegate.h

 // 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.
 
 #ifndef CHROME_BROWSER_PASSWORD_MANAGER_SAVE_PASSWORD_INFOBAR_DELEGATE_H_
 #define CHROME_BROWSER_PASSWORD_MANAGER_SAVE_PASSWORD_INFOBAR_DELEGATE_H_
 
 #include "base/basictypes.h"
-#include "base/compiler_specific.h"
 #include "base/memory/scoped_ptr.h"
 #include "base/timer/elapsed_timer.h"
 #include "components/infobars/core/confirm_infobar_delegate.h"
-#include "components/infobars/core/infobar_delegate.h"
 #include "components/password_manager/core/browser/password_form_manager.h"
 #include "components/password_manager/core/browser/password_manager_metrics_util.h"
 
 namespace content {
 class WebContents;
 }
 
+namespace password_manager {
+enum class CredentialSourceType;
+}
+
 // After a successful *new* login attempt, we take the PasswordFormManager in
 // provisional_save_manager_ and move it to a SavePasswordInfoBarDelegate while
 // the user makes up their mind with the "save password" infobar. Note if the
 // login is one we already know about, the end of the line is
 // provisional_save_manager_ because we just update it on success and so such
 // forms never end up in an infobar.
 class SavePasswordInfoBarDelegate : public ConfirmInfoBarDelegate {
  public:
   // If we won't be showing the one-click signin infobar, creates a save
   // password infobar and delegate and adds the infobar to the InfoBarService
   // for |web_contents|.  |uma_histogram_suffix| is empty, or one of the
   // "group_X" suffixes used in the histogram names for infobar usage reporting;
   // if empty, the usage is not reported, otherwise the suffix is used to choose
   // the right histogram.
   static void Create(
       content::WebContents* web_contents,
       scoped_ptr<password_manager::PasswordFormManager> form_to_save,
-      const std::string& uma_histogram_suffix);
-
- private:
-  SavePasswordInfoBarDelegate(
-      scoped_ptr<password_manager::PasswordFormManager> form_to_save,
-      const std::string& uma_histogram_suffix);
+      const std::string& uma_histogram_suffix,
+      password_manager::CredentialSourceType source_type);
 
   ~SavePasswordInfoBarDelegate() override;
 
+  SavePasswordInfoBarDelegate(

[Bernhard Bauer, 2015/03/02 10:28:09]

Does this constructor need to be public now?

[melandory, 2015/03/03 11:55:46]

Nope.
Done.

+      scoped_ptr<password_manager::PasswordFormManager> form_to_save,
+      const std::string& uma_histogram_suffix,
+      password_manager::CredentialSourceType source_type);
+
+  // If infobar was tiggered by Credential management API, then on Android

[Bernhard Bauer, 2015/03/02 10:28:09]

Nit: "triggered" (Tigger is something else [http://en.wikipedia.org/wiki/Tigger]
:) ).

Also: Articles ("If *the* infobar was triggered by *the* Credential management
API, then [...] it should display *the* 'More' button").

[melandory, 2015/03/03 11:55:46]

Done.

+  // it should display "More" button.
+  bool ShouldShowMoreButton();
+
   // ConfirmInfoBarDelegate:
   Type GetInfoBarType() const override;
   InfoBarAutomationType GetInfoBarAutomationType() const override;
   int GetIconID() const override;
   bool ShouldExpire(const NavigationDetails& details) const override;
   void InfoBarDismissed() override;
   base::string16 GetMessageText() const override;
   base::string16 GetButtonLabel(InfoBarButton button) const override;
   bool Accept() override;
   bool Cancel() override;
 
+ private:
   // The PasswordFormManager managing the form we're asking the user about,
   // and should update as per her decision.
   scoped_ptr<password_manager::PasswordFormManager> form_to_save_;
 
   // Used to track the results we get from the info bar.
   password_manager::metrics_util::ResponseType infobar_response_;
 
   // Measures the "Save password?" prompt lifetime. Used to report an UMA
   // signal.
   base::ElapsedTimer timer_;
 
   // The group name corresponding to the domain name of |form_to_save_| if the
   // form is on a monitored domain. Otherwise, an empty string.
   const std::string uma_histogram_suffix_;
 
+  // Determines source from where infobar was triggered.

[Mike West, 2015/03/02 11:54:27]

Nit: This doesn't "determine", it "records" or something similar.

[melandory, 2015/03/03 11:55:46]

Done.

+  // Infobar appearance (title, buttons) depends on value of this parameter.
+  password_manager::CredentialSourceType source_type_;
+
   DISALLOW_COPY_AND_ASSIGN(SavePasswordInfoBarDelegate);
 };
 
 #endif  // CHROME_BROWSER_PASSWORD_MANAGER_SAVE_PASSWORD_INFOBAR_DELEGATE_H_