Revert of Add new HTTP/2 and HPACK decoder in net/http2/.

Revert of Add new HTTP/2 and HPACK decoder in net/http2/. (patchset #19 id:400001 of https://codereview.chromium.org/2293613002/ )

Reason for revert:
DecodeBufferDeathTest.NonNullBufferRequired failing on chromium.mac/Mac10.9 Tests (dbg), 
Win7 Tests (dbg)(1) [2 since first detection]

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.win%2FWin7_Tests__dbg__1_%2F55329%2F%2B%2Frecipes%2Fsteps%2Fnet_unittests%2F0%2Flogs%2FDecodeBufferDeathTest.NonNullBufferRequired%2F0

Original issue's description:
> Add new HTTP/2 and HPACK decoder in net/http2/.
> 
> A follow-up CL will add flags and new classes to net/spdy
> that will benefit from this new implementation.
> 
> This CL lands all server changes in the http2 directory
> up to 129999723 by jamessynge.
> 
> BUG=488484
> 
> Committed: https://crrev.com/167743e8a7af542f317afde21d4ba19adc5b1f8f
> Cr-Commit-Position: refs/heads/master@{#436466}

TBR=jamessynge@chromium.org,rch@chromium.org,bnc@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=488484

[Adam Rice, 2016-12-06 01:59:23]

The CQ bit was checked by ricea@chromium.org

[Adam Rice, 2016-12-06 01:59:24]

Created Revert of Add new HTTP/2 and HPACK decoder in net/http2/.

[commit-bot: I haz the power, 2016-12-06 01:59:57]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-12-06 02:00:49]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2016-12-06 02:00:50]

Failed to apply patch for net/net.gypi:
While running git apply --index -p1;
  error: patch failed: net/net.gypi:784
  error: net/net.gypi: patch does not apply

Patch:       net/net.gypi
Index: net/net.gypi
diff --git a/net/net.gypi b/net/net.gypi
index
fb5c7bb4349a615c474dedfb51427138f7269dbd..3a532c164ac540aaece97d4454b5a1301531f146
100644
--- a/net/net.gypi
+++ b/net/net.gypi
@@ -784,69 +784,6 @@
       'http/url_security_manager.h',
       'http/url_security_manager_posix.cc',
       'http/url_security_manager_win.cc',
-      'http2/decoder/decode_buffer.cc',
-      'http2/decoder/decode_buffer.h',
-      'http2/decoder/decode_http2_structures.cc',
-      'http2/decoder/decode_http2_structures.h',
-      'http2/decoder/decode_status.cc',
-      'http2/decoder/decode_status.h',
-      'http2/decoder/frame_decoder_state.cc',
-      'http2/decoder/frame_decoder_state.h',
-      'http2/decoder/http2_frame_decoder.cc',
-      'http2/decoder/http2_frame_decoder.h',
-      'http2/decoder/http2_frame_decoder_listener.cc',
-      'http2/decoder/http2_frame_decoder_listener.h',
-      'http2/decoder/http2_structure_decoder.cc',
-      'http2/decoder/http2_structure_decoder.h',
-      'http2/decoder/payload_decoders/altsvc_payload_decoder.cc',
-      'http2/decoder/payload_decoders/altsvc_payload_decoder.h',
-      'http2/decoder/payload_decoders/continuation_payload_decoder.cc',
-      'http2/decoder/payload_decoders/continuation_payload_decoder.h',
-      'http2/decoder/payload_decoders/data_payload_decoder.cc',
-      'http2/decoder/payload_decoders/data_payload_decoder.h',
-      'http2/decoder/payload_decoders/goaway_payload_decoder.cc',
-      'http2/decoder/payload_decoders/goaway_payload_decoder.h',
-      'http2/decoder/payload_decoders/headers_payload_decoder.cc',
-      'http2/decoder/payload_decoders/headers_payload_decoder.h',
-      'http2/decoder/payload_decoders/ping_payload_decoder.cc',
-      'http2/decoder/payload_decoders/ping_payload_decoder.h',
-      'http2/decoder/payload_decoders/priority_payload_decoder.cc',
-      'http2/decoder/payload_decoders/priority_payload_decoder.h',
-      'http2/decoder/payload_decoders/push_promise_payload_decoder.cc',
-      'http2/decoder/payload_decoders/push_promise_payload_decoder.h',
-      'http2/decoder/payload_decoders/rst_stream_payload_decoder.cc',
-      'http2/decoder/payload_decoders/rst_stream_payload_decoder.h',
-      'http2/decoder/payload_decoders/settings_payload_decoder.cc',
-      'http2/decoder/payload_decoders/settings_payload_decoder.h',
-      'http2/decoder/payload_decoders/unknown_payload_decoder.cc',
-      'http2/decoder/payload_decoders/unknown_payload_decoder.h',
-      'http2/decoder/payload_decoders/window_update_payload_decoder.cc',
-      'http2/decoder/payload_decoders/window_update_payload_decoder.h',
-      'http2/hpack/decoder/hpack_block_decoder.cc',
-      'http2/hpack/decoder/hpack_block_decoder.h',
-      'http2/hpack/decoder/hpack_decoder_string_buffer.cc',
-      'http2/hpack/decoder/hpack_decoder_string_buffer.h',
-      'http2/hpack/decoder/hpack_entry_decoder.cc',
-      'http2/hpack/decoder/hpack_entry_decoder.h',
-      'http2/hpack/decoder/hpack_entry_decoder_listener.cc',
-      'http2/hpack/decoder/hpack_entry_decoder_listener.h',
-      'http2/hpack/decoder/hpack_entry_type_decoder.cc',
-      'http2/hpack/decoder/hpack_entry_type_decoder.h',
-      'http2/hpack/decoder/hpack_string_decoder.cc',
-      'http2/hpack/decoder/hpack_string_decoder.h',
-      'http2/hpack/decoder/hpack_string_decoder_listener.cc',
-      'http2/hpack/decoder/hpack_string_decoder_listener.h',
-      'http2/hpack/decoder/hpack_varint_decoder.cc',
-      'http2/hpack/decoder/hpack_varint_decoder.h',
-      'http2/hpack/http2_hpack_constants.cc',
-      'http2/hpack/http2_hpack_constants.h',
-      'http2/hpack/huffman/http2_hpack_huffman_decoder.cc',
-      'http2/hpack/huffman/http2_hpack_huffman_decoder.h',
-      'http2/http2_constants.cc',
-      'http2/http2_constants.h',
-      'http2/http2_structures.cc',
-      'http2/http2_structures.h',
-      'http2/tools/http2_bug_tracker.h',
       'nqe/cached_network_quality.cc',
       'nqe/cached_network_quality.h',
       'nqe/effective_connection_type.cc',
