Commit f7709215 authored by Rob Buis's avatar Rob Buis Committed by Commit Bot

Fix typos in web_package

Some typo and suggested corrections.

Bug: 803774

Change-Id: Ia695bb55f52b9c5777c6ace22db5fcf093c948f7
Reviewed-on: https://chromium-review.googlesource.com/1203371Reviewed-by: default avatarKouhei Ueno <kouhei@chromium.org>
Commit-Queue: Rob Buis <rbuis@igalia.com>
Cr-Commit-Position: refs/heads/master@{#588480}
parent 4f265d39
...@@ -46,7 +46,6 @@ class CONTENT_EXPORT SignedExchangeCertificateChain { ...@@ -46,7 +46,6 @@ class CONTENT_EXPORT SignedExchangeCertificateChain {
private: private:
scoped_refptr<net::X509Certificate> cert_; scoped_refptr<net::X509Certificate> cert_;
// Version b1 specific fields:
std::string ocsp_; std::string ocsp_;
std::string sct_; std::string sct_;
}; };
......
...@@ -23,7 +23,7 @@ namespace content { ...@@ -23,7 +23,7 @@ namespace content {
namespace { namespace {
// IsStateful{Request,Response}Header return true if |name| is a stateful // IsStateful{Request,Response}Header returns true if |name| is a stateful
// header field. Stateful header fields will cause validation failure of // header field. Stateful header fields will cause validation failure of
// signed exchanges. // signed exchanges.
// Note that |name| must be lower-cased. // Note that |name| must be lower-cased.
...@@ -210,7 +210,7 @@ bool ParseResponseMap(const cbor::CBORValue& value, ...@@ -210,7 +210,7 @@ bool ParseResponseMap(const cbor::CBORValue& value,
return false; return false;
} }
// 4. If exchange’s headers contain a stateful header field, as defined in // 4. If exchange’s headers contains a stateful header field, as defined in
// Section 4.1, return “invalid”. [spec text] // Section 4.1, return “invalid”. [spec text]
if (IsStatefulResponseHeader(name_str)) { if (IsStatefulResponseHeader(name_str)) {
signed_exchange_utils::ReportErrorAndTraceEvent( signed_exchange_utils::ReportErrorAndTraceEvent(
......
...@@ -33,7 +33,7 @@ class CONTENT_EXPORT SignedExchangeEnvelope { ...@@ -33,7 +33,7 @@ class CONTENT_EXPORT SignedExchangeEnvelope {
// Parse headers from the application/signed-exchange;v=b2 format. // Parse headers from the application/signed-exchange;v=b2 format.
// https://wicg.github.io/webpackage/draft-yasskin-httpbis-origin-signed-exchanges-impl.html#application-signed-exchange // https://wicg.github.io/webpackage/draft-yasskin-httpbis-origin-signed-exchanges-impl.html#application-signed-exchange
// //
// This also performs the step 1, 3 and 4 of "Cross-origin trust" validation. // This also performs the steps 1, 3 and 4 of "Cross-origin trust" validation.
// https://wicg.github.io/webpackage/draft-yasskin-httpbis-origin-signed-exchanges-impl.html#cross-origin-trust // https://wicg.github.io/webpackage/draft-yasskin-httpbis-origin-signed-exchanges-impl.html#cross-origin-trust
static base::Optional<SignedExchangeEnvelope> Parse( static base::Optional<SignedExchangeEnvelope> Parse(
const GURL& fallback_url, const GURL& fallback_url,
......
...@@ -140,7 +140,7 @@ SignedExchangeHandler::SignedExchangeHandler( ...@@ -140,7 +140,7 @@ SignedExchangeHandler::SignedExchangeHandler(
if (!IsSupportedSignedExchangeVersion(version_)) { if (!IsSupportedSignedExchangeVersion(version_)) {
signed_exchange_utils::ReportErrorAndTraceEvent( signed_exchange_utils::ReportErrorAndTraceEvent(
devtools_proxy_.get(), devtools_proxy_.get(),
base::StringPrintf("Unsupported version of the content type. Currentry " base::StringPrintf("Unsupported version of the content type. Currently "
"content type must be " "content type must be "
"\"application/signed-exchange;v=b2\". But the " "\"application/signed-exchange;v=b2\". But the "
"response content type was \"%s\"", "response content type was \"%s\"",
......
...@@ -45,12 +45,12 @@ class SignedExchangeCertificateChain; ...@@ -45,12 +45,12 @@ class SignedExchangeCertificateChain;
class SignedExchangeDevToolsProxy; class SignedExchangeDevToolsProxy;
// SignedExchangeHandler reads "application/signed-exchange" format from a // SignedExchangeHandler reads "application/signed-exchange" format from a
// net::SourceStream, parse and verify the signed exchange, and report // net::SourceStream, parses and verifies the signed exchange, and reports
// the result asynchronously via SignedExchangeHandler::ExchangeHeadersCallback. // the result asynchronously via SignedExchangeHandler::ExchangeHeadersCallback.
// //
// Note that verifying a signed exchange requires an associated certificate // Note that verifying a signed exchange requires an associated certificate
// chain. SignedExchangeHandler creates a SignedExchangeCertFetcher to // chain. SignedExchangeHandler creates a SignedExchangeCertFetcher to
// fetch the certificate chain over network, and verify it with the // fetch the certificate chain over the network, and verifies it with the
// net::CertVerifier. // net::CertVerifier.
class CONTENT_EXPORT SignedExchangeHandler { class CONTENT_EXPORT SignedExchangeHandler {
public: public:
......
...@@ -21,7 +21,7 @@ namespace content { ...@@ -21,7 +21,7 @@ namespace content {
class SignedExchangeDevToolsProxy; class SignedExchangeDevToolsProxy;
// SignedExchangeSignatureHeaderField provides parser for signed exchange's // SignedExchangeSignatureHeaderField provides a parser for signed exchange's
// `Signature` header field. // `Signature` header field.
// https://wicg.github.io/webpackage/draft-yasskin-httpbis-origin-signed-exchanges-impl.html // https://wicg.github.io/webpackage/draft-yasskin-httpbis-origin-signed-exchanges-impl.html
class CONTENT_EXPORT SignedExchangeSignatureHeaderField { class CONTENT_EXPORT SignedExchangeSignatureHeaderField {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment