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

Unified Diff: components/security_interstitials/core/ssl_error_ui.cc

Issue 1772143002: Use network time for bad clock interstitial. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: estark review 1 Created 4 years, 9 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: components/security_interstitials/core/ssl_error_ui.cc
diff --git a/components/security_interstitials/core/ssl_error_ui.cc b/components/security_interstitials/core/ssl_error_ui.cc
index a02f0b016e5aebf0c65ef6e5064b15fe245d8d99..2d28d5ac061910b8c526746843f81aed4f962b98 100644
--- a/components/security_interstitials/core/ssl_error_ui.cc
+++ b/components/security_interstitials/core/ssl_error_ui.cc
@@ -24,13 +24,11 @@ bool IsMasked(int options, SSLErrorUI::SSLErrorOptionsMask mask) {
} // namespace
-SSLErrorUI::SSLErrorUI(const GURL& request_url,
- int cert_error,
- const net::SSLInfo& ssl_info,
- int display_options,
- const base::Time& time_triggered,
- const std::string& languages,
- ControllerClient* controller)
+SSLErrorUI::SSLErrorUI(
+ const GURL& request_url, int cert_error, const net::SSLInfo& ssl_info,
+ int display_options, const base::Time& time_triggered,
+ const network_time::NetworkTimeTracker* network_time_tracker,
+ const std::string& languages, ControllerClient* controller)
: request_url_(request_url),
cert_error_(cert_error),
ssl_info_(ssl_info),
@@ -46,9 +44,9 @@ SSLErrorUI::SSLErrorUI(const GURL& request_url,
controller_->metrics_helper()->RecordUserDecision(MetricsHelper::SHOW);
controller_->metrics_helper()->RecordUserInteraction(
MetricsHelper::TOTAL_VISITS);
- ssl_errors::RecordUMAStatistics(soft_override_enabled_, time_triggered_,
- request_url, cert_error_,
- *ssl_info_.cert.get());
+ ssl_errors::RecordUMAStatistics(
+ soft_override_enabled_, time_triggered_, network_time_tracker,
+ request_url, cert_error_, *ssl_info_.cert.get());
}
SSLErrorUI::~SSLErrorUI() {

Powered by Google App Engine
This is Rietveld 408576698