Chromium Code Reviews| OLD | NEW |
|---|---|
| 1 // Copyright 2013 The Chromium Authors. All rights reserved. | 1 // Copyright 2013 The Chromium Authors. All rights reserved. |
| 2 // Use of this source code is governed by a BSD-style license that can be | 2 // Use of this source code is governed by a BSD-style license that can be |
| 3 // found in the LICENSE file. | 3 // found in the LICENSE file. |
| 4 // | 4 // |
| 5 // The local-auth module allows for user authentication in the case when | 5 // The local-auth module allows for user authentication in the case when |
| 6 // on-line authentication is not possible (e.g. there is no network | 6 // on-line authentication is not possible (e.g. there is no network |
| 7 // connection). | 7 // connection). |
| 8 | 8 |
| 9 #ifndef CHROME_BROWSER_SIGNIN_LOCAL_AUTH_H_ | 9 #ifndef CHROME_BROWSER_SIGNIN_LOCAL_AUTH_H_ |
| 10 #define CHROME_BROWSER_SIGNIN_LOCAL_AUTH_H_ | 10 #define CHROME_BROWSER_SIGNIN_LOCAL_AUTH_H_ |
| (...skipping 15 matching lines...) Expand all Loading... | |
| 26 | 26 |
| 27 void SetLocalAuthCredentials(const Profile* profile, | 27 void SetLocalAuthCredentials(const Profile* profile, |
| 28 const std::string& password); | 28 const std::string& password); |
| 29 | 29 |
| 30 bool ValidateLocalAuthCredentials(size_t profile_info_index, | 30 bool ValidateLocalAuthCredentials(size_t profile_info_index, |
| 31 const std::string& password); | 31 const std::string& password); |
| 32 | 32 |
| 33 bool ValidateLocalAuthCredentials(const Profile* profile, | 33 bool ValidateLocalAuthCredentials(const Profile* profile, |
| 34 const std::string& password); | 34 const std::string& password); |
| 35 | 35 |
| 36 // Return only the first |len_bits| bits of the string |str|. | |
| 37 std::string TruncateStringByBits(const std::string& str, const size_t len_bits); | |
|
bcwhite
2015/01/21 21:04:40
Why does this need to be public?
Mike Lerman
2015/01/27 21:02:26
So I can unit test it.
jww
2015/01/29 11:44:45
For unit tests, I've generally made methods privat
Mike Lerman
2015/01/29 16:57:42
I would - if this was a class. Sadly, it's a names
jww
2015/01/29 17:40:57
Lulz, totally missed that. Nevermind :-)
| |
| 38 | |
| 36 } // namespace chrome | 39 } // namespace chrome |
| 37 | 40 |
| 38 #endif // CHROME_BROWSER_SIGNIN_LOCAL_AUTH_H_ | 41 #endif // CHROME_BROWSER_SIGNIN_LOCAL_AUTH_H_ |
| OLD | NEW |