Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(400)

Unified Diff: components/password_manager/content/browser/credential_manager_impl.cc

Issue 2083433002: Mojo: Remove url type converters. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: rebase Created 4 years, 6 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: components/password_manager/content/browser/credential_manager_impl.cc
diff --git a/components/password_manager/content/browser/credential_manager_impl.cc b/components/password_manager/content/browser/credential_manager_impl.cc
index 46df09d1b2d4335d2b20d0927566a21aad0db790..ba0cf7dda227d84da9820bc3aaca0a94be96ac08 100644
--- a/components/password_manager/content/browser/credential_manager_impl.cc
+++ b/components/password_manager/content/browser/credential_manager_impl.cc
@@ -19,7 +19,6 @@
#include "components/password_manager/core/common/credential_manager_types.h"
#include "components/password_manager/core/common/password_manager_pref_names.h"
#include "content/public/browser/web_contents.h"
-#include "mojo/common/url_type_converters.h"
namespace password_manager {
@@ -150,7 +149,7 @@ void CredentialManagerImpl::ScheduleRequireMediationTask(
void CredentialManagerImpl::Get(bool zero_click_only,
bool include_passwords,
- mojo::Array<mojo::String> federations,
+ mojo::Array<GURL> federations,
const GetCallback& callback) {
PasswordStore* store = GetPasswordStore();
if (pending_request_ || !store) {
@@ -177,12 +176,11 @@ void CredentialManagerImpl::Get(bool zero_click_only,
GetSynthesizedFormForOrigin(),
base::Bind(&CredentialManagerImpl::ScheduleRequestTask,
weak_factory_.GetWeakPtr(), callback, zero_click_only,
- include_passwords, federations.To<std::vector<GURL>>()));
+ include_passwords, federations.PassStorage()));
} else {
std::vector<std::string> no_affiliated_realms;
ScheduleRequestTask(callback, zero_click_only, include_passwords,
- federations.To<std::vector<GURL>>(),
- no_affiliated_realms);
+ federations.PassStorage(), no_affiliated_realms);
}
}

Powered by Google App Engine
This is Rietveld 408576698