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

Unified Diff: components/cast_certificate/cast_crl.h

Issue 2303673004: Hook up Chrome Cast sender to Cast CRL. (Closed)
Patch Set: Cleaned up unused headers. Created 4 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: components/cast_certificate/cast_crl.h
diff --git a/components/cast_certificate/cast_crl.h b/components/cast_certificate/cast_crl.h
index 0ab6964fa03c25c26bdadea73e601d8b0aa8054b..934ace480efacd91be408f336a23a91c31350789 100644
--- a/components/cast_certificate/cast_crl.h
+++ b/components/cast_certificate/cast_crl.h
@@ -53,13 +53,15 @@ class CastCRL {
std::unique_ptr<CastCRL> ParseAndVerifyCRL(const std::string& crl_proto,
const base::Time& time);
-// Exposed only for testing, not for use in production code.
-//
// This is an overloaded version of ParseAndVerifyCRL that allows
-// the input of a custom TrustStore.
-std::unique_ptr<CastCRL> ParseAndVerifyCRLForTest(const std::string& crl_proto,
- const base::Time& time,
- net::TrustStore* trust_store);
+// the input of a custom TrustStore. |trust_store| must not be nullptr.
+//
+// For production use pass |trust_store| as nullptr to use the production trust
eroman 2016/09/17 00:44:41 This doesn't agree with comment above.
ryanchung 2016/09/22 21:38:51 Done. Allowing nullptr to be passed.
+// store.
+std::unique_ptr<CastCRL> ParseAndVerifyCRLUsingCustomTrustStore(
+ const std::string& crl_proto,
+ const base::Time& time,
+ net::TrustStore* trust_store);
} // namespace cast_certificate

Powered by Google App Engine
This is Rietveld 408576698