Bug 1239870 - Part 1: Remove declaration of test_utils from header. r=bwc
authorEric Rahm <erahm@mozilla.com>
Tue, 09 Feb 2016 10:02:37 -0800
changeset 322275 19d1c699e24cca7d81a44ba7d31fce0b4e4d754e
parent 322274 7314001ba54ce6358b7090c488130738d50858fb
child 322276 7ed796792608f7ab558a0bf35328acd2e33bef48
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
-