Chromium Code Reviews

Unified Diff: mojo/mojo.gyp

Issue 477923004: Create native_viewport_service, don't build it into mojo_shell unless Android (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Remove workaround for thunks Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments.
Jump to:
View side-by-side diff with in-line comments
Index: mojo/mojo.gyp
diff --git a/mojo/mojo.gyp b/mojo/mojo.gyp
index c94cf09f336d76813dfe24442d3b45d8bd118973..fd3b31b3b15ce75fa051e1b986a751dcdd0591d8 100644
--- a/mojo/mojo.gyp
+++ b/mojo/mojo.gyp
@@ -38,7 +38,7 @@
'mojo_html_viewer',
'mojo_js',
'mojo_launcher',
- 'mojo_native_viewport_service_lib',
+ 'mojo_native_viewport_service',
'mojo_network_service',
'mojo_pepper_container_app',
'mojo_png_viewer',
@@ -144,8 +144,6 @@
'../base/base.gyp:base',
'../base/base.gyp:base_static',
'../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
- '../net/net.gyp:net',
- '../url/url.gyp:url_lib',
'mojo_application_manager',
'mojo_base.gyp:mojo_application_bindings',
'mojo_base.gyp:mojo_common_lib',
@@ -153,7 +151,6 @@
'mojo_base.gyp:mojo_system_impl',
'mojo_base.gyp:mojo_application_chromium',
'mojo_external_service_bindings',
- 'mojo_native_viewport_service_lib',
'mojo_network_bindings',
'mojo_spy',
],
@@ -191,10 +188,13 @@
'shell/test_child_process.h',
'shell/ui_application_loader_android.cc',
'shell/ui_application_loader_android.h',
- 'shell/view_manager_loader.cc',
- 'shell/view_manager_loader.h',
],
'conditions': [
+ ['component=="shared_library"', {
+ 'dependencies': [
+ '../ui/gl/gl.gyp:gl',
+ ],
+ }],
['OS=="linux"', {
'dependencies': [
'../build/linux/system.gyp:dbus',
@@ -204,25 +204,13 @@
['OS=="android"', {
'dependencies': [
'mojo_network_service_lib',
+ 'mojo_native_viewport_service_lib',
],
'sources': [
'shell/network_application_loader.cc',
'shell/network_application_loader.h',
],
}],
- ['use_aura==1', {
- 'dependencies': [
- # These are only necessary as long as we hard code use of ViewManager.
- '../skia/skia.gyp:skia',
- 'mojo_view_manager',
- 'mojo_view_manager_bindings',
- ],
- }, { # use_aura==0
- 'sources!': [
- 'shell/view_manager_loader.cc',
- 'shell/view_manager_loader.h',
- ],
- }],
],
},
{
@@ -248,8 +236,6 @@
'type': 'executable',
'dependencies': [
'../base/base.gyp:base',
- '../ui/gl/gl.gyp:gl',
- '../url/url.gyp:url_lib',
'mojo_application_manager',
'mojo_base.gyp:mojo_common_lib',
'mojo_base.gyp:mojo_environment_chromium',
@@ -277,8 +263,6 @@
'../base/base.gyp:test_support_base',
'../testing/gtest.gyp:gtest',
'../net/net.gyp:net_test_support',
- # TODO(vtl): We don't currently need this, but I imagine we will soon.
- # '../ui/gl/gl.gyp:gl',
'../url/url.gyp:url_lib',
'mojo_application_manager',
'mojo_base.gyp:mojo_common_lib',
@@ -412,7 +396,6 @@
'../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
'../ui/gfx/gfx.gyp:gfx',
'../ui/gfx/gfx.gyp:gfx_geometry',
- '../ui/gl/gl.gyp:gl',
'mojo_base.gyp:mojo_application_bindings',
'mojo_base.gyp:mojo_common_lib',
'mojo_base.gyp:mojo_environment_chromium',
@@ -477,7 +460,6 @@
'../ui/compositor/compositor.gyp:compositor',
'../ui/events/events.gyp:events',
'../ui/events/events.gyp:events_base',
- '../ui/gl/gl.gyp:gl',
'../webkit/common/gpu/webkit_gpu.gyp:webkit_gpu',
'mojo_cc_support',
'mojo_native_viewport_bindings',

Powered by Google App Engine