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

Unified Diff: chrome/browser/about_flags.cc

Issue 2588133002: Make form-not-secure controllable as its own separate Finch feature (Closed)
Patch Set: rebase Created 4 years 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
« no previous file with comments | « chrome/app/generated_resources.grd ('k') | chrome/browser/ssl/security_state_tab_helper_browser_tests.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/about_flags.cc
diff --git a/chrome/browser/about_flags.cc b/chrome/browser/about_flags.cc
index 62bd08b0d916a3c260b741141b9aae427707ea59..2403b2da7752909a5f0f681293dce1592ca578cb 100644
--- a/chrome/browser/about_flags.cc
+++ b/chrome/browser/about_flags.cc
@@ -52,6 +52,7 @@
#include "components/omnibox/browser/omnibox_switches.h"
#include "components/password_manager/core/common/password_manager_features.h"
#include "components/proximity_auth/switches.h"
+#include "components/security_state/core/security_state.h"
#include "components/security_state/core/switches.h"
#include "components/signin/core/common/signin_switches.h"
#include "components/spellcheck/common/spellcheck_features.h"
@@ -210,11 +211,7 @@ const FeatureEntry::Choice kMarkHttpAsChoices[] = {
security_state::switches::kMarkHttpAsDangerous},
{IDS_MARK_HTTP_WITH_PASSWORDS_OR_CC_WITH_CHIP,
security_state::switches::kMarkHttpAs,
- security_state::switches::kMarkHttpWithPasswordsOrCcWithChip},
- {IDS_MARK_HTTP_WITH_PASSWORDS_OR_CC_WITH_CHIP_AND_FORM_WARNING,
- security_state::switches::kMarkHttpAs,
- security_state::switches::
- kMarkHttpWithPasswordsOrCcWithChipAndFormWarning}};
+ security_state::switches::kMarkHttpWithPasswordsOrCcWithChip}};
const FeatureEntry::Choice kDataReductionProxyLoFiChoices[] = {
{ IDS_GENERIC_EXPERIMENT_CHOICE_DEFAULT, "", "" },
@@ -1498,6 +1495,9 @@ const FeatureEntry kFeatureEntries[] = {
#endif // OS_CHROMEOS
{"mark-non-secure-as", IDS_MARK_HTTP_AS_NAME, IDS_MARK_HTTP_AS_DESCRIPTION,
kOsAll, MULTI_VALUE_TYPE(kMarkHttpAsChoices)},
+ {"enable-http-form-warning", IDS_ENABLE_HTTP_FORM_WARNING_NAME,
+ IDS_ENABLE_HTTP_FORM_WARNING_DESCRIPTION, kOsAll,
+ FEATURE_VALUE_TYPE(security_state::kHttpFormWarningFeature)},
{"enable-site-per-process", IDS_FLAGS_SITE_PER_PROCESS_NAME,
IDS_FLAGS_SITE_PER_PROCESS_DESCRIPTION, kOsAll,
SINGLE_VALUE_TYPE(switches::kSitePerProcess)},
@@ -2142,14 +2142,13 @@ const FeatureEntry kFeatureEntries[] = {
{"enable-expanded-autofill-credit-card-popup",
IDS_FLAGS_ENABLE_EXPANDED_AUTOFILL_CREDIT_CARD_POPUP_LAYOUT,
IDS_FLAGS_ENABLE_EXPANDED_AUTOFILL_CREDIT_CARD_POPUP_LAYOUT_DESCRIPTION,
- kOsAndroid,
- FEATURE_WITH_VARIATIONS_VALUE_TYPE(
- autofill::kAutofillCreditCardPopupLayout,
- kAutofillCreditCardPopupLayoutFeatureVariations,
- "AutofillCreditCardPopupLayout")},
- {"native-android-history-manager", IDS_NATIVE_ANDROID_HISTORY_MANAGER,
- IDS_NATIVE_ANDROID_HISTORY_MANAGER_DESCRIPTION, kOsAndroid,
- FEATURE_VALUE_TYPE(chrome::android::kNativeAndroidHistoryManager)},
+ kOsAndroid, FEATURE_WITH_VARIATIONS_VALUE_TYPE(
+ autofill::kAutofillCreditCardPopupLayout,
+ kAutofillCreditCardPopupLayoutFeatureVariations,
+ "AutofillCreditCardPopupLayout")},
+ {"native-android-history-manager", IDS_NATIVE_ANDROID_HISTORY_MANAGER,
+ IDS_NATIVE_ANDROID_HISTORY_MANAGER_DESCRIPTION, kOsAndroid,
+ FEATURE_VALUE_TYPE(chrome::android::kNativeAndroidHistoryManager)},
#endif // OS_ANDROID
{"enable-faster-location-reload", IDS_FLAGS_FASTER_LOCATION_RELOAD_NAME,
IDS_FLAGS_FASTER_LOCATION_RELOAD_DESCRIPTION, kOsAll,
« no previous file with comments | « chrome/app/generated_resources.grd ('k') | chrome/browser/ssl/security_state_tab_helper_browser_tests.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698