Credential saving clank infobar for Smart Lock.

chrome/browser/password_manager/chrome_password_manager_client.cc

Index: chrome/browser/password_manager/chrome_password_manager_client.cc
diff --git a/chrome/browser/password_manager/chrome_password_manager_client.cc b/chrome/browser/password_manager/chrome_password_manager_client.cc
index bc8c710db1f11c742537d876db1446e74199a28c..f9eed89f4c79ae9b96ecd3f4c9588bb1227d7745 100644
--- a/chrome/browser/password_manager/chrome_password_manager_client.cc
+++ b/chrome/browser/password_manager/chrome_password_manager_client.cc
@@ -189,8 +189,8 @@ bool ChromePasswordManagerClient::PromptUserToSavePassword(
         password_manager::metrics_util::GroupIdToString(
             password_manager::metrics_util::MonitoredDomainGroupId(
                 form_to_save->realm(), GetPrefs())));
-    SavePasswordInfoBarDelegate::Create(
-        web_contents(), form_to_save.Pass(), uma_histogram_suffix);
+    SavePasswordInfoBarDelegate::Create(web_contents(), form_to_save.Pass(),

[gone, 2015/03/02 17:39:38]

nit: if the arguments don't fit, c++ style puts them on their own lines IIRC

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

I was also surprised when I saw how clang format formatted this line, but
apparently it's allowed:
http://google-styleguide.googlecode.com/svn/trunk/cppguide.html#Function_Decl...
see second example.

I'll put all arguments on separate line.

[Bernhard Bauer, 2015/03/03 16:54:20]

Yeah, the "either all arguments on one line, or each on a separate line" rule
only applies to method headers.

+                                        uma_histogram_suffix, type);
   }
   return true;
 }