Enable BWE logging to command line when rtc_enable_bwe_test_logging is set to true

webrtc/modules/remote_bitrate_estimator/remote_bitrate_estimator.gypi

 # Copyright (c) 2011 The WebRTC project authors. All Rights Reserved.
 #
 # Use of this source code is governed by a BSD-style license
 # that can be found in the LICENSE file in the root of the source
 # tree. An additional intellectual property rights grant can be found
 # in the file PATENTS.  All contributing project authors may
 # be found in the AUTHORS file in the root of the source tree.
 
 {
   'includes': [
     '../../build/common.gypi',
   ],
   'variables': {
     # Set this to true to enable BWE test logging.
     'enable_bwe_test_logging%': 0,

[kjellander_webrtc, 2016/09/07 09:22:02]

Please remove this.

I thought when doing that it would need to be added to webrtc/build/common.gypi
but when I checked it's already there:
https://cs.chromium.org/chromium/src/third_party/webrtc/build/common.gypi?rcl...

I guess this one was missed in a previous CL.

   },
   'targets': [
     {
       'target_name': 'remote_bitrate_estimator',
       'type': 'static_library',
       'dependencies': [
         '<(webrtc_root)/common.gyp:webrtc_common',
         '<(webrtc_root)/system_wrappers/system_wrappers.gyp:system_wrappers',
       ],
       'sources': [
@@ skipping 14 matching lines @@
         'remote_bitrate_estimator_single_stream.cc',
         'remote_bitrate_estimator_single_stream.h',
         'remote_estimator_proxy.cc',
         'remote_estimator_proxy.h',
         'send_time_history.cc',
         'transport_feedback_adapter.cc',
         'transport_feedback_adapter.h',
         'test/bwe_test_logging.h',
       ], # source
       'conditions': [
-        ['enable_bwe_test_logging==1', {
-          'defines': [ 'BWE_TEST_LOGGING_COMPILE_TIME_ENABLE=1' ],
-          'sources': [
-            'test/bwe_test_logging.cc'
+        ['include_tests==0', {
+          'conditions': [
+            ['enable_bwe_test_logging==1', {
+              'defines': [ 'BWE_TEST_LOGGING_COMPILE_TIME_ENABLE=1' ],
+              'sources': [
+                'test/bwe_test_logging.cc'
+              ],
+            }, {
+              'defines': [ 'BWE_TEST_LOGGING_COMPILE_TIME_ENABLE=0' ],
+            }],
           ],
-        }, {
-          'defines': [ 'BWE_TEST_LOGGING_COMPILE_TIME_ENABLE=0' ],
         }],
       ],
     },
   ], # targets
   'conditions': [
     ['include_tests==1', {
       'targets': [
         {
           'target_name': 'bwe_simulator',
           'type': 'static_library',
           'dependencies': [
             '<(webrtc_root)/modules/modules.gyp:paced_sender',
             '<(webrtc_root)/test/test.gyp:test_support',
             '<(DEPTH)/testing/gtest.gyp:gtest',
             '<(DEPTH)/testing/gmock.gyp:gmock',
           ],
           'sources': [
+            'bwe_simulations.cc',
             'test/bwe.cc',
             'test/bwe.h',
             'test/bwe_test.cc',
             'test/bwe_test.h',
             'test/bwe_test_baselinefile.cc',
             'test/bwe_test_baselinefile.h',
             'test/bwe_test_fileutils.cc',
             'test/bwe_test_fileutils.h',
             'test/bwe_test_framework.cc',
             'test/bwe_test_framework.h',
@@ skipping 79 matching lines @@
             ],
           },
           'sources': [
             'tools/bwe_rtp_play.cc',
           ], # source
         },
       ],
     }],  # include_tests==1
   ],
 }