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

Unified Diff: chrome/browser/extensions/api/enterprise_platform_keys/enterprise_platform_keys_api.cc

Issue 564043004: Remove implicit conversions from scoped_refptr to T* in extensions code. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 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/extensions/api/enterprise_platform_keys/enterprise_platform_keys_api.cc
diff --git a/chrome/browser/extensions/api/enterprise_platform_keys/enterprise_platform_keys_api.cc b/chrome/browser/extensions/api/enterprise_platform_keys/enterprise_platform_keys_api.cc
index 613a99db303a769b6739c12397fe4150e9c3ed00..f120fc4e17a27f798b28ea533275590c19349d1d 100644
--- a/chrome/browser/extensions/api/enterprise_platform_keys/enterprise_platform_keys_api.cc
+++ b/chrome/browser/extensions/api/enterprise_platform_keys/enterprise_platform_keys_api.cc
@@ -212,7 +212,7 @@ EnterprisePlatformKeysImportCertificateFunction::Run() {
const std::string& cert_der = params->certificate;
scoped_refptr<net::X509Certificate> cert_x509 =
net::X509Certificate::CreateFromBytes(cert_der.data(), cert_der.size());
- if (!cert_x509)
+ if (!cert_x509.get())
return RespondNow(Error(kErrorInvalidX509Cert));
chromeos::platform_keys::ImportCertificate(
@@ -250,7 +250,7 @@ EnterprisePlatformKeysRemoveCertificateFunction::Run() {
const std::string& cert_der = params->certificate;
scoped_refptr<net::X509Certificate> cert_x509 =
net::X509Certificate::CreateFromBytes(cert_der.data(), cert_der.size());
- if (!cert_x509)
+ if (!cert_x509.get())
return RespondNow(Error(kErrorInvalidX509Cert));
chromeos::platform_keys::RemoveCertificate(

Powered by Google App Engine
This is Rietveld 408576698