@@ -1741,68 +1678,6 @@
       'http/transport_security_persister_unittest.cc',
       'http/transport_security_state_unittest.cc',
       'http/url_security_manager_unittest.cc',
-      'http2/decoder/decode_buffer_test.cc',
-      'http2/decoder/decode_http2_structures_test.cc',
-      'http2/decoder/frame_decoder_state_test_util.cc',
-      'http2/decoder/frame_decoder_state_test_util.h',
-      'http2/decoder/frame_parts.cc',
-      'http2/decoder/frame_parts.h',
-      'http2/decoder/frame_parts_collector.cc',
-      'http2/decoder/frame_parts_collector.h',
-      'http2/decoder/frame_parts_collector_listener.cc',
-      'http2/decoder/frame_parts_collector_listener.h',
-      'http2/decoder/http2_frame_decoder_listener_test_util.cc',
-      'http2/decoder/http2_frame_decoder_listener_test_util.h',
-      'http2/decoder/http2_frame_decoder_test.cc',
-      'http2/decoder/http2_structure_decoder_test.cc',
-      'http2/decoder/http2_structure_decoder_test_util.h',
-      'http2/decoder/payload_decoders/altsvc_payload_decoder_test.cc',
-      'http2/decoder/payload_decoders/continuation_payload_decoder_test.cc',
-      'http2/decoder/payload_decoders/data_payload_decoder_test.cc',
-      'http2/decoder/payload_decoders/goaway_payload_decoder_test.cc',
-      'http2/decoder/payload_decoders/headers_payload_decoder_test.cc',
-      'http2/decoder/payload_decoders/payload_decoder_base_test_util.cc',
-      'http2/decoder/payload_decoders/payload_decoder_base_test_util.h',
-      'http2/decoder/payload_decoders/ping_payload_decoder_test.cc',
-      'http2/decoder/payload_decoders/priority_payload_decoder_test.cc',
-      'http2/decoder/payload_decoders/push_promise_payload_decoder_test.cc',
-      'http2/decoder/payload_decoders/rst_stream_payload_decoder_test.cc',
-      'http2/decoder/payload_decoders/settings_payload_decoder_test.cc',
-      'http2/decoder/payload_decoders/unknown_payload_decoder_test.cc',
-      'http2/decoder/payload_decoders/window_update_payload_decoder_test.cc',
-      'http2/hpack/decoder/hpack_block_collector.cc',
-      'http2/hpack/decoder/hpack_block_collector.h',
-      'http2/hpack/decoder/hpack_block_decoder_test.cc',
-      'http2/hpack/decoder/hpack_decoder_string_buffer_test.cc',
-      'http2/hpack/decoder/hpack_entry_collector.cc',
-      'http2/hpack/decoder/hpack_entry_collector.h',
-      'http2/hpack/decoder/hpack_entry_decoder_test.cc',
-      'http2/hpack/decoder/hpack_entry_type_decoder_test.cc',
-      'http2/hpack/decoder/hpack_string_collector.cc',
-      'http2/hpack/decoder/hpack_string_collector.h',
-      'http2/hpack/decoder/hpack_string_decoder_test.cc',
-      'http2/hpack/decoder/hpack_varint_decoder_test.cc',
-      'http2/hpack/http2_hpack_constants_test.cc',
-      'http2/hpack/huffman/http2_hpack_huffman_decoder_test.cc',
-      'http2/hpack/tools/hpack_block_builder.cc',
-      'http2/hpack/tools/hpack_block_builder.h',
-      'http2/hpack/tools/hpack_block_builder_test.cc',
-      'http2/hpack/tools/hpack_example.cc',
-      'http2/hpack/tools/hpack_example.h',
-      'http2/http2_constants_test.cc',
-      'http2/http2_constants_test_util.cc',
-      'http2/http2_constants_test_util.h',
-      'http2/http2_structures_test.cc',
-      'http2/http2_structures_test_util.cc',
-      'http2/http2_structures_test_util.h',
-      'http2/tools/failure.cc',
-      'http2/tools/failure.h',
-      'http2/tools/http2_frame_builder.cc',
-      'http2/tools/http2_frame_builder.h',
-      'http2/tools/http2_random.cc',
-      'http2/tools/http2_random.h',
-      'http2/tools/random_decoder_test.cc',
-      'http2/tools/random_decoder_test.h',
       'log/bounded_file_net_log_observer_unittest.cc',
       'log/net_log_capture_mode_unittest.cc',
       'log/net_log_unittest.cc',

