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

Side by Side Diff: chrome/browser/ssl/ssl_client_auth_observer.h

Issue 648653003: Standardize usage of virtual/override/final in chrome/browser/ (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #ifndef CHROME_BROWSER_SSL_SSL_CLIENT_AUTH_OBSERVER_H_ 5 #ifndef CHROME_BROWSER_SSL_SSL_CLIENT_AUTH_OBSERVER_H_
6 #define CHROME_BROWSER_SSL_SSL_CLIENT_AUTH_OBSERVER_H_ 6 #define CHROME_BROWSER_SSL_SSL_CLIENT_AUTH_OBSERVER_H_
7 7
8 #include "base/callback.h" 8 #include "base/callback.h"
9 #include "base/memory/ref_counted.h" 9 #include "base/memory/ref_counted.h"
10 #include "content/public/browser/notification_observer.h" 10 #include "content/public/browser/notification_observer.h"
11 #include "content/public/browser/notification_registrar.h" 11 #include "content/public/browser/notification_registrar.h"
12 12
13 namespace net { 13 namespace net {
14 class SSLCertRequestInfo; 14 class SSLCertRequestInfo;
15 class X509Certificate; 15 class X509Certificate;
16 } 16 }
17 17
18 namespace content { 18 namespace content {
19 class BrowserContext; 19 class BrowserContext;
20 } 20 }
21 21
22 class SSLClientAuthObserver : public content::NotificationObserver { 22 class SSLClientAuthObserver : public content::NotificationObserver {
23 public: 23 public:
24 SSLClientAuthObserver( 24 SSLClientAuthObserver(
25 const content::BrowserContext* browser_context, 25 const content::BrowserContext* browser_context,
26 const scoped_refptr<net::SSLCertRequestInfo>& cert_request_info, 26 const scoped_refptr<net::SSLCertRequestInfo>& cert_request_info,
27 const base::Callback<void(net::X509Certificate*)>& callback); 27 const base::Callback<void(net::X509Certificate*)>& callback);
28 virtual ~SSLClientAuthObserver(); 28 ~SSLClientAuthObserver() override;
29 29
30 // UI should implement this to close the dialog. 30 // UI should implement this to close the dialog.
31 virtual void OnCertSelectedByNotification() = 0; 31 virtual void OnCertSelectedByNotification() = 0;
32 32
33 // Send content the certificate. Can also call with NULL if the user 33 // Send content the certificate. Can also call with NULL if the user
34 // cancelled. Derived classes must use this instead of caching the callback 34 // cancelled. Derived classes must use this instead of caching the callback
35 // and calling it directly. 35 // and calling it directly.
36 void CertificateSelected(net::X509Certificate* cert); 36 void CertificateSelected(net::X509Certificate* cert);
37 37
38 // Begins observing notifications from other SSLClientAuthHandler instances. 38 // Begins observing notifications from other SSLClientAuthHandler instances.
39 // If another instance chooses a cert for a matching SSLCertRequestInfo, we 39 // If another instance chooses a cert for a matching SSLCertRequestInfo, we
40 // will also use the same cert and OnCertSelectedByNotification will be called 40 // will also use the same cert and OnCertSelectedByNotification will be called
41 // so that the cert selection UI can be closed. 41 // so that the cert selection UI can be closed.
42 void StartObserving(); 42 void StartObserving();
43 43
44 // Stops observing notifications. We will no longer act on client auth 44 // Stops observing notifications. We will no longer act on client auth
45 // notifications. 45 // notifications.
46 void StopObserving(); 46 void StopObserving();
47 47
48 net::SSLCertRequestInfo* cert_request_info() const { 48 net::SSLCertRequestInfo* cert_request_info() const {
49 return cert_request_info_.get(); 49 return cert_request_info_.get();
50 } 50 }
51 51
52 private: 52 private:
53 // content::NotificationObserver implementation: 53 // content::NotificationObserver implementation:
54 virtual void Observe(int type, 54 void Observe(int type,
55 const content::NotificationSource& source, 55 const content::NotificationSource& source,
56 const content::NotificationDetails& details) override; 56 const content::NotificationDetails& details) override;
57 57
58 const content::BrowserContext* browser_context_; 58 const content::BrowserContext* browser_context_;
59 scoped_refptr<net::SSLCertRequestInfo> cert_request_info_; 59 scoped_refptr<net::SSLCertRequestInfo> cert_request_info_;
60 base::Callback<void(net::X509Certificate*)> callback_; 60 base::Callback<void(net::X509Certificate*)> callback_;
61 content::NotificationRegistrar notification_registrar_; 61 content::NotificationRegistrar notification_registrar_;
62 62
63 DISALLOW_COPY_AND_ASSIGN(SSLClientAuthObserver); 63 DISALLOW_COPY_AND_ASSIGN(SSLClientAuthObserver);
64 }; 64 };
65 65
66 #endif // CHROME_BROWSER_SSL_SSL_CLIENT_AUTH_OBSERVER_H_ 66 #endif // CHROME_BROWSER_SSL_SSL_CLIENT_AUTH_OBSERVER_H_
OLDNEW
« no previous file with comments | « chrome/browser/ssl/ssl_browser_tests.cc ('k') | chrome/browser/ssl/ssl_client_certificate_selector_test.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698