Password bubble: Add a test for displayed password length.

chrome/browser/ui/passwords/manage_passwords_bubble_model.cc

 // Copyright 2013 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 "chrome/browser/ui/passwords/manage_passwords_bubble_model.h"
 
 #include "chrome/browser/password_manager/password_store_factory.h"
 #include "chrome/browser/ui/browser.h"
 #include "chrome/browser/ui/browser_finder.h"
 #include "chrome/browser/ui/chrome_pages.h"
@@ skipping 81 matching lines @@
     autofill::PasswordForm password_form) {
   ManagePasswordsBubbleUIController::FromWebContents(web_contents_)->
       RemoveFromBestMatches(password_form);
 }
 
 void ManagePasswordsBubbleModel::WebContentsDestroyed(
     content::WebContents* web_contents) {
   // 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);
+}