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

Unified Diff: chrome/browser/extensions/api/passwords_private/passwords_private_apitest.cc

Issue 1907813002: Make the URLs in the password exceptions list clickable. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@remove-implicit-update.gitbr
Patch Set: Created 4 years, 8 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: chrome/browser/extensions/api/passwords_private/passwords_private_apitest.cc
diff --git a/chrome/browser/extensions/api/passwords_private/passwords_private_apitest.cc b/chrome/browser/extensions/api/passwords_private/passwords_private_apitest.cc
index d24bbe3052b32de31bbfb076a2a23286e97ccc3b..74f16c32bd973be19e7c42f2bec051077db18683 100644
--- a/chrome/browser/extensions/api/passwords_private/passwords_private_apitest.cc
+++ b/chrome/browser/extensions/api/passwords_private/passwords_private_apitest.cc
@@ -44,10 +44,13 @@ api::passwords_private::PasswordUiEntry CreateEntry(size_t num) {
return entry;
}
-std::string CreateException(size_t num) {
+api::passwords_private::ExceptionPair CreateException(size_t num) {
+ api::passwords_private::ExceptionPair exception;
std::stringstream ss;
ss << "http://exception" << num << ".com";
- return ss.str();
+ exception.exception_url = ss.str();
+ exception.link_url = ss.str();
stevenjb 2016/04/22 16:11:20 nit: we should probably test with a different link
hcarmona 2016/04/23 01:46:08 Done.
+ return exception;
}
// A test PasswordsPrivateDelegate implementation which uses mock data.
@@ -84,7 +87,8 @@ class TestDelegate : public PasswordsPrivateDelegate {
router->OnPasswordExceptionsListChanged(current_exceptions_);
}
- const std::vector<std::string>* GetPasswordExceptionsList() const override {
+ const std::vector<api::passwords_private::ExceptionPair>*
+ GetPasswordExceptionsList() const override {
return &current_exceptions_;
}
@@ -129,7 +133,7 @@ class TestDelegate : public PasswordsPrivateDelegate {
// observers are added, this delegate can send the current lists without
// having to request them from |password_manager_presenter_| again.
std::vector<api::passwords_private::PasswordUiEntry> current_entries_;
- std::vector<std::string> current_exceptions_;
+ std::vector<api::passwords_private::ExceptionPair> current_exceptions_;
Profile* profile_;
};

Powered by Google App Engine
This is Rietveld 408576698