Replace the existing SpdyHeaderBlock typedef with a class.

net/spdy/hpack/hpack_encoder_test.cc

 // Copyright 2014 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/hpack/hpack_encoder.h"
 
 #include <map>
 #include <string>
 
 #include "testing/gmock/include/gmock/gmock.h"
@@ skipping 274 matching lines @@
   EXPECT_EQ(expected_out, actual_out);
 }
 
 TEST_F(HpackEncoderTest, MultipleEncodingPasses) {
   // Pass 1.
   {
     SpdyHeaderBlock headers;
     headers["key1"] = "value1";
     headers["cookie"] = "a=bb";
 
+    ExpectIndex(IndexOf(key_1_));
     ExpectIndex(IndexOf(cookie_a_));
-    ExpectIndex(IndexOf(key_1_));
     CompareWithExpectedEncoding(headers);
   }
   // Header table is:
   // 65: key1: value1
   // 64: key2: value2
   // 63: cookie: a=bb
   // 62: cookie: c=dd
   // Pass 2.
   {
     SpdyHeaderBlock headers;
     headers["key2"] = "value2";
     headers["cookie"] = "c=dd; e=ff";
 
-    ExpectIndex(IndexOf(cookie_c_));
+    // "key2: value2"
+    ExpectIndex(64);
+    // "cookie: c=dd"
+    ExpectIndex(62);
     // This cookie evicts |key1| from the dynamic table.
     ExpectIndexedLiteral(peer_.table()->GetByName("cookie"), "e=ff");
-    // "key2: value2"
-    ExpectIndex(65);
 
     CompareWithExpectedEncoding(headers);
   }
   // Header table is:
   // 65: key2: value2
   // 64: cookie: a=bb
   // 63: cookie: c=dd
   // 62: cookie: e=ff
   // Pass 3.
   {
     SpdyHeaderBlock headers;
     headers["key2"] = "value2";
     headers["cookie"] = "a=bb; b=cc; c=dd";
 
+    // "key2: value2"
+    ExpectIndex(65);
     // "cookie: a=bb"
     ExpectIndex(64);
     // This cookie evicts |key2| from the dynamic table.
     ExpectIndexedLiteral(peer_.table()->GetByName("cookie"), "b=cc");
     // "cookie: c=dd"
     ExpectIndex(64);
-    // "key2: value2"
-    ExpectIndexedLiteral("key2", "value2");
 
     CompareWithExpectedEncoding(headers);
   }
 }
 
 TEST_F(HpackEncoderTest, PseudoHeadersFirst) {
   SpdyHeaderBlock headers;
+  // A pseudo-header that should not be indexed.
+  headers[":path"] = "/spam/eggs.html";
   // A pseudo-header to be indexed.
   headers[":authority"] = "www.example.com";
-  // A pseudo-header that should not be indexed.
-  headers[":path"] = "/spam/eggs.html";
   // A regular header which precedes ":" alphabetically, should still be encoded
   // after pseudo-headers.
   headers["-foo"] = "bar";
   headers["foo"] = "bar";
   headers["cookie"] = "c=dd";
 
-  // Pseudo-headers are encoded in alphabetical order.
+  // Headers are indexed in the order in which they were added.
   // This entry pushes "cookie: a=bb" back to 63.
+  ExpectNonIndexedLiteral(":path", "/spam/eggs.html");
   ExpectIndexedLiteral(peer_.table()->GetByName(":authority"),
                        "www.example.com");
-  ExpectNonIndexedLiteral(":path", "/spam/eggs.html");
-  // Regular headers are endoded in alphabetical order.
-  // This entry pushes "cookie: a=bb" back to 64.
   ExpectIndexedLiteral("-foo", "bar");
-  ExpectIndex(64);
   ExpectIndexedLiteral("foo", "bar");
+  ExpectIndexedLiteral(peer_.table()->GetByName("cookie"), "c=dd");
   CompareWithExpectedEncoding(headers);
 }
 
 TEST_F(HpackEncoderTest, CookieToCrumbs) {
   test::HpackEncoderPeer peer(NULL);
   std::vector<StringPiece> out;
 
   // Leading and trailing whitespace is consumed. A space after ';' is consumed.
   // All other spaces remain. ';' at beginning and end of string produce empty
   // crumbs.
@@ skipping 82 matching lines @@
   expected_.AppendUint32(62);
   expected_.AppendPrefix(kStringLiteralIdentityEncoded);
   expected_.AppendUint32(3);
   expected_.AppendBytes("bar");
   CompareWithExpectedEncoding(headers);
 }
 
 }  // namespace
 
 }  // namespace net