commit b75791185133bde02d01a078272f0e1de10950db
author Matt Menke <mmenke@chromium.org> Wed Jun 26 00:30:46 2019
committer Commit Bot <commit-bot@chromium.org> Wed Jun 26 00:30:46 2019
tree 3a83ff562318738c6381893ba2554a6f666a089f
parent febc547159134758df2c979342e742501d86d5e1

Fix typos in HttpServerPropertiesImpl, and make canonical_suffixes_ const.

Bug: None
Change-Id: I0d6e717a313ff7693bf5ce7ca38ce2f92c7cb0ae
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1676905
Commit-Queue: Ryan Hamilton <rch@chromium.org>
Reviewed-by: Ryan Hamilton <rch@chromium.org>
Cr-Commit-Position: refs/heads/master@{#672343}

net/http/http_server_properties_impl.cc

diff --git a/net/http/http_server_properties_impl.cc b/net/http/http_server_properties_impl.cc
index c6c7148..bf614b3 100644
--- a/net/http/http_server_properties_impl.cc
+++ b/net/http/http_server_properties_impl.cc
@@ -29,13 +29,10 @@
                              : base::DefaultTickClock::GetInstance()),
       clock_(clock ? clock : base::DefaultClock::GetInstance()),
       broken_alternative_services_(this, tick_clock_),
+      canonical_suffixes_({".ggpht.com", ".c.youtube.com", ".googlevideo.com",
+                           ".googleusercontent.com"}),
       quic_server_info_map_(kDefaultMaxQuicServerEntries),
-      max_server_configs_stored_in_properties_(kDefaultMaxQuicServerEntries) {
-  canonical_suffixes_.push_back(".ggpht.com");
-  canonical_suffixes_.push_back(".c.youtube.com");
-  canonical_suffixes_.push_back(".googlevideo.com");
-  canonical_suffixes_.push_back(".googleusercontent.com");
-}
+      max_server_configs_stored_in_properties_(kDefaultMaxQuicServerEntries) {}
 
 HttpServerPropertiesImpl::HttpServerPropertiesImpl()
     : HttpServerPropertiesImpl(nullptr, nullptr) {}

net/http/http_server_properties_impl.h

diff --git a/net/http/http_server_properties_impl.h b/net/http/http_server_properties_impl.h
index 5ae2d6b..863d153 100644
--- a/net/http/http_server_properties_impl.h
+++ b/net/http/http_server_properties_impl.h
@@ -159,7 +159,7 @@
       CanonicalAltSvcMap;
   typedef base::flat_map<HostPortPair, quic::QuicServerId>
       CanonicalServerInfoMap;
-  typedef std::vector<std::string> CanonicalSufficList;
+  typedef std::vector<std::string> CanonicalSuffixList;
   typedef std::set<HostPortPair> Http11ServerHostPortSet;
 
   // Return the iterator for |server|, or for its canonical host, or end.
@@ -202,9 +202,9 @@
   // to an actual origin, which has a plausible alternate protocol mapping.
   CanonicalAltSvcMap canonical_alt_svc_map_;
 
-  // Contains list of suffixes (for exmaple ".c.youtube.com",
+  // Contains list of suffixes (for example ".c.youtube.com",
   // ".googlevideo.com", ".googleusercontent.com") of canonical hostnames.
-  CanonicalSufficList canonical_suffixes_;
+  const CanonicalSuffixList canonical_suffixes_;
 
   QuicServerInfoMap quic_server_info_map_;
 
@@ -212,7 +212,7 @@
   // and have a corresponding entry in |quic_server_info_map_|. The map can be
   // used to quickly look for server info for hosts that share the same
   // canonical suffix but don't have exact match in |quic_server_info_map_|. The
-  // map exists solely to improve the search performance. It only contais
+  // map exists solely to improve the search performance. It only contains
   // derived data that can be recalculated by traversing
   // |quic_server_info_map_|.
   CanonicalServerInfoMap canonical_server_info_map_;