QuicServer: Use Chrome's header parsing rather than Balsa

net/net.gyp

 # Copyright 2013 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.
 
 {
   'variables': {
     'chromium_code': 1,
 
     'linux_link_kerberos%': 0,
     'use_tracing_cache_backend%': 0,
@@ skipping 523 matching lines @@
       'type': '<(gtest_target_type)',
       'dependencies': [
         '../base/base.gyp:base',
         '../base/base.gyp:base_i18n',
         '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
         '../crypto/crypto.gyp:crypto',
         '../testing/gmock.gyp:gmock',
         '../testing/gtest.gyp:gtest',
         '../third_party/zlib/zlib.gyp:zlib',
         '../url/url.gyp:url_lib',
-        'balsa',
         'http_server',
         'net',
         'net_derived_sources',
         'net_test_support',
         'quic_ported_server',
       ],
       'sources': [
         '<@(net_test_sources)',
       ],
       'conditions': [
@@ skipping 555 matching lines @@
         'tools/dump_cache/url_to_filename_encoder.cc',
         'tools/dump_cache/url_to_filename_encoder.h',
         'tools/dump_cache/url_utilities.h',
         'tools/dump_cache/url_utilities.cc',
       ],
       # TODO(jschuh): crbug.com/167187 fix size_t to int truncations.
       'msvs_disabled_warnings': [4267, ],
     },
     {
       # This is a temporary target which will be merged into 'net' once the
       # dependency on balsa is eliminated and the classes are actually used.

[wtc, 2014/06/26 21:23:16]

If this comment is right, we should be able to merge quic_ported_server into net
now, right?

[dmziegler, 2014/06/27 01:11:37]

It would be *possible*, but do we actually want to bloat the binary before it's
necessary?

[wtc, 2014/06/27 21:00:38]

The binary will only be bigger when 'net' is built as a shared library. If 'net'
is built as a static library, the unused object files are discarded when the
static library is linked with an executable program or shared library.

[dmziegler, 2014/07/01 18:23:38]

Ah, I didn't know that. Great, I'll move it into 'net' in another CL.

       'target_name': 'quic_ported_server',
       'type': 'static_library',
       'dependencies': [
         '../base/base.gyp:base',
         '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
         '../url/url.gyp:url_lib',
-        'balsa',
         'net',
       ],
       'sources': [
         'quic/quic_dispatcher.cc',
         'quic/quic_dispatcher.h',
         'quic/quic_in_memory_cache.cc',
         'quic/quic_in_memory_cache.h',
         'quic/quic_per_connection_packet_writer.cc',
         'quic/quic_per_connection_packet_writer.h',
         'quic/quic_server.cc',
@@ skipping 303 matching lines @@
           'dependencies': [
             '../base/base.gyp:base',
             'flip_in_mem_edsm_server_base',
             'net',
           ],
           'sources': [
             'tools/flip_server/flip_in_mem_edsm_server.cc',
           ],
         },
         {
           'target_name': 'quic_base',

[wtc, 2014/06/26 21:23:16]

Do we still need the quic_base target?

[dmziegler, 2014/06/27 01:11:37]

For now, yes -- it still runs more unit tests than the ported code, and
quic_client still depends on it. That will change soon, though.

           'type': 'static_library',
           'dependencies': [
             '../base/base.gyp:base',
             '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
             '../url/url.gyp:url_lib',
             'balsa',
             'epoll_server',
             'net',
           ],
           'sources': [
@@ skipping 238 matching lines @@
             'net_unittests.isolate',
           ],
           'sources': [
             'net_unittests.isolate',
           ],
         },
       ],
     }],
   ],
 }