Revert 201501 "Land Recent QUIC changes."

Broke chromium.chrome build:
../../net/quic/test_tools/crypto_test_utils.cc:400:error: 'v' may be used uninitialized in this function


> Land Recent QUIC changes.
> 
> Use example.com as test hostname.
> 
> Merge internal change: 46048925
> 
> Only updating the time of last packet once we've done our best to verify packet validity.
> 
> Merge internal change: 46044184
> 
> Improve logging behavior in quic_connection.h.
> 
> Merge internal change: 46033559
> 
> Moving the public flags to the beginning of the header in preparation for variable length guids.
> 
> Merge internal change: 45980153
> 
> QUIC: change the GCM tag size to 12 bytes.
> 
> Merge internal change: 45973625
> 
> QUIC: add some crypto tests.
> 
> The client code tries to be correct, which can hamper some tests that wish to
> send invalid requests.
> 
> This CL contains some utilities for constructing arbitrary handshake messages
> and some tests that exercise the server crypto code.
> 
> Merge internal change: 45972782
> 
> Not allowing retransmissions to affect client timeouts.  Fixes a serious
> bug where if client vanishes and we have unacked packets, the connection
> could live on forever.
> 
> Merge internal change: 45935953
> 
> Address wtc's comments on cl/44272981.
> 
> Merge internal change: 45917323
> 
> QUIC: don't CHECK when QUIC is enabled without any certificates loaded.
> 
> Without certificates we don't have any key material for the source-address
> token nor server config and so QUIC isn't setup at server load time. However,
> if QUIC is enabled anyway then it'll crash.
> 
> This change removes the CHECK and has every crypto handshake fail instead.
> 
> (I have tests for the recent SNI change pending, into which a test for this
> will fall nicely, hopefully this afternoon. But I'm prioritising this change
> for now rather than waiting for the test CL to land.)
> 
> Merge internal change: 45914344
> 
> Merging cleanup changes from chromium
> 
> Merge internal change: 45797529
> 
> QUIC: pad client hello messages and require padding on the server.
> 
> This reduces any amplification factor that an attacker might get from us. I've
> picked a minimum size of 512 bytes out of thin air.
> 
> Satyam has a change pending that bumps the version to 2 so I've omitted that here.
> 
> Merge internal change: 45779287
> 
> QUIC: small fixes
> 
> * Don't send invalid SNIs as a client.
> * Don't require an SNI as a server.
> * Don't ignore client hello processing errors.
> 
> Merge internal change: 45774287
> 
> QUIC - set QUIC max stream per connections based on SNI.
> 
> Merge internal change: 45656436
> 
> - Enabled EndToEnd's Timeout unittest.
> - Ported IsValidSNI and NormalizeHostname from internal code.
> 
> R=rch@chromium.org
> 
> Review URL: https://chromiumcodereview.appspot.com/15385004

TBR=rtenneti@chromium.org
Review URL: https://codereview.chromium.org/15737008

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@201516 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 0 insertions, 19 deletions

diff --git a/net/quic/crypto/crypto_handshake.h b/net/quic/crypto/crypto_handshake.h
index 96a78a2..0774905 100644
--- a/net/quic/crypto/crypto_handshake.h
+++ b/net/quic/crypto/crypto_handshake.h
@@ -96,21 +96,6 @@ class NET_EXPORT_PRIVATE CryptoHandshakeMessage {
   QuicErrorCode GetUint32(QuicTag tag, uint32* out) const;
   QuicErrorCode GetUint64(QuicTag tag, uint64* out) const;
 
-  // size returns 4 (message tag) + 2 (uint16, number of entries) +
-  // (4 (tag) + 4 (end offset))*tag_value_map_.size() + ∑ value sizes.
-  size_t size() const;
-
-  // set_minimum_size sets the minimum number of bytes that the message should
-  // consume. The CryptoFramer will add a PAD tag as needed when serializing in
-  // order to ensure this. Setting a value of 0 disables padding.
-  //
-  // Padding is useful in order to ensure that messages are a minimum size. A
-  // QUIC server can require a minimum size in order to reduce the
-  // amplification factor of any mirror DoS attack.
-  void set_minimum_size(size_t min_bytes);
-
-  size_t minimum_size() const;
-
   // DebugString returns a multi-line, string representation of the message
   // suitable for including in debug output.
   std::string DebugString() const;
@@ -130,8 +115,6 @@ class NET_EXPORT_PRIVATE CryptoHandshakeMessage {
   QuicTag tag_;
   QuicTagValueMap tag_value_map_;
 
-  size_t minimum_size_;
-
   // The serialized form of the handshake message. This member is constructed
   // lasily.
   mutable scoped_ptr<QuicData> serialized_;
@@ -159,8 +142,6 @@ struct NET_EXPORT_PRIVATE QuicCryptoNegotiatedParameters {
   CrypterPair initial_crypters;
   CrypterPair forward_secure_crypters;
   std::string server_config_id;
-  // Normalized SNI: converted to lower case and trailing '.' removed.
-  std::string sni;
   std::string client_nonce;
   std::string server_nonce;
   // hkdf_input_suffix contains the HKDF input following the label: the GUID,