Relanding: Removing #defines previously used for building without BoringSSL/OpenSSL.

webrtc/BUILD.gn

 # Copyright (c) 2014 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.
 
 # TODO(kjellander): Rebase this to webrtc/build/common.gypi changes after r6330.
 
@@ skipping 12 matching lines @@
   defines = []
   cflags = []
   ldflags = []
   if (build_with_mozilla) {
     defines += [ "WEBRTC_MOZILLA_BUILD" ]
   }
   if (build_with_chromium) {
     defines = [
       # TODO(kjellander): Cleanup unused ones and move defines closer to
       # the source when webrtc:4256 is completed.
-      "FEATURE_ENABLE_SSL",
       "FEATURE_ENABLE_VOICEMAIL",
       "EXPAT_RELATIVE_PATH",
       "GTEST_RELATIVE_PATH",
       "NO_MAIN_THREAD_WRAPPING",
       "NO_SOUND_SYSTEM",
       "WEBRTC_CHROMIUM_BUILD",
     ]
     include_dirs = [
       # The overrides must be included first as that is the mechanism for
       # selecting the override headers in Chromium.
@@ skipping 81 matching lines @@
   if (rtc_enable_external_auth) {
     defines += [ "ENABLE_EXTERNAL_AUTH" ]
   }
 
   if (build_with_chromium) {
     defines += [
       # NOTICE: Since common_inherited_config is used in public_configs for our
       # targets, there's no point including the defines in that config here.
       # TODO(kjellander): Cleanup unused ones and move defines closer to the
       # source when webrtc:4256 is completed.
-      "HAVE_OPENSSL_SSL_H",

[kjellander_webrtc, 2017/01/20 07:33:52]

This define is still mentioned in
https://cs.chromium.org/chromium/src/third_party/usrsctp/localchanges.patch but
I don't see the output of that patch living anywhere. Should we clean it up?

[Taylor Brandstetter, 2017/01/20 18:26:15]

It looks like localchanges.patch is dead, and just wasn't removed by
https://chromiumcodereview.appspot.com/23960004. I'll remove it.

       "HAVE_SRTP",
       "HAVE_WEBRTC_VIDEO",
       "HAVE_WEBRTC_VOICE",
       "LOGGING_INSIDE_WEBRTC",
-      "SSL_USE_OPENSSL",

[kjellander_webrtc, 2017/01/20 07:33:52]

Same as for HAVE_OPENSSL_SSL_H above.

       "USE_WEBRTC_DEV_BRANCH",
     ]
   } else {
     if (is_posix) {
       # Enable more warnings: -Wextra is currently disabled in Chromium.
       cflags = [
         "-Wextra",
 
         # Repeat some flags that get overridden by -Wextra.
         "-Wno-unused-parameter",
@@ skipping 346 matching lines @@
       ]
 
       deps = [
         "//base:base_java_test_support",
         "//webrtc/examples:AppRTCMobile_javalib",
         "//webrtc/sdk/android:libjingle_peerconnection_java",
       ]
     }
   }
 }