Added a net::BidirectionalStream to expose a bidirectional streaming interface

build/common.gypi

 # 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.
 
 # IMPORTANT:
 # Please don't directly include this file if you are building via gyp_chromium,
 # since gyp_chromium is automatically forcing its inclusion.
 {
   # Variables expected to be overriden on the GYP command line (-D) or by
   # ~/.gyp/include.gypi.
@@ skipping 579 matching lines @@
 
       # Enables support for background apps.
       'enable_background%': 1,
 
       # Enable the task manager by default.
       'enable_task_manager%': 1,
 
       # Enables used resource whitelist generation; disabled by default.
       'enable_resource_whitelist_generation%': 0,
 
+      # Enables BidrectionalSteam; disabled by default.
+      'enable_bidirectional_stream': 0,

[mef, 2015/12/17 22:42:09]

do we need to explicitly enable it in cronet?

[xunjieli, 2015/12/18 15:15:23]

In Cronet, we can use the preprocessor define ENABLE_BIDIRECTIONAL_STREAM. But
yes, we will need a follow-up CL to enable this when running net_unittests on
Cronet bots.

[mef, 2015/12/18 15:45:03]

Hrm, just setting the define ENABLE_BIDIRECTIONAL_STREAM on Cronet project won't
help, but it shouldn't be a problem to set in cr_cronet.py and on bots.

[xunjieli, 2015/12/18 15:59:29]

Ah, yes, you are right! The ENABLE_BIDIRECTIONAL_STREAM needs to be set when
compiling net/, so we either need a custom net target, or use the variable when
running gyp. We probably also need to have a custom Cronet build too for this
purpose. I don't imagine cronet customers of the UrlRequest interface is
interested in the BidirectionalStream support. 

+
       # Enable FILE support by default.
       'disable_file_support%': 0,
 
       # Enable FTP support by default.
       'disable_ftp_support%': 0,
 
       # Use of precompiled headers on Windows.
       #
       # This variable may be explicitly set to 1 (enabled) or 0
       # (disabled) in ~/.gyp/include.gypi or via the GYP command line.
@@ skipping 603 matching lines @@
     'test_isolation_mode%': '<(test_isolation_mode)',
     'enable_basic_printing%': '<(enable_basic_printing)',
     'enable_print_preview%': '<(enable_print_preview)',
     'enable_spellcheck%': '<(enable_spellcheck)',
     'use_browser_spellchecker%': '<(use_browser_spellchecker)',
     'cld_version%': '<(cld_version)',
     'cld2_table_size%': '<(cld2_table_size)',
     'enable_captive_portal_detection%': '<(enable_captive_portal_detection)',
     'disable_file_support%': '<(disable_file_support)',
     'disable_ftp_support%': '<(disable_ftp_support)',
+    'enable_bidirectional_stream%': '<(enable_bidirectional_stream)',
     'enable_task_manager%': '<(enable_task_manager)',
     'sas_dll_path%': '<(sas_dll_path)',
     'wix_path%': '<(wix_path)',
     'use_libjpeg_turbo%': '<(use_libjpeg_turbo)',
     'use_system_libjpeg%': '<(use_system_libjpeg)',
     'android_channel%': '<(android_channel)',
     'icu_use_data_file_flag%': '<(icu_use_data_file_flag)',
     'gyp_managed_install%': 0,
     'create_standalone_apk%': 1,
     'enable_app_list%': '<(enable_app_list)',
@@ skipping 5176 matching lines @@
     # settings in target dicts.  SYMROOT is a special case, because many other
     # Xcode variables depend on it, including variables such as
     # PROJECT_DERIVED_FILE_DIR.  When a source group corresponding to something
     # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
     # files to appear (when present) in the UI as actual files and not red
     # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
     # and therefore SYMROOT, needs to be set at the project level.
     'SYMROOT': '<(DEPTH)/xcodebuild',
   },
 }