This CL does the following: (1) Pass the max_length attribute to the password generator; (2) Update…

chrome/browser/autofill/password_generator.h

 // Copyright (c) 2012 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_AUTOFILL_PASSWORD_GENERATOR_H_
 #define CHROME_BROWSER_AUTOFILL_PASSWORD_GENERATOR_H_
 #pragma once
 
 #include <string>
 
 #include "base/basictypes.h"
 
+const unsigned int kDefaultPasswordLength = 12;
+
 namespace autofill {
 
 // Class to generate random passwords. Currently we just use a generic algorithm
 // for all sites, but eventually we can incorporate additional information to
 // determine passwords that are likely to be accepted (i.e. use pattern field,
 // previous generated passwords, crowdsourcing, etc.)
 class PasswordGenerator {
  public:
   PasswordGenerator();
+  // |max_length| is used as a hint for the generated password's length.
+  explicit PasswordGenerator(int max_length);
   ~PasswordGenerator();
 
-  // Returns a random password. The string is guaranteed to be printable and
-  // will not include whitespace characters.
-  std::string Generate();
+  // Returns a random password such that:
+  // (1) Each character is guaranteed to be a non-whitespace printable ASCII
+  //     character.
+  // (2) The generated password will contain AT LEAST one upper case letter, one
+  //     lower case letter, one digit, and one other symbol.
+  // (3) The password length will be equal to |password_length_| (see comment
+  //     for the constructor).
+  std::string Generate() const;
 
  private:
+  // The length of the generated password.
+  int password_length_;
+
+  // A helper function to set the |password_length_| according to |max_length|.
+  int SetLength(int max_length);

[Ilya Sherman, 2012/06/01 19:32:10]

nit: Please tuck this into the anonymous namespace in the implementation file,
and name it something more like "GetLengthBasedOnHint()" (a terser name might be
better, if you can think of a good one).

[zysxqn, 2012/06/01 22:32:54]

Done.

 
   DISALLOW_COPY_AND_ASSIGN(PasswordGenerator);
 };
 
 }  // namespace autofill
 
 #endif  // CHROME_BROWSER_AUTOFILL_PASSWORD_GENERATOR_H_