Log GOAWAY frame debug data.

net/spdy/buffered_spdy_framer.cc

Index: net/spdy/buffered_spdy_framer.cc
diff --git a/net/spdy/buffered_spdy_framer.cc b/net/spdy/buffered_spdy_framer.cc
index 2fa15b8b9b860c2582e6ec1fb8cfc508af023bfa..a5e300b26f39e237c45d114df7283ed4bb5dd9f8 100644
--- a/net/spdy/buffered_spdy_framer.cc
+++ b/net/spdy/buffered_spdy_framer.cc
@@ -226,7 +226,22 @@ void BufferedSpdyFramer::OnRstStream(SpdyStreamId stream_id,
 }
 void BufferedSpdyFramer::OnGoAway(SpdyStreamId last_accepted_stream_id,
                                   SpdyGoAwayStatus status) {
-  visitor_->OnGoAway(last_accepted_stream_id, status);
+  DCHECK(!go_away_fields_.get());
+  go_away_fields_.reset(new GoAwayFields());
+  go_away_fields_->last_accepted_stream_id = last_accepted_stream_id;
+  go_away_fields_->status = status;
+}
+
+bool BufferedSpdyFramer::OnGoAwayFrameData(const char* goaway_data,
+                                           size_t len) {
+  if (len > 0) {
+    go_away_fields_->debug_data.append(goaway_data, len);
+    return true;
+  }
+  visitor_->OnGoAway(go_away_fields_->last_accepted_stream_id,
+                     go_away_fields_->status, go_away_fields_->debug_data);
+  go_away_fields_.reset(nullptr);
+  return true;
 }

[Ryan Hamilton, 2015/10/05 21:17:55]

This method wasn't overridden previously, right? What does the default
implementation do? I'm guessing it just returned true...

[Bence, 2015/10/06 15:18:57]

Exactly: it returns true.
https://code.google.com/p/chromium/codesearch#chromium/src/net/spdy/spdy_fram...

 
 void BufferedSpdyFramer::OnWindowUpdate(SpdyStreamId stream_id,