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

Unified Diff: chrome/browser/ssl/chrome_security_state_model_client.h

Issue 2410043003: Add a console messsage for HTTP-bad (Closed)
Patch Set: Created 4 years, 2 months 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
Index: chrome/browser/ssl/chrome_security_state_model_client.h
diff --git a/chrome/browser/ssl/chrome_security_state_model_client.h b/chrome/browser/ssl/chrome_security_state_model_client.h
index c5f8854fa5cb259bdbd8340acb1b8bdcd77a3bd5..9c71b9c6b0b0317682e92f8011550ce91303e205 100644
--- a/chrome/browser/ssl/chrome_security_state_model_client.h
+++ b/chrome/browser/ssl/chrome_security_state_model_client.h
@@ -10,6 +10,7 @@
#include "base/macros.h"
#include "components/security_state/security_state_model.h"
#include "components/security_state/security_state_model_client.h"
+#include "content/public/browser/web_contents_observer.h"
#include "content/public/browser/web_contents_user_data.h"
#include "third_party/WebKit/public/platform/WebSecurityStyle.h"
@@ -22,7 +23,8 @@ class WebContents;
// information that it needs to determine the page's security status.
class ChromeSecurityStateModelClient
: public security_state::SecurityStateModelClient,
- public content::WebContentsUserData<ChromeSecurityStateModelClient> {
+ public content::WebContentsUserData<ChromeSecurityStateModelClient>,
+ public content::WebContentsObserver {
public:
~ChromeSecurityStateModelClient() override;
@@ -43,12 +45,17 @@ class ChromeSecurityStateModelClient
bool UsedPolicyInstalledCertificate() override;
bool IsOriginSecure(const GURL& url) override;
+ // content::WebContentsObserver:
+ void DidFinishNavigation(
+ content::NavigationHandle* navigation_handle) override;
+
private:
explicit ChromeSecurityStateModelClient(content::WebContents* web_contents);
friend class content::WebContentsUserData<ChromeSecurityStateModelClient>;
content::WebContents* web_contents_;
std::unique_ptr<security_state::SecurityStateModel> security_state_model_;
+ mutable bool logged_http_warning_on_current_navigation_;
DISALLOW_COPY_AND_ASSIGN(ChromeSecurityStateModelClient);
};

Powered by Google App Engine
This is Rietveld 408576698