Remove SpdyFramerVisitorInterface::OnControlFrameHeaderData().

net/spdy/spdy_framer_test.cc

 // Copyright (c) 2012 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/spdy_framer.h"
 
 #include <stdlib.h>
 #include <string.h>
 
 #include <algorithm>
@@ skipping 147 matching lines @@
                            size_t len) override {
       LOG(FATAL);
     }
 
     void OnStreamEnd(SpdyStreamId stream_id) override { LOG(FATAL); }
 
     void OnStreamPadding(SpdyStreamId stream_id, size_t len) override {
       LOG(FATAL);
     }
 
-    bool OnControlFrameHeaderData(SpdyStreamId stream_id,
-                                  const char* header_data,
-                                  size_t len) override {
-      LOG(FATAL);
-      return true;
-    }
-
     void OnRstStream(SpdyStreamId stream_id,
                      SpdyRstStreamStatus status) override {
       LOG(FATAL);
     }
     void OnSetting(SpdySettingsIds id, uint8_t flags, uint32_t value) override {
       LOG(FATAL);
     }
     void OnPing(SpdyPingId unique_id, bool is_ack) override { LOG(FATAL); }
     void OnSettingsEnd() override { LOG(FATAL); }
     void OnGoAway(SpdyStreamId last_accepted_stream_id,
@@ skipping 199 matching lines @@
 
   void OnHeaderFrameEnd(SpdyStreamId stream_id, bool end_headers) override {
     CHECK(headers_handler_ != nullptr);
     headers_ = headers_handler_->decoded_block().Clone();
     header_bytes_received_ = headers_handler_->header_bytes_parsed();
     if (end_headers) {
       headers_handler_.reset();
     }
   }
 
-  bool OnControlFrameHeaderData(SpdyStreamId stream_id,
-                                const char* header_data,
-                                size_t len) override {
-    VLOG(1) << "OnControlFrameHeaderData(" << stream_id << ", data, " << len
-            << ")";
-    ++control_frame_header_data_count_;
-    CHECK_EQ(header_stream_id_, stream_id);
-    if (len == 0) {
-      ++zero_length_control_frame_header_data_count_;
-      // Indicates end-of-header-block.
-      headers_.clear();
-      CHECK(header_buffer_valid_);
-      return framer_.ParseHeaderBlockInBuffer(header_buffer_.get(),
-                                              header_buffer_length_, &headers_);
-    }
-    const size_t available = header_buffer_size_ - header_buffer_length_;
-    if (len > available) {
-      header_buffer_valid_ = false;
-      return false;
-    }
-    memcpy(header_buffer_.get() + header_buffer_length_, header_data, len);
-    header_buffer_length_ += len;
-    return true;
-  }
-
   void OnSynStream(SpdyStreamId stream_id,
                    SpdyStreamId associated_stream_id,
                    SpdyPriority priority,
                    bool fin,
                    bool unidirectional) override {
     VLOG(1) << "OnSynStream(" << stream_id << ", " << associated_stream_id
             << ", " << priority << ", " << (fin ? 1 : 0) << ", "
             << (unidirectional ? 1 : 0) << ")";
     ++syn_frame_count_;
     if (framer_.protocol_version() == SPDY3) {
@@ skipping 5995 matching lines @@
 
     EXPECT_EQ(1, visitor->data_frame_count_);
     EXPECT_EQ(strlen(four_score), static_cast<unsigned>(visitor->data_bytes_));
     EXPECT_EQ(0, visitor->headers_frame_count_);
   }
 }
 
 }  // namespace test
 
 }  // namespace net