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

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: Sync'd to revision p349162. Created 5 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
« no previous file with comments | « android_webview/browser/net/aw_url_request_context_getter.cc ('k') | chrome/browser/io_thread.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 7274371a7fe0cce6168f4adf3f30a4bbf99916e4..dc1bd8b5f9a6fac1b177680648490edf13bdb694 100644
--- a/chrome/browser/extensions/api/platform_keys/verify_trust_api.cc
+++ b/chrome/browser/extensions/api/platform_keys/verify_trust_api.cc
@@ -178,13 +178,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);
« no previous file with comments | « android_webview/browser/net/aw_url_request_context_getter.cc ('k') | chrome/browser/io_thread.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698