Add NetLog event for invalid Http/2 response header

net/spdy/chromium/buffered_spdy_framer_unittest.cc

Index: net/spdy/chromium/buffered_spdy_framer_unittest.cc
diff --git a/net/spdy/chromium/buffered_spdy_framer_unittest.cc b/net/spdy/chromium/buffered_spdy_framer_unittest.cc
index 3f3868f2a80444ec51ef5402a0dd6e836788fdeb..62de0fbb4f371f6689a8289efd21b1585d5240a0 100644
--- a/net/spdy/chromium/buffered_spdy_framer_unittest.cc
+++ b/net/spdy/chromium/buffered_spdy_framer_unittest.cc
@@ -8,6 +8,7 @@
 #include <utility>
 
 #include "base/logging.h"
+#include "net/log/net_log_with_source.h"
 #include "net/spdy/chromium/spdy_test_util_common.h"
 #include "testing/platform_test.h"
 
@@ -18,7 +19,7 @@ namespace {
 class TestBufferedSpdyVisitor : public BufferedSpdyFramerVisitorInterface {
  public:
   TestBufferedSpdyVisitor()
-      : buffered_spdy_framer_(),
+      : buffered_spdy_framer_(NetLogWithSource()),
         error_count_(0),
         setting_count_(0),
         headers_frame_count_(0),
@@ -198,7 +199,7 @@ TEST_F(BufferedSpdyFramerTest, HeaderListTooLarge) {
   headers["foo"] = long_header_value;
   SpdyHeadersIR headers_ir(/*stream_id=*/1, std::move(headers));
 
-  BufferedSpdyFramer framer;
+  BufferedSpdyFramer framer((NetLogWithSource()));

[Bence, 2017/05/10 12:43:29]

I think there is an extra pair of () here that should be removed, here and five
more instances below.

[xunjieli, 2017/05/10 18:24:59]

Done. Compiler complains that I need to extra (). I have un-inlined the
initialization. hopefully looks slightly better.

   SpdySerializedFrame control_frame = framer.SerializeFrame(headers_ir);
 
   TestBufferedSpdyVisitor visitor;
@@ -239,7 +240,7 @@ TEST_F(BufferedSpdyFramerTest, ReadHeadersHeaderBlock) {
   headers["gamma"] = "delta";
   SpdyHeadersIR headers_ir(/*stream_id=*/1, headers.Clone());
 
-  BufferedSpdyFramer framer;
+  BufferedSpdyFramer framer((NetLogWithSource()));
   SpdySerializedFrame control_frame = framer.SerializeFrame(headers_ir);
 
   TestBufferedSpdyVisitor visitor;
@@ -254,7 +255,7 @@ TEST_F(BufferedSpdyFramerTest, ReadPushPromiseHeaderBlock) {
   SpdyHeaderBlock headers;
   headers["alpha"] = "beta";
   headers["gamma"] = "delta";
-  BufferedSpdyFramer framer;
+  BufferedSpdyFramer framer((NetLogWithSource()));
   SpdyPushPromiseIR push_promise_ir(/*stream_id=*/1, /*promised_stream_id=*/2,
                                     headers.Clone());
   SpdySerializedFrame control_frame = framer.SerializeFrame(push_promise_ir);
@@ -272,7 +273,7 @@ TEST_F(BufferedSpdyFramerTest, ReadPushPromiseHeaderBlock) {
 TEST_F(BufferedSpdyFramerTest, GoAwayDebugData) {
   SpdyGoAwayIR go_ir(/*last_accepted_stream_id=*/2, ERROR_CODE_FRAME_SIZE_ERROR,
                      "foo");
-  BufferedSpdyFramer framer;
+  BufferedSpdyFramer framer((NetLogWithSource()));
   SpdySerializedFrame goaway_frame = framer.SerializeFrame(go_ir);
 
   TestBufferedSpdyVisitor visitor;
@@ -294,7 +295,7 @@ TEST_F(BufferedSpdyFramerTest, OnAltSvcOnStreamZero) {
   altsvc_ir.add_altsvc(alternative_service);
   const char altsvc_origin[] = "https://www.example.org";
   altsvc_ir.set_origin(altsvc_origin);
-  BufferedSpdyFramer framer;
+  BufferedSpdyFramer framer((NetLogWithSource()));
   SpdySerializedFrame altsvc_frame(framer.SerializeFrame(altsvc_ir));
 
   TestBufferedSpdyVisitor visitor;
@@ -315,7 +316,7 @@ TEST_F(BufferedSpdyFramerTest, OnAltSvcOnNonzeroStream) {
       "quic", "alternative.example.org", 443, 86400,
       SpdyAltSvcWireFormat::VersionVector());
   altsvc_ir.add_altsvc(alternative_service);
-  BufferedSpdyFramer framer;
+  BufferedSpdyFramer framer((NetLogWithSource()));
   SpdySerializedFrame altsvc_frame(framer.SerializeFrame(altsvc_ir));
 
   TestBufferedSpdyVisitor visitor;