diff options
author | guoweis <guoweis@chromium.org> | 2014-09-19 12:18:38 -0700 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2014-09-19 19:18:53 +0000 |
commit | b35d075d92b70680c815ba0787215aceb695d0b0 (patch) | |
tree | e5aa90b5cde68fab023a4260be6490b6f02422a6 /third_party | |
parent | cb0114633bb478c4636126d9d2ed87c1580e9ff8 (diff) | |
download | chromium_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.cc | 36 |
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); |