Add CertIssuerSourceAia: authorityInfoAccess fetching for CertPathBuilder.

net/cert/internal/cert_issuer_source_aia.h

+// Copyright 2016 The Chromium Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style license that can be
+// found in the LICENSE file.
+
+#ifndef NET_CERT_INTERNAL_CERT_ISSUER_SOURCE_AIA_H_
+#define NET_CERT_INTERNAL_CERT_ISSUER_SOURCE_AIA_H_
+
+#include "base/strings/string_piece.h"
+#include "net/cert/cert_net_fetcher.h"
+#include "net/cert/internal/cert_issuer_source.h"
+
+namespace net {
+
+class NET_EXPORT CertIssuerSourceAia : public CertIssuerSource {
+ public:
+  // Creates CertIssuerSource that will use |cert_fetcher| to retrieve issuers
+  // using AuthorityInfoAccess URIs. |cert_fetcher| must outlive the
+  // CertIssuerSourceAia.  CertIssuerSourceAia must be created and used only on
+  // the network thread.
+  explicit CertIssuerSourceAia(CertNetFetcher* cert_fetcher);
+  ~CertIssuerSourceAia() override;
+
+  // CertIssuerSource implementation:
+  void SyncGetIssuersOf(
+      const ParsedCertificate* cert,
+      std::vector<scoped_refptr<ParsedCertificate>>* issuers) override;
+  void AsyncGetIssuersOf(const ParsedCertificate* cert,
+                         const IssuerCallback& issuers_callback,
+                         std::unique_ptr<Request>* out_req) override;
+
+ private:
+  class AiaRequest : public Request {

[eroman, 2016/06/04 01:08:47]

Can you move this into the .cc file?

[mattm, 2016/06/04 03:24:08]

Done.

+   public:
+    explicit AiaRequest(const IssuerCallback& issuers_callback);
+    ~AiaRequest() override;
+
+    // CertIssuerSource::Request implementation.
+    CompletionStatus GetNext(
+        scoped_refptr<ParsedCertificate>* out_cert) override;
+
+    void AddCertFetcherRequest(
+        std::unique_ptr<CertNetFetcher::Request> cert_fetcher_request);
+
+    void FetchCallback(Error, const std::vector<uint8_t>&);
+
+   private:
+    IssuerCallback issuers_callback_;
+    std::vector<std::unique_ptr<CertNetFetcher::Request>>
+        cert_fetcher_requests_;
+    unsigned pending_requests_ = 0;

[eroman, 2016/06/04 01:08:47]

size_t ?

[mattm, 2016/06/04 03:24:08]

oh yeah, that would make sense since they're getting stuck in a vector. done.

+    std::vector<scoped_refptr<ParsedCertificate>> results_;
+    size_t current_result_ = 0;
+
+    DISALLOW_COPY_AND_ASSIGN(AiaRequest);
+  };
+
+  CertNetFetcher* cert_fetcher_;
+
+  DISALLOW_COPY_AND_ASSIGN(CertIssuerSourceAia);
+};
+
+}  // namespace net
+
+#endif  // NET_CERT_INTERNAL_CERT_ISSUER_SOURCE_AIA_H_