Fix EV PageInfo check
Before we required Organization, Country, and Locality to be set, but EV
certificates require Organization, Country, and at least one of Locality
OR State. This updates the check in PageInfoBubbleView to only check the
fields that are actually used in the UI, and adds a regression test (and
certificate) exercising this case.
Fixed: 1069113
Change-Id: I5a5eff339eb875fa4d36c61f0e5ffc1d4ba95ce3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2142714
Reviewed-by: Mustafa Emre Acer <[email protected]>
Reviewed-by: Ryan Sleevi <[email protected]>
Commit-Queue: Christopher Thompson <[email protected]>
Cr-Commit-Position: refs/heads/master@{#758296}
diff --git a/net/BUILD.gn b/net/BUILD.gn
index dd1fe1f4..82f8544 100644
--- a/net/BUILD.gn
+++ b/net/BUILD.gn
@@ -1961,6 +1961,8 @@
"data/ssl/certificates/duplicate_cn_2.pem",
"data/ssl/certificates/eku-test-root.pem",
"data/ssl/certificates/empty_subject_cert.der",
+ "data/ssl/certificates/ev_test.pem",
+ "data/ssl/certificates/ev_test_state_only.pem",
"data/ssl/certificates/expired_cert.pem",
"data/ssl/certificates/explicit-policy-chain.pem",
"data/ssl/certificates/foaf.me.chromium-test-cert.der",
