summaryrefslogtreecommitdiffstats
path: root/third_party
diff options
context:
space:
mode:
authorguoweis <guoweis@chromium.org>2014-09-19 12:18:38 -0700
committerCommit bot <commit-bot@chromium.org>2014-09-19 19:18:53 +0000
commitb35d075d92b70680c815ba0787215aceb695d0b0 (patch)
treee5aa90b5cde68fab023a4260be6490b6f02422a6 /third_party
parentcb0114633bb478c4636126d9d2ed87c1580e9ff8 (diff)
downloadchromium_src-b35d075d92b70680c815ba0787215aceb695d0b0.zip
chromium_src-b35d075d92b70680c815ba0787215aceb695d0b0.tar.gz
chromium_src-b35d075d92b70680c815ba0787215aceb695d0b0.tar.bz2
Move FindFullName outside LIBPEERCONNECTION_LIB definition block. This is to solve a link break when new finch experiment is defined inside peerconnection.cc while libpeerconnection.so is built as a shared library.
BUG= Review URL: https://codereview.chromium.org/582133002 Cr-Commit-Position: refs/heads/master@{#295758}
Diffstat (limited to 'third_party')
-rw-r--r--third_party/libjingle/overrides/init_webrtc.cc36
1 files changed, 19 insertions, 17 deletions
diff --git a/third_party/libjingle/overrides/init_webrtc.cc b/third_party/libjingle/overrides/init_webrtc.cc
index e11f78d..4a2fe73 100644
--- a/third_party/libjingle/overrides/init_webrtc.cc
+++ b/third_party/libjingle/overrides/init_webrtc.cc
@@ -32,6 +32,16 @@ void AddTraceEvent(char phase,
NULL, flags);
}
+// Define webrtc:field_trial::FindFullName to provide webrtc with a field trial
+// implementation.
+namespace webrtc {
+namespace field_trial {
+std::string FindFullName(const std::string& trial_name) {
+ return base::FieldTrialList::FindFullName(trial_name);
+}
+} // namespace field_trial
+} // namespace webrtc
+
#if defined(LIBPEERCONNECTION_LIB)
// libpeerconnection is being compiled as a static lib. In this case
@@ -43,17 +53,6 @@ bool InitializeWebRtcModule() {
return true;
}
-// Define webrtc:field_trial::FindFullName to provide webrtc with a field trial
-// implementation. When compiled as a static library this can be done directly
-// and without pointers to functions.
-namespace webrtc {
-namespace field_trial {
-std::string FindFullName(const std::string& trial_name) {
- return base::FieldTrialList::FindFullName(trial_name);
-}
-} // namespace field_trial
-} // namespace webrtc
-
#else // !LIBPEERCONNECTION_LIB
// When being compiled as a shared library, we need to bridge the gap between
@@ -127,13 +126,16 @@ bool InitializeWebRtcModule() {
InitDiagnosticLoggingDelegateFunctionFunction init_diagnostic_logging = NULL;
bool init_ok = initialize_module(*CommandLine::ForCurrentProcess(),
#if !defined(OS_MACOSX) && !defined(OS_ANDROID)
- &Allocate, &Dellocate,
+ &Allocate,
+ &Dellocate,
#endif
- &base::FieldTrialList::FindFullName,
- logging::GetLogMessageHandler(),
- &GetCategoryGroupEnabled, &AddTraceEvent,
- &g_create_webrtc_media_engine, &g_destroy_webrtc_media_engine,
- &init_diagnostic_logging);
+ &webrtc::field_trial::FindFullName,
+ logging::GetLogMessageHandler(),
+ &GetCategoryGroupEnabled,
+ &AddTraceEvent,
+ &g_create_webrtc_media_engine,
+ &g_destroy_webrtc_media_engine,
+ &init_diagnostic_logging);
if (init_ok)
rtc::SetExtraLoggingInit(init_diagnostic_logging);