Use loadable module instead of shared library.

mojo/mojo.gyp

 # Copyright 2013 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'target_defaults': {
     'conditions': [
       ['mojo_shell_debug_url != ""', {
         'defines': [
           'MOJO_SHELL_DEBUG=1',
           'MOJO_SHELL_DEBUG_URL="<(mojo_shell_debug_url)"',
          ],
       }],
     ],
   },
   'variables': {
     'chromium_code': 1,
     'mojo_shell_debug_url%': "",
+    'common_sources': [

[viettrungluu, 2014/06/30 16:27:18]

Probably you should have a comment explaining why we have variables for these.
Something like: "We build two versions of these libraries, a version for use in
Chromium and the shell and an "embedded" version for use in loadable
apps/services.".

[qsr, 2014/07/01 08:09:35]

Done.

+      'common/common_type_converters.cc',
+      'common/common_type_converters.h',
+      'common/data_pipe_utils.cc',
+      'common/data_pipe_utils.h',
+      'common/handle_watcher.cc',
+      'common/handle_watcher.h',
+      'common/message_pump_mojo.cc',
+      'common/message_pump_mojo.h',
+      'common/message_pump_mojo_handler.h',
+      'common/time_helper.cc',
+      'common/time_helper.h',
+    ],
+    'environment_chromium_sources': [
+      'environment/default_async_waiter_impl.cc',
+      'environment/default_async_waiter_impl.h',
+      'environment/default_logger_impl.cc',
+      'environment/default_logger_impl.h',
+      'environment/environment.cc',
+      # TODO(vtl): This is kind of ugly. (See TODO in logging.h.)
+      "public/cpp/environment/logging.h",
+      "public/cpp/environment/lib/logging.h",
+    ],
   },
   'includes': [
     'mojo_apps.gypi',
     'mojo_examples.gypi',
     'mojo_public.gypi',
     'mojo_services.gypi',
   ],
   'targets': [
     {
       'target_name': 'mojo',
       'type': 'none',
       'dependencies': [
         'mojo_apps_js_unittests',
         'mojo_compositor_app',
+        'mojo_common_embedded_lib',
         'mojo_common_lib',
         'mojo_common_unittests',
         'mojo_cpp_bindings',
         'mojo_geometry_lib',
         'mojo_html_viewer',
         'mojo_js',
         'mojo_js_bindings',
         'mojo_js_unittests',
         'mojo_launcher',
         'mojo_message_generator',
@@ skipping 272 matching lines @@
       'type': 'static_library',
       'dependencies': [
         '../base/base.gyp:base',
       ],
       'sources': [
         'common/test/test_support_impl.cc',
         'common/test/test_support_impl.h',
       ],
     },
     {
+      'target_name': 'mojo_common_embedded_lib',

[viettrungluu, 2014/06/30 16:27:17]

It'd be nice if the non-embedded and embedded versions were in the same order in
the .gyp file (e.g., non-embedded, then embedded).

You should also have comments before each along the lines of "For use in
Chromium and the shell."/"For use in loadable apps and services.".

[qsr, 2014/07/01 08:09:35]

Done.

+      'type': 'static_library',
+      'defines': [
+        'MOJO_COMMON_EMBEDDED',
+      ],
+      'dependencies': [
+        '..//base/base.gyp:base',
+        '..//base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
+        'mojo_system',
+      ],
+      'sources': [
+        '<@(common_sources)',
+      ],
+      'all_dependent_settings': {
+        'defines': [
+          'MOJO_COMMON_EMBEDDED',
+        ],
+      },
+    },
+    {
       # GN version: //mojo/common
       'target_name': 'mojo_common_lib',
       'type': '<(component)',
       'defines': [
         'MOJO_COMMON_IMPLEMENTATION',
       ],
       'dependencies': [
         '../base/base.gyp:base',
         '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
         'mojo_system_impl',
       ],
       'export_dependent_settings': [
+        '../base/base.gyp:base',
         '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
         'mojo_system_impl',
       ],
       'sources': [
-        'common/common_type_converters.cc',
-        'common/common_type_converters.h',
-        'common/data_pipe_utils.cc',
-        'common/data_pipe_utils.h',
-        'common/handle_watcher.cc',
-        'common/handle_watcher.h',
-        'common/message_pump_mojo.cc',
-        'common/message_pump_mojo.h',
-        'common/message_pump_mojo_handler.h',
-        'common/time_helper.cc',
-        'common/time_helper.h',
+        '<@(common_sources)',
       ],
     },
     {
       'target_name': 'mojo_common_test_support',
       'type': 'static_library',
       'dependencies': [
         '../base/base.gyp:base',
         '../base/base.gyp:test_support_base',
         '../testing/gtest.gyp:gtest',
         'mojo_system_impl',
@@ skipping 21 matching lines @@
         'mojo_run_all_unittests',
       ],
       'sources': [
         'common/common_type_converters_unittest.cc',
         'common/handle_watcher_unittest.cc',
         'common/message_pump_mojo_unittest.cc',
         'common/test/multiprocess_test_helper_unittest.cc',
       ],
     },
     {
       # GN version: //mojo/environment:chromium

[viettrungluu, 2014/06/30 16:27:18]

Presumably you need to update the GN build as well.

[qsr, 2014/07/01 08:09:35]

Done.

       'target_name': 'mojo_environment_chromium',
       'type': 'static_library',
       'dependencies': [
+        '../base/base.gyp:base',
+        '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
         'mojo_common_lib',
-        'mojo_environment_chromium_impl',
+        'mojo_system_impl',
+      ],
+      'export_dependent_settings': [
+        '../base/base.gyp:base',
+        '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
       ],
       'sources': [
-        'environment/environment.cc',
-        # TODO(vtl): This is kind of ugly. (See TODO in logging.h.)
-        "public/cpp/environment/logging.h",
-        "public/cpp/environment/lib/logging.h",
-      ],
-      'include_dirs': [
-        '..',
-      ],
-      'export_dependent_settings': [
-        'mojo_environment_chromium_impl',
+        '<@(environment_chromium_sources)',
       ],
     },
     {
-      # GN version: //mojo/environment:chromium_impl
-      'target_name': 'mojo_environment_chromium_impl',
-      'type': '<(component)',
-      'defines': [
-        'MOJO_ENVIRONMENT_IMPL_IMPLEMENTATION',
-      ],
+      'target_name': 'mojo_environment_chromium_embedded',
+      'type': 'static_library',
       'dependencies': [
         '../base/base.gyp:base',
         '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
-        'mojo_common_lib'
+        'mojo_common_embedded_lib',
+        'mojo_system',
+      ],
+      'export_dependent_settings': [
+        '../base/base.gyp:base',
+        '../base/third_party/dynamic_annotations/dynamic_annotations.gyp:dynamic_annotations',
       ],
       'sources': [
-        'environment/default_async_waiter_impl.cc',
-        'environment/default_async_waiter_impl.h',
-        'environment/default_logger_impl.cc',
-        'environment/default_logger_impl.h',
-      ],
-      'include_dirs': [
-        '..',
+        '<@(environment_chromium_sources)',
       ],
     },
     {
       # GN version: //mojo/service_manager
       'target_name': 'mojo_service_manager',
       'type': '<(component)',
       'defines': [
         'MOJO_SERVICE_MANAGER_IMPLEMENTATION',
       ],
       'dependencies': [
@@ skipping 120 matching lines @@
         }],
       ],
     },
     {
       'target_name': 'mojo_shell_test_support',
       'type': 'static_library',
       'dependencies': [
         '../base/base.gyp:base',
         '../base/base.gyp:base_static',
         '../url/url.gyp:url_lib',
+        'mojo_common_lib',
         'mojo_service_manager',
         'mojo_shell_lib',
         'mojo_system_impl',
       ],
       'sources': [
         'shell/shell_test_helper.cc',
         'shell/shell_test_helper.h',
       ],
     },
     {
@@ skipping 408 matching lines @@
             'views/views_init_internal.h',
           ],
           'defines': [
             'MOJO_VIEWS_IMPLEMENTATION',
           ],
         },
       ],
     }],
   ],
 }