QUIC: don't ignore SetKey and SetNoncePrefix return values.

net/quic/crypto/crypto_utils.h

 // Copyright (c) 2013 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.
 //
 // Some helpers for quic crypto
 
 #ifndef NET_QUIC_CRYPTO_CRYPTO_UTILS_H_
 #define NET_QUIC_CRYPTO_CRYPTO_UTILS_H_
 
 #include <string>
@@ skipping 37 matching lines @@
   // Returns |hostname|. NormalizeHostname() doesn't support IP address
   // literals. IsValidSNI() should be called before calling NormalizeHostname().
   static std::string NormalizeHostname(const char* hostname);
 
   // DeriveKeys populates |out->encrypter| and |out->decrypter| given the
   // contents of |premaster_secret|, |client_nonce|, |server_nonce| and
   // |hkdf_input|. |aead| determines which cipher will be used. |perspective|
   // controls whether the server's keys are assigned to |encrypter| or
   // |decrypter|. |server_nonce| is optional and, if non-empty, is mixed into
   // the key derivation.
-  static void DeriveKeys(base::StringPiece premaster_secret,
+  static bool DeriveKeys(base::StringPiece premaster_secret,

[wtc, 2013/10/09 23:01:28]

Nit: document the return value.

                          QuicTag aead,
                          base::StringPiece client_nonce,
                          base::StringPiece server_nonce,
                          const std::string& hkdf_input,
                          Perspective perspective,
                          CrypterPair* out);
 };
 
 }  // namespace net
 
 #endif  // NET_QUIC_CRYPTO_CRYPTO_UTILS_H_