| Index: chrome/browser/ui/passwords/password_manager_presenter.cc
|
| diff --git a/chrome/browser/ui/passwords/password_manager_presenter.cc b/chrome/browser/ui/passwords/password_manager_presenter.cc
|
| index aa723080fb0d7b93c4ab4fc4f97752d9a1b66329..10d51a8dc1928f0dfb504d04a552843fcf0e1f5c 100644
|
| --- a/chrome/browser/ui/passwords/password_manager_presenter.cc
|
| +++ b/chrome/browser/ui/passwords/password_manager_presenter.cc
|
| @@ -51,18 +51,8 @@ using password_manager::PasswordStore;
|
|
|
| namespace {
|
|
|
| -const int kAndroidAppSchemeAndDelimiterLength = 10; // Length of 'android://'.
|
| -
|
| const char kSortKeyPartsSeparator = ' ';
|
|
|
| -// Reverse order of subdomains in hostname.
|
| -std::string SplitByDotAndReverse(StringPiece host) {
|
| - std::vector<std::string> parts =
|
| - base::SplitString(host, ".", base::TRIM_WHITESPACE, base::SPLIT_WANT_ALL);
|
| - std::reverse(parts.begin(), parts.end());
|
| - return base::JoinString(parts, ".");
|
| -}
|
| -
|
| // Helper function that returns the type of the entry (non-Android credentials,
|
| // Android w/ affiliated web realm (i.e. clickable) or w/o web realm).
|
| std::string GetEntryTypeCode(bool is_android_uri, bool is_clickable) {
|
| @@ -88,9 +78,11 @@ std::string CreateSortKey(const autofill::PasswordForm& form,
|
| form, &is_android_uri, &link_url, &is_clickable);
|
|
|
| if (!is_clickable) { // e.g. android://com.example.r => r.example.com.
|
| - origin = SplitByDotAndReverse(
|
| - StringPiece(&origin[kAndroidAppSchemeAndDelimiterLength],
|
| - origin.length() - kAndroidAppSchemeAndDelimiterLength));
|
| + origin = password_manager::SplitByDotAndReverse(
|
| + StringPiece(
|
| + &origin[password_manager::kAndroidAppSchemeAndDelimiterLength],
|
| + origin.length() -
|
| + password_manager::kAndroidAppSchemeAndDelimiterLength));
|
| }
|
|
|
| std::string site_name =
|
| @@ -99,7 +91,7 @@ std::string CreateSortKey(const autofill::PasswordForm& form,
|
| if (site_name.empty()) // e.g. localhost.
|
| site_name = origin;
|
| std::string key =
|
| - site_name + SplitByDotAndReverse(StringPiece(
|
| + site_name + password_manager::SplitByDotAndReverse(StringPiece(
|
| &origin[0], origin.length() - site_name.length()));
|
|
|
| if (entry_type == PasswordEntryType::SAVED) {
|
|
|