Http Bad: Add "Learn more" sublabel to warning message and make it clickable

components/autofill/core/browser/autofill_external_delegate.cc

Index: components/autofill/core/browser/autofill_external_delegate.cc
diff --git a/components/autofill/core/browser/autofill_external_delegate.cc b/components/autofill/core/browser/autofill_external_delegate.cc
index 44264662a03bf37a9fe63bfc4918df5166910b90..b4d9d85432fb16b0c29c0379d68ec8c17c4f1696 100644
--- a/components/autofill/core/browser/autofill_external_delegate.cc
+++ b/components/autofill/core/browser/autofill_external_delegate.cc
@@ -30,6 +30,9 @@ namespace autofill {
 
 namespace {
 
+const char kSecurityIndicatorHelpCenterURL[] =
+    "https://support.google.com/chrome/answer/95617?hl=en";
+
 // Returns true if the suggestion entry is an Autofill warning message.
 // Warning messages should display on top of suggestion list.
 bool IsAutofillWarningEntry(int frontend_id) {
@@ -225,6 +228,8 @@ void AutofillExternalDelegate::DidAcceptSuggestion(const base::string16& value,
         &AutofillExternalDelegate::OnCreditCardScanned, GetWeakPtr()));
   } else if (identifier == POPUP_ITEM_ID_CREDIT_CARD_SIGNIN_PROMO) {
     manager_->client()->StartSigninFlow();
+  } else if (identifier == POPUP_ITEM_ID_HTTP_NOT_SECURE_WARNING_MESSAGE) {
+    manager_->OpenUrl(GURL(kSecurityIndicatorHelpCenterURL));

[Mathieu, 2016/11/16 20:03:46]

use manager_->client()->OpenUrl directly?

[lshang, 2016/11/17 07:17:23]

Done.

   } else {
     if (identifier > 0)  // Denotes an Autofill suggestion.
       AutofillMetrics::LogAutofillSuggestionAcceptedIndex(position);