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

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

Issue 2448943002: Refactor SecurityStateModel/Clients for simplicity and reusability. (Closed)
Patch Set: Remove *SecurityModelClient. 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.h
diff --git a/chrome/browser/ssl/chrome_security_state_model.h b/chrome/browser/ssl/chrome_security_state_model.h
new file mode 100644
index 0000000000000000000000000000000000000000..3520d3e4484c02b50347b1912d5cfb7167321e1d
--- /dev/null
+++ b/chrome/browser/ssl/chrome_security_state_model.h
@@ -0,0 +1,67 @@
+// Copyright 2015 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_SSL_CHROME_SECURITY_STATE_MODEL_H_
+#define CHROME_BROWSER_SSL_CHROME_SECURITY_STATE_MODEL_H_
+
+#include <memory>
+
+#include "base/gtest_prod_util.h"
+#include "base/macros.h"
+#include "components/security_state/content/web_contents_security_state_model.h"
+#include "components/security_state/core/security_state_model.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"
+
+namespace content {
+struct SecurityStyleExplanations;
+class NavigationEntry;
+class NavigationHandle;
+class WebContents;
+} // namespace content
+
+// Provides a WebContentsSecurityStateModel with chrome-specific information
+// that it needs to determine the page's security status.
+class ChromeSecurityStateModel
+ : public content::WebContentsObserver,
+ public content::WebContentsUserData<ChromeSecurityStateModel> {
+ public:
+ ~ChromeSecurityStateModel() override;
+
+ // See WebContentsSecurityStateModel::GetSecurityInfo.
+ void GetSecurityInfo(
+ security_state::SecurityStateModel::SecurityInfo* result) const;
+
+ // Called when the NavigationEntry's SSLStatus changes.
+ void VisibleSSLStateChanged();
+
+ // content::WebContentsObserver:
+ void DidFinishNavigation(
+ content::NavigationHandle* navigation_handle) override;
+
+ private:
+ explicit ChromeSecurityStateModel(content::WebContents* web_contents);
+ friend class content::WebContentsUserData<ChromeSecurityStateModel>;
+
+ // For web_contents_model_.
+ FRIEND_TEST_ALL_PREFIXES(ChromeSecurityStateModelTest,
+ VisibleSecurityStateNonsecureFormInputs);
+
+ bool UsedPolicyInstalledCertificate() const;
+ bool GetMalwareStatus() const;
+
+ std::unique_ptr<security_state::WebContentsSecurityStateModel>
+ web_contents_model_;
+
+ // True if a console has been logged about an omnibox warning that
+ // will be shown in future versions of Chrome for insecure HTTP
+ // pages. This message should only be logged once per main-frame
+ // navigation.
+ bool logged_http_warning_on_current_navigation_;
+
+ DISALLOW_COPY_AND_ASSIGN(ChromeSecurityStateModel);
+};
+
+#endif // CHROME_BROWSER_SSL_CHROME_SECURITY_STATE_MODEL_H_

Powered by Google App Engine
This is Rietveld 408576698