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

Unified Diff: net/cert/internal/test_helpers.h

Issue 1214933009: Class for parsing and evaluating RFC 5280 NameConstraints. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@compare_DN2
Patch Set: win compile fix 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
Index: net/cert/internal/test_helpers.h
diff --git a/net/cert/internal/test_helpers.h b/net/cert/internal/test_helpers.h
index d10148191a4f8a1bb6c9a05ebaa2edaea3c291c5..ad010caa4139da0d4f57932ed6cc5ad97af7295c 100644
--- a/net/cert/internal/test_helpers.h
+++ b/net/cert/internal/test_helpers.h
@@ -33,6 +33,13 @@ bool operator==(const Input& a, const Input& b);
// from accidentally passing an r-value.
der::Input InputFromString(const std::string* s);
+// Parses |s| as a DER Sequence TLV and returns a der::Input corresponding to
Ryan Sleevi 2015/09/24 22:06:20 s/Sequence/SEQUENCE/
mattm 2015/09/30 04:52:32 Done.
+// the value portion. On error returns an empty der::Input and adds a gtest
+// failure.
+//
+// Returned value's lifetime validity is the same as for InputFromString.
Ryan Sleevi 2015/09/24 22:06:20 Rather than back-reference to another function, it
mattm 2015/09/30 04:52:32 Done.
+der::Input SequenceValueFromString(const std::string* s);
Ryan Sleevi 2015/09/24 22:06:20 It's not entirely clear to me why this is needed;
mattm 2015/09/30 04:52:32 Depending on the context, Names can be implicitly
+
// Helper structure that maps a PEM block header (for instance "CERTIFICATE") to
// the destination where the value for that block should be written.
struct PemBlockMapping {

Powered by Google App Engine
This is Rietveld 408576698