Chromium Code Reviews| Index: chromeos/attestation/attestation_flow.h |
| diff --git a/chromeos/attestation/attestation_flow.h b/chromeos/attestation/attestation_flow.h |
| index 8e7bb1801354b6ed117221634dda1437aefcd611..d660c2acc3dca23fb67ca281dd730d11cb1efb60 100644 |
| --- a/chromeos/attestation/attestation_flow.h |
| +++ b/chromeos/attestation/attestation_flow.h |
| @@ -9,6 +9,7 @@ |
| #include "base/basictypes.h" |
| #include "base/callback_forward.h" |
| +#include "base/memory/scoped_ptr.h" |
| #include "base/memory/weak_ptr.h" |
| #include "chromeos/chromeos_export.h" |
| #include "chromeos/dbus/dbus_method_call_status.h" |
| @@ -52,9 +53,10 @@ class CHROMEOS_EXPORT AttestationFlow { |
| const std::string& pem_certificate_chain)> |
| CertificateCallback; |
| + // This class takes ownership only of |server_proxy|. |
|
Mattias Nissler (ping if slow)
2013/04/02 12:49:39
nit: Comment is redundant given that you declare a
dkrahn
2013/04/04 21:27:27
Done.
|
| AttestationFlow(cryptohome::AsyncMethodCaller* async_caller, |
| CryptohomeClient* cryptohome_client, |
| - ServerProxy* server_proxy); |
| + scoped_ptr<ServerProxy> server_proxy); |
| virtual ~AttestationFlow(); |
| // Asynchronously gets an attestation certificate bound to the given name. |
| @@ -161,7 +163,7 @@ class CHROMEOS_EXPORT AttestationFlow { |
| base::WeakPtrFactory<AttestationFlow> weak_factory_; |
| cryptohome::AsyncMethodCaller* async_caller_; |
| CryptohomeClient* cryptohome_client_; |
| - ServerProxy* server_proxy_; |
| + scoped_ptr<ServerProxy> server_proxy_; |
| DISALLOW_COPY_AND_ASSIGN(AttestationFlow); |
| }; |