summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--chrome/browser_tests.isolate15
-rw-r--r--chrome/chrome.isolate4
-rw-r--r--chrome/interactive_ui_tests.isolate3
-rw-r--r--chrome/sync_integration_tests.isolate2
-rw-r--r--chrome/unit_tests.isolate3
-rw-r--r--content/content_browsertests.isolate5
-rw-r--r--content/content_unittests.isolate3
-rw-r--r--third_party/libjingle/libjingle.gyp15
-rw-r--r--third_party/libjingle/overrides/init_webrtc.cc19
9 files changed, 14 insertions, 55 deletions
diff --git a/chrome/browser_tests.isolate b/chrome/browser_tests.isolate
index 1fff561..a7eb600 100644
--- a/chrome/browser_tests.isolate
+++ b/chrome/browser_tests.isolate
@@ -14,16 +14,15 @@
],
'isolate_dependency_tracked': [
'../testing/xvfb.py',
+ '<(PRODUCT_DIR)/DumpRenderTree<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/chrome.pak',
'<(PRODUCT_DIR)/chrome<(EXECUTABLE_SUFFIX)',
- '<(PRODUCT_DIR)/DumpRenderTree<(EXECUTABLE_SUFFIX)',
- '<(PRODUCT_DIR)/lib/libpeerconnection.so',
'<(PRODUCT_DIR)/libclearkeycdm.so',
'<(PRODUCT_DIR)/libclearkeycdmadapter.so',
'<(PRODUCT_DIR)/libffmpegsumo.so',
'<(PRODUCT_DIR)/libosmesa.so',
- '<(PRODUCT_DIR)/libppapi_tests.so',
'<(PRODUCT_DIR)/libppGoogleNaClPluginChrome.so',
+ '<(PRODUCT_DIR)/libppapi_tests.so',
'<(PRODUCT_DIR)/nacl_helper<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/nacl_helper_bootstrap<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
@@ -101,11 +100,10 @@
'variables': {
'isolate_dependency_tracked': [
'<(PRODUCT_DIR)/ffmpegsumo.so',
- '<(PRODUCT_DIR)/libpeerconnection.so',
- '<(PRODUCT_DIR)/plugins/npapi_test_plugin.plugin/Contents/Info.plist',
- '<(PRODUCT_DIR)/plugins/npapi_test_plugin.plugin/Contents/MacOS/npapi_test_plugin',
'<(PRODUCT_DIR)/plugins/TestNetscapePlugIn.plugin/Contents/Info.plist',
'<(PRODUCT_DIR)/plugins/TestNetscapePlugIn.plugin/Contents/MacOS/TestNetscapePlugIn',
+ '<(PRODUCT_DIR)/plugins/npapi_test_plugin.plugin/Contents/Info.plist',
+ '<(PRODUCT_DIR)/plugins/npapi_test_plugin.plugin/Contents/MacOS/npapi_test_plugin',
'<(PRODUCT_DIR)/ppapi_nacl_tests_pnacl_newlib_x32.nexe',
'<(PRODUCT_DIR)/ppapi_tests.plugin/Contents/MacOS/ppapi_tests',
],
@@ -131,19 +129,18 @@
'../native_client/build/build_nexe.py',
'../third_party/cygwin/bin/bash.exe',
'../third_party/cygwin/setup_mount.bat',
+ '<(PRODUCT_DIR)/D3DCompiler_43.dll',
'<(PRODUCT_DIR)/clearkeycdm.dll',
'<(PRODUCT_DIR)/clearkeycdmadapter.dll',
- '<(PRODUCT_DIR)/D3DCompiler_43.dll',
'<(PRODUCT_DIR)/d3dcompiler_46.dll',
'<(PRODUCT_DIR)/ffmpegsumo.dll',
'<(PRODUCT_DIR)/icudt.dll',
'<(PRODUCT_DIR)/libEGL.dll',
'<(PRODUCT_DIR)/libGLESv2.dll',
- '<(PRODUCT_DIR)/libpeerconnection.dll',
'<(PRODUCT_DIR)/nacl64<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/osmesa.dll',
- '<(PRODUCT_DIR)/ppapi_tests.dll',
'<(PRODUCT_DIR)/ppGoogleNaClPluginChrome.dll',
+ '<(PRODUCT_DIR)/ppapi_tests.dll',
'<(PRODUCT_DIR)/security_tests.dll',
'tools/build/repack_locales.py',
],
diff --git a/chrome/chrome.isolate b/chrome/chrome.isolate
index d115c8e..86233c4 100644
--- a/chrome/chrome.isolate
+++ b/chrome/chrome.isolate
@@ -7,7 +7,6 @@
'variables': {
'isolate_dependency_tracked': [
'<(PRODUCT_DIR)/chrome.pak',
- '<(PRODUCT_DIR)/lib/libpeerconnection.so',
'<(PRODUCT_DIR)/libclearkeycdm.so',
'<(PRODUCT_DIR)/libclearkeycdmadapter.so',
'<(PRODUCT_DIR)/libffmpegsumo.so',
@@ -52,16 +51,15 @@
['OS=="win"', {
'variables': {
'isolate_dependency_tracked': [
+ '<(PRODUCT_DIR)/D3DCompiler_43.dll',
'<(PRODUCT_DIR)/chrome.dll',
'<(PRODUCT_DIR)/clearkeycdm.dll',
'<(PRODUCT_DIR)/clearkeycdmadapter.dll',
- '<(PRODUCT_DIR)/D3DCompiler_43.dll',
'<(PRODUCT_DIR)/d3dcompiler_46.dll',
'<(PRODUCT_DIR)/ffmpegsumo.dll',
'<(PRODUCT_DIR)/icudt.dll',
'<(PRODUCT_DIR)/libEGL.dll',
'<(PRODUCT_DIR)/libGLESv2.dll',
- '<(PRODUCT_DIR)/libpeerconnection.dll',
'<(PRODUCT_DIR)/nacl64<(EXECUTABLE_SUFFIX)',
'<(PRODUCT_DIR)/osmesa.dll',
'<(PRODUCT_DIR)/ppGoogleNaClPluginChrome.dll',
diff --git a/chrome/interactive_ui_tests.isolate b/chrome/interactive_ui_tests.isolate
index cf3b315..ca57765 100644
--- a/chrome/interactive_ui_tests.isolate
+++ b/chrome/interactive_ui_tests.isolate
@@ -15,7 +15,6 @@
'isolate_dependency_tracked': [
'../testing/xvfb.py',
'<(PRODUCT_DIR)/chrome.pak',
- '<(PRODUCT_DIR)/lib/libpeerconnection.so',
'<(PRODUCT_DIR)/libffmpegsumo.so',
'<(PRODUCT_DIR)/locales/fr.pak',
'<(PRODUCT_DIR)/pyproto/google/__init__.py',
@@ -63,7 +62,6 @@
'variables': {
'isolate_dependency_tracked': [
'<(PRODUCT_DIR)/ffmpegsumo.so',
- '<(PRODUCT_DIR)/libpeerconnection.so',
],
'isolate_dependency_untracked': [
'<(PRODUCT_DIR)/Chromium Framework.framework/',
@@ -91,7 +89,6 @@
'<(PRODUCT_DIR)/icudt.dll',
'<(PRODUCT_DIR)/libEGL.dll',
'<(PRODUCT_DIR)/libGLESv2.dll',
- '<(PRODUCT_DIR)/libpeerconnection.dll',
'<(PRODUCT_DIR)/ppGoogleNaClPluginChrome.dll',
],
'isolate_dependency_untracked': [
diff --git a/chrome/sync_integration_tests.isolate b/chrome/sync_integration_tests.isolate
index 84a6186..76cf74e 100644
--- a/chrome/sync_integration_tests.isolate
+++ b/chrome/sync_integration_tests.isolate
@@ -16,7 +16,6 @@
'../testing/xvfb.py',
'<(PRODUCT_DIR)/chrome.pak',
'<(PRODUCT_DIR)/chrome_100_percent.pak',
- '<(PRODUCT_DIR)/lib/libpeerconnection.so',
'<(PRODUCT_DIR)/libffmpegsumo.so',
'<(PRODUCT_DIR)/libppGoogleNaClPluginChrome.so',
'<(PRODUCT_DIR)/locales/en-US.pak',
@@ -58,7 +57,6 @@
'<(PRODUCT_DIR)/sync_integration_tests<(EXECUTABLE_SUFFIX)',
],
'isolate_dependency_tracked': [
- '<(PRODUCT_DIR)/libpeerconnection.so',
'<(PRODUCT_DIR)/ffmpegsumo.so',
],
'isolate_dependency_untracked': [
diff --git a/chrome/unit_tests.isolate b/chrome/unit_tests.isolate
index 6679932..7bf9a51 100644
--- a/chrome/unit_tests.isolate
+++ b/chrome/unit_tests.isolate
@@ -14,7 +14,6 @@
'isolate_dependency_tracked': [
'../testing/xvfb.py',
'<(PRODUCT_DIR)/chrome.pak',
- '<(PRODUCT_DIR)/lib/libpeerconnection.so',
'<(PRODUCT_DIR)/libffmpegsumo.so',
'<(PRODUCT_DIR)/locales/fr.pak',
'<(PRODUCT_DIR)/xdisplaycheck<(EXECUTABLE_SUFFIX)',
@@ -65,7 +64,6 @@
'variables': {
'isolate_dependency_tracked': [
'<(PRODUCT_DIR)/ffmpegsumo.so',
- '<(PRODUCT_DIR)/libpeerconnection.so',
],
'isolate_dependency_untracked': [
'<(PRODUCT_DIR)/Chromium Framework.framework/',
@@ -91,7 +89,6 @@
'isolate_dependency_tracked': [
'<(PRODUCT_DIR)/ffmpegsumo.dll',
'<(PRODUCT_DIR)/icudt.dll',
- '<(PRODUCT_DIR)/libpeerconnection.dll',
],
'isolate_dependency_untracked': [
'../ppapi/lib/gl/include/KHR/',
diff --git a/content/content_browsertests.isolate b/content/content_browsertests.isolate
index ec5134b..2ed7523 100644
--- a/content/content_browsertests.isolate
+++ b/content/content_browsertests.isolate
@@ -29,7 +29,6 @@
'isolate_dependency_tracked': [
'<(PRODUCT_DIR)/content_shell.pak',
'<(PRODUCT_DIR)/fonts.conf',
- '<(PRODUCT_DIR)/lib/libpeerconnection.so',
'<(PRODUCT_DIR)/libclearkeycdm.so',
'<(PRODUCT_DIR)/libclearkeycdmadapter.so',
'<(PRODUCT_DIR)/libffmpegsumo.so',
@@ -51,10 +50,9 @@
'variables': {
'isolate_dependency_untracked': [
'<(PRODUCT_DIR)/clearkeycdmadapter.plugin',
- '<(PRODUCT_DIR)/Content Shell.app/',
'<(PRODUCT_DIR)/content_resources.pak',
+ '<(PRODUCT_DIR)/Content Shell.app/',
'<(PRODUCT_DIR)/ffmpegsumo.so',
- '<(PRODUCT_DIR)/libpeerconnection.so',
'<(PRODUCT_DIR)/plugins/npapi_test_plugin.plugin',
'<(PRODUCT_DIR)/plugins/TestNestscapePlugin.plugin',
],
@@ -64,7 +62,6 @@
'variables': {
'isolate_dependency_tracked': [
'<(PRODUCT_DIR)/ffmpegsumo.dll',
- '<(PRODUCT_DIR)/libpeerconnection.dll',
],
'isolate_dependency_untracked': [
# The two directories below are only needed for
diff --git a/content/content_unittests.isolate b/content/content_unittests.isolate
index 71a3870..efcd163 100644
--- a/content/content_unittests.isolate
+++ b/content/content_unittests.isolate
@@ -12,7 +12,6 @@
'<(PRODUCT_DIR)/content_unittests<(EXECUTABLE_SUFFIX)',
],
'isolate_dependency_untracked': [
- '<(PRODUCT_DIR)/lib/libpeerconnection.so',
'<(PRODUCT_DIR)/libffmpegsumo.so',
'<(PRODUCT_DIR)/test_data/',
],
@@ -38,7 +37,6 @@
'isolate_dependency_untracked': [
'<(PRODUCT_DIR)/content_resources.pak',
'<(PRODUCT_DIR)/ffmpegsumo.so',
- '<(PRODUCT_DIR)/libpeerconnection.so',
],
},
}],
@@ -56,7 +54,6 @@
'isolate_dependency_untracked': [
'<(PRODUCT_DIR)/ffmpegsumo.dll',
'<(PRODUCT_DIR)/icudt.dll',
- '<(PRODUCT_DIR)/libpeerconnection.dll',
],
},
}],
diff --git a/third_party/libjingle/libjingle.gyp b/third_party/libjingle/libjingle.gyp
index 61b2c58..aa39c5a 100644
--- a/third_party/libjingle/libjingle.gyp
+++ b/third_party/libjingle/libjingle.gyp
@@ -11,13 +11,8 @@
'libjingle_additional_deps%': [],
'libjingle_peerconnection_additional_deps%': [],
'libjingle_source%': "source",
- 'conditions': [
- ['OS=="win" or OS=="mac" or OS=="linux"', {
- 'libpeer_target_type%': 'loadable_module',
- }, {
- 'libpeer_target_type%': 'static_library',
- }],
- ],
+ 'libpeer_target_type%': 'static_library',
+ 'libpeer_allocator_shim%': 0,
},
'target_defaults': {
'defines': [
@@ -765,7 +760,8 @@
'<(libjingle_source)/talk/session/tunnel/tunnelsessionclient.h',
],
'conditions': [
- ['libpeer_target_type!="static_library" and OS!="mac"', {
+ ['libpeer_allocator_shim==1 and '
+ 'libpeer_target_type!="static_library" and OS!="mac"', {
'sources': [
'overrides/allocator_shim/allocator_stub.cc',
'overrides/allocator_shim/allocator_stub.h',
@@ -870,7 +866,8 @@
'libjingle_webrtc',
],
'conditions': [
- ['libpeer_target_type!="static_library"', {
+ ['libpeer_allocator_shim==1 and '
+ 'libpeer_target_type!="static_library"', {
'sources': [
'overrides/initialize_module.cc',
],
diff --git a/third_party/libjingle/overrides/init_webrtc.cc b/third_party/libjingle/overrides/init_webrtc.cc
index 1376546..c73b270 100644
--- a/third_party/libjingle/overrides/init_webrtc.cc
+++ b/third_party/libjingle/overrides/init_webrtc.cc
@@ -51,21 +51,6 @@ static base::FilePath GetLibPeerConnectionPath() {
return path;
}
-#if !defined(OS_WIN)
-// On platforms other than Windows (i.e. mac and linux based ones),
-// the location of the .so in the installed layout will be different relative
-// to the loading module than it is in the build directory.
-// GetLibPeerConnectionPath returns the path as it will be in the installed
-// layout, but to support isolated tests, we need to also support loading the
-// so in the build layout, which we do here.
-// On Windows, these layouts are the same.
-static base::FilePath GetLibPeerConnectionPathForTests() {
- base::FilePath path;
- CHECK(PathService::Get(base::DIR_MODULE, &path));
- return path.Append(FILE_PATH_LITERAL("libpeerconnection.so"));
-}
-#endif
-
bool InitializeWebRtcModule() {
if (g_create_webrtc_media_engine)
return true; // InitializeWebRtcModule has already been called.
@@ -76,10 +61,6 @@ bool InitializeWebRtcModule() {
std::string error;
static base::NativeLibrary lib =
base::LoadNativeLibrary(path, &error);
-#if !defined(OS_WIN)
- if (!lib)
- lib = base::LoadNativeLibrary(GetLibPeerConnectionPathForTests(), &error);
-#endif
CHECK(lib);
InitializeModuleFunction initialize_module =