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

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

Issue 2842833003: Update SupportsUserData uses with unique_ptr. (Closed)
Patch Set: rebase Created 3 years, 8 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
« no previous file with comments | « chrome/browser/search/instant_io_context.cc ('k') | chrome/browser/ssl/ssl_error_handler.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: chrome/browser/ssl/ssl_error_handler.h
diff --git a/chrome/browser/ssl/ssl_error_handler.h b/chrome/browser/ssl/ssl_error_handler.h
index 319eb612184784a1e91c865a775d112b432bb441..c1ae36c9987e252ddbf944b4b293c8702386e863 100644
--- a/chrome/browser/ssl/ssl_error_handler.h
+++ b/chrome/browser/ssl/ssl_error_handler.h
@@ -62,6 +62,8 @@ class SSLErrorHandler : public content::WebContentsUserData<SSLErrorHandler>,
public:
typedef base::Callback<void(content::WebContents*)> TimerStartedCallback;
+ ~SSLErrorHandler() override;
+
// Events for UMA. Do not rename or remove values, add new values to the end.
// Public for testing.
enum UMAEvent {
@@ -143,8 +145,6 @@ class SSLErrorHandler : public content::WebContentsUserData<SSLErrorHandler>,
const base::Callback<void(content::CertificateRequestResultType)>&
callback);
- ~SSLErrorHandler() override;
-
// Called when an SSL cert error is encountered. Triggers a captive portal
// check and fires a one shot timer to wait for a "captive portal detected"
// result to arrive. Protected for testing.
« no previous file with comments | « chrome/browser/search/instant_io_context.cc ('k') | chrome/browser/ssl/ssl_error_handler.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698