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

Unified Diff: chrome/browser/ssl/ssl_error_handler.cc

Issue 2303413002: Simplify security_interstitials::ControllerClient and other related classes (Closed)
Patch Set: namespaces Created 4 years, 3 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/ssl_error_handler.cc
diff --git a/chrome/browser/ssl/ssl_error_handler.cc b/chrome/browser/ssl/ssl_error_handler.cc
index 1f2245178c76fdfbc86c9d95970aedfa6d0c717c..7014ebfb6ac52e7f809b91b97ddfe821f996604a 100644
--- a/chrome/browser/ssl/ssl_error_handler.cc
+++ b/chrome/browser/ssl/ssl_error_handler.cc
@@ -337,9 +337,9 @@ void SSLErrorHandler::ShowSSLInterstitial() {
RecordUMA(IsErrorOverridable() ? SHOW_SSL_INTERSTITIAL_OVERRIDABLE
: SHOW_SSL_INTERSTITIAL_NONOVERRIDABLE);
- (new SSLBlockingPage(web_contents_, cert_error_, ssl_info_, request_url_,
- options_mask_, base::Time::NowFromSystemTime(),
- std::move(ssl_cert_reporter_), callback_))
+ (SSLBlockingPage::Create(web_contents_, cert_error_, ssl_info_, request_url_,
+ options_mask_, base::Time::NowFromSystemTime(),
+ std::move(ssl_cert_reporter_), callback_))
->Show();
// Once an interstitial is displayed, no need to keep the handler around.
// This is the equivalent of "delete this".

Powered by Google App Engine
This is Rietveld 408576698