Bug 1239870 - Part 1: Remove declaration of test_utils from header. r=bwc
☠☠ backed out by 22424f6eeb30 ☠ ☠
authorEric Rahm <erahm@mozilla.com>
Tue, 09 Feb 2016 10:02:37 -0800
changeset 322233 eb99ab06414dd713fb54f428c54d1a958cdd32ac
parent 322232 a03ba8b50b93853954a37f72b45dbd2f2fd478b3
child 322234 ba077a3afbc702e15fe5a727dc00aabfc4f1becc
push id5913
push userjlund@mozilla.com
push dateMon, 25 Apr 2016 16:57:49 +0000
treeherdermozilla-beta@dcaf0a6fa115 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersbwc
bugs1239870
milestone47.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1239870 - Part 1: Remove declaration of test_utils from header. r=bwc Once files are all compiled into the same gtest this will cause duplicate symbol errors. It's also unused.
media/mtransport/test/mtransport_test_utils.h
--- a/media/mtransport/test/mtransport_test_utils.h
+++ b/media/mtransport/test/mtransport_test_utils.h
@@ -96,23 +96,17 @@ class MtransportTestUtils {
   nsCOMPtr<nsIEventTarget> sts_target_;
   nsCOMPtr<nsPISocketTransportService> sts_;
 #ifdef MOZ_CRASHREPORTER
   nsCOMPtr<nsICrashReporter> crashreporter_;
 #endif
 };
 
 
-MtransportTestUtils *mtransport_test_utils;
-
-#define SETUP_MTRANSPORT_TEST_UTILS() \
-  MtransportTestUtils utils_; mtransport_test_utils = &utils_
-
 #define CHECK_ENVIRONMENT_FLAG(envname) \
   char *test_flag = getenv(envname); \
   if (!test_flag || strcmp(test_flag, "1")) { \
     printf("To run this test set %s=1 in your environment\n", envname); \
     exit(0); \
   } \
 
 
 #endif
-