Password bubble: Add a test for displayed password length.

chrome/browser/ui/passwords/manage_passwords_bubble_model.cc

Index: chrome/browser/ui/passwords/manage_passwords_bubble_model.cc
diff --git a/chrome/browser/ui/passwords/manage_passwords_bubble_model.cc b/chrome/browser/ui/passwords/manage_passwords_bubble_model.cc
index 5751f60fda08440d22d0ab8b8911c173cebaf6cf..96ebee5879533e21f04f89e4df03e79016153659 100644
--- a/chrome/browser/ui/passwords/manage_passwords_bubble_model.cc
+++ b/chrome/browser/ui/passwords/manage_passwords_bubble_model.cc
@@ -99,3 +99,12 @@ void ManagePasswordsBubbleModel::WebContentsDestroyed(
   // The WebContents have been destroyed.
   web_contents_ = NULL;
 }
+
+// static
+base::string16 ManagePasswordsBubbleModel::GetPasswordDisplayString(

[Patrick Dubroy, 2014/04/02 09:41:50]

I definitely think adding some tests for the ManagePasswordsBubbleModel is a
good idea, but I'm not sure this method actually belongs on the model. On OS X,
for example, we'd probably use NSSecureTextField rather than manually replacing
the characters with bullets.

+    const base::string16& password) {
+  const wchar_t kPasswordBullet = 0x2022;
+  const size_t kMaxPasswordChar = 22;
+  return base::string16(std::min(password.length(), kMaxPasswordChar),
+                        kPasswordBullet);
+}