Change HpackHuffmanDecoder::CanonicalToSource() return type.

net/spdy/core/hpack/hpack_huffman_decoder_test.cc

 // 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.
 
 #include "net/spdy/core/hpack/hpack_huffman_decoder.h"
 
 #include <bitset>
 #include <limits>
 
 #include "base/logging.h"
@@ skipping 29 matching lines @@
  public:
   static HuffmanCodeLength CodeLengthOfPrefix(HuffmanWord value) {
     return HpackHuffmanDecoder::CodeLengthOfPrefix(value);
   }
 
   static HuffmanWord DecodeToCanonical(HuffmanCodeLength code_length,
                                        HuffmanWord bits) {
     return HpackHuffmanDecoder::DecodeToCanonical(code_length, bits);
   }
 
-  static char CanonicalToSource(HuffmanWord canonical) {
+  static uint8_t CanonicalToSource(HuffmanWord canonical) {
     return HpackHuffmanDecoder::CanonicalToSource(canonical);
   }
 };
 
 // Tests of the ability to decode the HPACK Huffman Code, defined in:
 //     https://httpwg.github.io/specs/rfc7541.html#huffman.code
 class HpackHuffmanDecoderTest : public ::testing::Test {
  protected:
   HpackHuffmanDecoderTest() : table_(ObtainHpackHuffmanTable()) {}
 
   // Since kHpackHuffmanCode doesn't include the canonical symbol value,
   // this helper helps us to decode directly to the source symbol, allowing
   // for EOS.
   uint16_t DecodeToSource(HuffmanCodeLength code_length, HuffmanWord bits) {
     HuffmanWord canonical =
         HpackHuffmanDecoderPeer::DecodeToCanonical(code_length, bits);
     EXPECT_LE(canonical, 256u);
     if (canonical == 256u) {
       return canonical;
     }
-    return static_cast<unsigned char>(
-        HpackHuffmanDecoderPeer::CanonicalToSource(canonical));
+    return HpackHuffmanDecoderPeer::CanonicalToSource(canonical);
   }
 
   void EncodeString(SpdyStringPiece input, SpdyString* encoded) {
     HpackOutputStream output_stream;
     table_.EncodeString(input, &output_stream);
     encoded->clear();
     output_stream.TakeString(encoded);
     // Verify EncodedSize() agrees with EncodeString().
     EXPECT_EQ(encoded->size(), table_.EncodedSize(input));
   }
@@ skipping 163 matching lines @@
     }
     EncodeString(input, &encoded);
     HpackInputStream input_stream(encoded);
     EXPECT_TRUE(HpackHuffmanDecoder::DecodeString(&input_stream, &decoded));
     EXPECT_EQ(input, decoded);
   }
 }
 
 }  // namespace test
 }  // namespace net