Don't autofill credit cards on non-secure pages

components/autofill/core/browser/test_autofill_client.h

Index: components/autofill/core/browser/test_autofill_client.h
diff --git a/components/autofill/core/browser/test_autofill_client.h b/components/autofill/core/browser/test_autofill_client.h
index 251181bccd2e4a8f7e738c30bc87d32eb7196130..e0940e174121626a109c295f02948971fde30077 100644
--- a/components/autofill/core/browser/test_autofill_client.h
+++ b/components/autofill/core/browser/test_autofill_client.h
@@ -56,7 +56,9 @@ class TestAutofillClient : public AutofillClient {
                              const base::string16& profile_full_name) override;
   void OnFirstUserGestureObserved() override;
   void LinkClicked(const GURL& url, WindowOpenDisposition disposition) override;
+  bool IsContextSecure(const GURL& form_origin) override;

[Evan Stade, 2015/06/29 17:38:54]

nit: is_context_secure() const (and inline the impl)

(simple setters and getters are exceptions to the no-inline-in-header-file rule)

[sigbjorn, 2015/06/30 07:12:13]

This is an override, so that means I'd have to change the name of this function
elsewhere too. Elsewhere it is not a getter. Did you mean to change the name
throughout?

[Evan Stade, 2015/06/30 15:47:13]

ah, yea, nevermind about that one

 
+  void SetContextSecurity(bool is_context_secure);

[Evan Stade, 2015/06/29 17:38:54]

nit: set_is_context_secure() (and inline the impl)

   void SetPrefs(scoped_ptr<PrefService> prefs) { prefs_ = prefs.Pass(); }
 
   rappor::TestRapporService* test_rappor_service() {
@@ -70,6 +72,8 @@ class TestAutofillClient : public AutofillClient {
   scoped_ptr<FakeIdentityProvider> identity_provider_;
   scoped_ptr<rappor::TestRapporService> rappor_service_;
 
+  bool is_context_secure_;
+
   DISALLOW_COPY_AND_ASSIGN(TestAutofillClient);
 };