Remove unneeded JNI registrations.

mojo/mojo_public.gypi

 {
   'targets': [
     {
       'target_name': 'mojo_system',
       'type': 'static_library',
       'defines': [
         'MOJO_SYSTEM_IMPLEMENTATION',
       ],
       'include_dirs': [
         '..',
       ],
       'direct_dependent_settings': {
         'include_dirs': [
           '..',
         ],
       },
       'sources': [
         'public/c/system/async_waiter.h',
         'public/c/system/core.h',
         'public/c/system/macros.h',
         'public/c/system/system_export.h',
         'public/platform/native/system_thunks.cc',
         'public/platform/native/system_thunks.h',
       ],

[DaveMoore, 2014/04/21 23:34:51]

The only special case related to exports that I know of is in mojo_system. I
believe if you rebase you'll see a recent change that explictly removes the
--exclude-libs=ALL from targets dependent on this, because part of the api to
users of this library is an exported function. Can this new script be used to
somehow automate this in a better way?

Also, I'm not sure why the 3 shared libraries you special case here are
different from the other mojo applications. Is it now a constraint that if a
library exports any api that it needs to have special code for android?

[Yaron, 2014/04/22 01:49:04]

This sort of already was the case because of exclude-libs=ALL since typically
everything's a static library. mojo is the first place I'm seeing in chromium
where we use a separate shared library in a non-component build. Previously we'd
only need need to export JNI_OnLoad and the JNI generator would generate all
needed associations for jni calls.

     },
     {
       'target_name': 'mojo_gles2',
       'type': 'shared_library',
       'defines': [
         'MOJO_GLES2_IMPLEMENTATION',
         'GLES2_USE_MOJO',
       ],
       'include_dirs': [
         '..',
       ],
       'dependencies': [
         '../third_party/khronos/khronos.gyp:khronos_headers'
       ],
       'direct_dependent_settings': {
         'include_dirs': [
           '..',
         ],
         'defines': [
           'GLES2_USE_MOJO',
         ],
       },
       'sources': [
         'public/c/gles2/gles2.h',
         'public/c/gles2/gles2_export.h',
         'public/gles2/gles2_private.cc',
         'public/gles2/gles2_private.h',
       ],
       'conditions': [
+        ['OS=="android"', {
+          'ldflags!': [
+            # Remove default export list because this lib has different exports.
+            '-Wl,--version-script="<(android_linker_script)"',
+          ],
+          'ldflags': [
+            # Don't export symbols from statically linked libraries.
+            '-Wl,--exclude-libs=ALL',
+          ],
+        }],
         ['OS=="mac"', {
           'xcode_settings': {
             # Make it a run-path dependent library.
             'DYLIB_INSTALL_NAME_BASE': '@loader_path',
           },
         }],
       ],
     },
     {
       'target_name': 'mojo_test_support',
       'type': 'shared_library',
       'defines': [
         'MOJO_TEST_SUPPORT_IMPLEMENTATION',
       ],
       'include_dirs': [
         '..',
       ],
       'direct_dependent_settings': {
         'include_dirs': [
           '..',
         ],
       },
       'sources': [
         'public/c/test_support/test_support.h',
         'public/c/test_support/test_support_export.h',
         'public/tests/test_support_private.cc',
         'public/tests/test_support_private.h',
       ],
       'conditions': [
+        ['OS=="android"', {
+          'ldflags!': [
+            # Remove default export list because this lib has different exports.
+            '-Wl,--version-script="<(android_linker_script)"',
+          ],
+          'ldflags': [
+            # Don't export symbols from statically linked libraries.
+            '-Wl,--exclude-libs=ALL',
+          ],
+        }],
         ['OS=="mac"', {
           'xcode_settings': {
             # Make it a run-path dependent library.
             'DYLIB_INSTALL_NAME_BASE': '@loader_path',
           },
         }],
       ],
     },
     {
       'target_name': 'mojo_public_test_utils',
@@ skipping 255 matching lines @@
       ],
       'dependencies': [
         'mojo_shell_bindings',
       ],
       'export_dependent_settings': [
         'mojo_shell_bindings',
       ],
     },
   ],
 }