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

Unified Diff: components/password_manager/content/renderer/credential_manager_client_browsertest.cc

Issue 1159323003: Credential Management: Rename 'notifySignedOut' to 'requireUserMediation' (2/3) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@rename-password
Patch Set: Rebase. Created 5 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/renderer/credential_manager_client_browsertest.cc
diff --git a/components/password_manager/content/renderer/credential_manager_client_browsertest.cc b/components/password_manager/content/renderer/credential_manager_client_browsertest.cc
index 926755f02774b7c7c75d32b4ffa7c366b6bca1e3..c2f414c69ce716e082ebd7d65badc8b810fcd3fc 100644
--- a/components/password_manager/content/renderer/credential_manager_client_browsertest.cc
+++ b/components/password_manager/content/renderer/credential_manager_client_browsertest.cc
@@ -75,9 +75,9 @@ class MAYBE_CredentialManagerClientTest : public content::RenderViewTest {
break;
}
- case CredentialManagerHostMsg_NotifySignedOut::ID: {
+ case CredentialManagerHostMsg_RequireUserMediation::ID: {
base::Tuple<int> param;
- CredentialManagerHostMsg_NotifySignedOut::Read(message, &param);
+ CredentialManagerHostMsg_RequireUserMediation::Read(message, &param);
request_id = base::get<0>(param);
break;
}
@@ -189,19 +189,19 @@ TEST_F(MAYBE_CredentialManagerClientTest, SendNotifySignedIn) {
EXPECT_FALSE(callback_errored());
}
-TEST_F(MAYBE_CredentialManagerClientTest, SendNotifySignedOut) {
+TEST_F(MAYBE_CredentialManagerClientTest, SendRequestUserMediation) {
int request_id;
- EXPECT_FALSE(ExtractRequestId(CredentialManagerHostMsg_NotifySignedOut::ID,
- request_id));
+ EXPECT_FALSE(ExtractRequestId(
+ CredentialManagerHostMsg_RequireUserMediation::ID, request_id));
scoped_ptr<TestNotificationCallbacks> callbacks(
new TestNotificationCallbacks(this));
- client_->dispatchSignedOut(callbacks.release());
+ client_->dispatchRequireUserMediation(callbacks.release());
- EXPECT_TRUE(ExtractRequestId(CredentialManagerHostMsg_NotifySignedOut::ID,
- request_id));
+ EXPECT_TRUE(ExtractRequestId(
+ CredentialManagerHostMsg_RequireUserMediation::ID, request_id));
- client_->OnAcknowledgeSignedOut(request_id);
+ client_->OnAcknowledgeRequireUserMediation(request_id);
EXPECT_TRUE(callback_succeeded());
EXPECT_FALSE(callback_errored());
}

Powered by Google App Engine
This is Rietveld 408576698