Change HpackHuffmanDecoder::CanonicalToSource() return type.

net/spdy/core/hpack/hpack_huffman_decoder.h

Index: net/spdy/core/hpack/hpack_huffman_decoder.h
diff --git a/net/spdy/core/hpack/hpack_huffman_decoder.h b/net/spdy/core/hpack/hpack_huffman_decoder.h
index d33fa58174f3597da07db36caaa3f772538fefc1..a749752e40fc6efedf8c0fcbf323703d0049e035 100644
--- a/net/spdy/core/hpack/hpack_huffman_decoder.h
+++ b/net/spdy/core/hpack/hpack_huffman_decoder.h
@@ -59,7 +59,7 @@ class SPDY_EXPORT_PRIVATE HpackHuffmanDecoder {
 
   // Converts a canonical symbol to the source symbol (the char in the original

[xunjieli, 2017/05/26 18:54:20]

nit: comment still mentions "char"

[Bence, 2017/05/26 18:57:50]

Done.

My understanding is that "char" refers to character (as "letter") as opposed to
a particular data type.  Changing to octet to align with language of RFC instead
of changing to uint8_t which looks a bit awkward.

   // string that was encoded).
-  static char CanonicalToSource(HuffmanWord canonical);
+  static uint8_t CanonicalToSource(HuffmanWord canonical);
 };
 
 }  // namespace net