Add button to page info to revoke user certificate decisions.

chrome/browser/ui/views/website_settings/website_settings_popup_view.h

Index: chrome/browser/ui/views/website_settings/website_settings_popup_view.h
diff --git a/chrome/browser/ui/views/website_settings/website_settings_popup_view.h b/chrome/browser/ui/views/website_settings/website_settings_popup_view.h
index 31ddff519cea7c599bc498f84f595d4d235c5b5e..d16219e7a7654865065321ad728f0a681bc2a954 100644
--- a/chrome/browser/ui/views/website_settings/website_settings_popup_view.h
+++ b/chrome/browser/ui/views/website_settings/website_settings_popup_view.h
@@ -118,7 +118,8 @@ class WebsiteSettingsPopupView
                               const base::string16& headline,
                               const base::string16& text,
                               views::Link* link,
-                              views::Link* secondary_link);
+                              views::Link* secondary_link,
+                              views::Link* reset_decisions_button);
   // Handles LinkClicked asynchronously.
   void HandleLinkClickedAsync(views::Link* source);
 
@@ -159,6 +160,9 @@ class WebsiteSettingsPopupView
   // Certificate Transparency info. If no such SCTs accompany the certificate
   // then |signed_certificate_timestamps_link_| is NULL.
   views::Link* signed_certificate_timestamps_link_;
+  // The button to reset the Allow/Deny certificate errors decision for the
+  // current host.
+  views::Link* reset_decisions_button_;

[Peter Kasting, 2014/07/26 02:40:39]

Your comment and variable name call this a button, but the type is Link.  These
need to be consistent.  If clicking this takes some action, as opposed to going
to another page or something, this should actually be a button, not a link.

[jww, 2014/07/27 17:29:26]

After going over the designs with ainsle@, this a link-like text even though it
is resetting the security decisions (and only closing the page info). This is
also consistent with the new SSL error interstitial, where, for example, the
proceed decision itself is a link, not a button. Would it still be better to
change the comment and variable this to 'link'?

[Peter Kasting, 2014/07/28 11:07:44]

An interstitial is web content, and a clickable target that causes that
interstitial to be navigated away makes sense to be a link.  The page info
dropdown is native UI, and buttons that take actions to affect browser state
should be buttons, not links.  We used to be inconsistent about this, e.g. by
having a "Remove" link (instead of button) on the star bubble, but we've fixed
all the cases I'm aware of, so I'm going to stand firm on this one.  If
necessary, put me in touch with Alex directly.

 
   // The id of the certificate provided by the site. If the site does not
   // provide a certificate then |cert_id_| is 0.