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

Unified Diff: chrome/browser/extensions/api/platform_keys/verify_trust_api.cc

Issue 1290243007: Shift URLRequestContextStorage over to taking scoped_ptrs. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@Paul_BuilderGrab
Patch Set: Lots of fixes driven by try jobs. Created 5 years, 4 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/platform_keys/verify_trust_api.cc
diff --git a/chrome/browser/extensions/api/platform_keys/verify_trust_api.cc b/chrome/browser/extensions/api/platform_keys/verify_trust_api.cc
index 3a2b5540b1a6851fcd0a0ed2b94ce725f9f61e76..98187da8b08f3dda9a51f0bdce72b644e2de4cb8 100644
--- a/chrome/browser/extensions/api/platform_keys/verify_trust_api.cc
+++ b/chrome/browser/extensions/api/platform_keys/verify_trust_api.cc
@@ -177,13 +177,11 @@ void VerifyTrustAPI::IOPart::Verify(scoped_ptr<Params> params,
return;
}
- net::CertVerifier* verifier = nullptr;
- if (ContainsKey(extension_to_verifier_, extension_id)) {
- verifier = extension_to_verifier_[extension_id].get();
- } else {
- verifier = net::CertVerifier::CreateDefault();
- extension_to_verifier_[extension_id] = make_linked_ptr(verifier);
+ if (!ContainsKey(extension_to_verifier_, extension_id)) {
+ extension_to_verifier_[extension_id] =
+ make_linked_ptr(net::CertVerifier::CreateDefault().release());
}
+ net::CertVerifier* verifier = extension_to_verifier_[extension_id].get();
scoped_ptr<net::CertVerifyResult> verify_result(new net::CertVerifyResult);
scoped_ptr<net::BoundNetLog> net_log(new net::BoundNetLog);

Powered by Google App Engine
This is Rietveld 408576698