Add button to page info to revoke user certificate decisions.

chrome/browser/ui/website_settings/website_settings.h

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #ifndef CHROME_BROWSER_UI_WEBSITE_SETTINGS_WEBSITE_SETTINGS_H_
 #define CHROME_BROWSER_UI_WEBSITE_SETTINGS_WEBSITE_SETTINGS_H_
 
 #include "base/memory/scoped_ptr.h"
 #include "base/strings/string16.h"
 #include "base/task/cancelable_task_tracker.h"
@@ skipping 99 matching lines @@
     return site_identity_details_;
   }
 
   base::string16 organization_name() const {
     return organization_name_;
   }
 
   // SiteDataObserver implementation.
   virtual void OnSiteDataAccessed() OVERRIDE;
 
+  // Returns the Omnibox URL of the website for which to display site
+  // permissions and site information.

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

Nit: Remove this comment.  The comment on the member declaration already gives
this information.

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

Done.

+  const GURL& GetSiteURL() const { return site_url_; }

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

Nit: Inlined accessors should be named unix_hacker()-style, with the same name
as the underlying member.

Place this up with the other accessors, in the same relative order as the
members are declared in.

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

Done.

+
  private:
   // Initializes the |WebsiteSettings|.
   void Init(Profile* profile,
             const GURL& url,
             const content::SSLStatus& ssl);
 
   // Sets (presents) the information about the site's permissions in the |ui_|.
   void PresentSitePermissions();
 
   // Sets (presents) the information about the site's data in the |ui_|.
@@ skipping 42 matching lines @@
 
   // TODO(markusheintz): Move the creation of all the base::string16 typed UI
   // strings below to the corresponding UI code, in order to prevent
   // unnecessary UTF-8 string conversions.
 
   // Details about the website's identity. If the website's identity has been
   // verified then |site_identity_details_| contains who verified the identity.
   // This string will be displayed in the UI.
   base::string16 site_identity_details_;
 
+  // Whether the user has made an explicit decision about how some certificate
+  // errors for this host should be handled.
+  bool certificate_decision_made_;
+
   // Details about the connection to the website. In case of an encrypted
   // connection |site_connection_details_| contains encryption details, like
   // encryption strength and ssl protocol version. This string will be
   // displayed in the UI.
   base::string16 site_connection_details_;
 
   // For websites that provided an EV certificate |orgainization_name_|
   // contains the organization name of the certificate. In all other cases
   // |organization_name| is an empty string. This string will be displayed in
   // the UI.
   base::string16 organization_name_;
 
   // The |CertStore| provides all X509Certificates.
   content::CertStore* cert_store_;
 
   // The |HostContentSettingsMap| is the service that provides and manages
   // content settings (aka. site permissions).
   HostContentSettingsMap* content_settings_;
 
   // Used to request the number of page visits.
   base::CancelableTaskTracker visit_count_task_tracker_;
 
   DISALLOW_COPY_AND_ASSIGN(WebsiteSettings);
 };
 
 #endif  // CHROME_BROWSER_UI_WEBSITE_SETTINGS_WEBSITE_SETTINGS_H_