Make form-not-secure controllable as its own separate Finch feature

components/autofill/core/browser/autofill_experiments.cc

Index: components/autofill/core/browser/autofill_experiments.cc
diff --git a/components/autofill/core/browser/autofill_experiments.cc b/components/autofill/core/browser/autofill_experiments.cc
index 28fdc27bff258eef2287595c7c983c2ddb3651ce..4f3ddd2d7e3ab6f2b957231a8151202287f7947c 100644
--- a/components/autofill/core/browser/autofill_experiments.cc
+++ b/components/autofill/core/browser/autofill_experiments.cc
@@ -16,7 +16,7 @@
 #include "components/autofill/core/common/autofill_pref_names.h"
 #include "components/autofill/core/common/autofill_switches.h"
 #include "components/prefs/pref_service.h"
-#include "components/security_state/core/switches.h"
+#include "components/security_state/core/security_state.h"
 #include "components/sync/driver/sync_service.h"
 #include "components/variations/variations_associated_data.h"
 #include "google_apis/gaia/gaia_auth_util.h"
@@ -229,11 +229,7 @@ bool IsCreditCardUploadEnabled(const PrefService* pref_service,
 }
 
 bool IsCreditCardAutofillHttpWarningEnabled() {

[Mathieu, 2016/12/19 20:15:49]

I think it's fine to remove this function and call security_state::IsHttp... in
the autofill code directly.

[estark, 2016/12/19 20:49:45]

Done.

-  std::string choice =
-      base::CommandLine::ForCurrentProcess()->GetSwitchValueASCII(
-          security_state::switches::kMarkHttpAs);
-  return choice == security_state::switches::
-                       kMarkHttpWithPasswordsOrCcWithChipAndFormWarning;
+  return security_state::IsHttpWarningInFormEnabled();
 }
 
 }  // namespace autofill