[Adam Rice, 2016-12-06 02:54:47]

Had a mid-air collision with another revert of the same CL. Closing.

[Adam Rice, 2016-12-06 02:55:02]

Description was changed from

==========
Revert of Add new HTTP/2 and HPACK decoder in net/http2/. (patchset #19
id:400001 of https://codereview.chromium.org/2293613002/ )

Reason for revert:
DecodeBufferDeathTest.NonNullBufferRequired failing on chromium.mac/Mac10.9
Tests (dbg), 
Win7 Tests (dbg)(1) [2 since first detection]

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.win%2FWin7_Test...

Original issue's description:
> Add new HTTP/2 and HPACK decoder in net/http2/.
> 
> A follow-up CL will add flags and new classes to net/spdy
> that will benefit from this new implementation.
> 
> This CL lands all server changes in the http2 directory
> up to 129999723 by jamessynge.
> 
> BUG=488484
> 
> Committed: https://crrev.com/167743e8a7af542f317afde21d4ba19adc5b1f8f
> Cr-Commit-Position: refs/heads/master@{#436466}

TBR=jamessynge@chromium.org,rch@chromium.org,bnc@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=488484
==========

to

==========
Revert of Add new HTTP/2 and HPACK decoder in net/http2/. (patchset #19
id:400001 of https://codereview.chromium.org/2293613002/ )

Reason for revert:
DecodeBufferDeathTest.NonNullBufferRequired failing on chromium.mac/Mac10.9
Tests (dbg), 
Win7 Tests (dbg)(1) [2 since first detection]

https://luci-logdog.appspot.com/v/?s=chromium%2Fbb%2Fchromium.win%2FWin7_Test...

Original issue's description:
> Add new HTTP/2 and HPACK decoder in net/http2/.
> 
> A follow-up CL will add flags and new classes to net/spdy
> that will benefit from this new implementation.
> 
> This CL lands all server changes in the http2 directory
> up to 129999723 by jamessynge.
> 
> BUG=488484
> 
> Committed: https://crrev.com/167743e8a7af542f317afde21d4ba19adc5b1f8f
> Cr-Commit-Position: refs/heads/master@{#436466}

TBR=jamessynge@chromium.org,rch@chromium.org,bnc@chromium.org
# Skipping CQ checks because original CL landed less than 1 days ago.
NOPRESUBMIT=true
NOTREECHECKS=true
NOTRY=true
BUG=488484
==========