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

Unified Diff: components/autofill/core/browser/test_autofill_client.cc

Issue 2672623005: Record Autofill form events specially for nonsecure pages (Closed)
Patch Set: fix test added in rebase Created 3 years, 10 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/autofill/core/browser/test_autofill_client.cc
diff --git a/components/autofill/core/browser/test_autofill_client.cc b/components/autofill/core/browser/test_autofill_client.cc
index 9989e0a87fe5334595377e364c9f3470bf115f36..7df6c300ce4745be653aef47fc8bbaa6ae0a70ec 100644
--- a/components/autofill/core/browser/test_autofill_client.cc
+++ b/components/autofill/core/browser/test_autofill_client.cc
@@ -11,7 +11,8 @@ namespace autofill {
TestAutofillClient::TestAutofillClient()
: token_service_(new FakeOAuth2TokenService()),
identity_provider_(new FakeIdentityProvider(token_service_.get())),
- rappor_service_(new rappor::TestRapporServiceImpl()) {}
+ rappor_service_(new rappor::TestRapporServiceImpl()),
+ form_origin_(GURL("https://example.test")) {}
TestAutofillClient::~TestAutofillClient() {
}
@@ -115,9 +116,9 @@ void TestAutofillClient::DidFillOrPreviewField(
void TestAutofillClient::OnFirstUserGestureObserved() {
}
-bool TestAutofillClient::IsContextSecure(const GURL& form_origin) {
+bool TestAutofillClient::IsContextSecure() {
// Simplified secure context check for tests.
- return form_origin.SchemeIs("https");
+ return form_origin_.SchemeIs("https");
}
bool TestAutofillClient::ShouldShowSigninPromo() {

Powered by Google App Engine
This is Rietveld 408576698