[Android] For shared library build, use shared versions of stlport and gnu_stl

build/common.gypi

Index: build/common.gypi
diff --git a/build/common.gypi b/build/common.gypi
index 319e866561961edfb79bcfc31b555486590d5a25..5624724e0cc33f4c0d7de5a75851677f77e9d13a 100644
--- a/build/common.gypi
+++ b/build/common.gypi
@@ -2785,7 +2785,13 @@
           ['use_system_stlport==1', {
             'android_stlport_library': 'stlport',
           }, {
-            'android_stlport_library': 'stlport_static',
+            'conditions': [
+              ['component=="shared_library"', {
+                  'android_stlport_library': 'stlport_shared',
+              }, {
+                  'android_stlport_library': 'stlport_static',
+              }],
+            ],
           }],
         ],
 
@@ -2879,6 +2885,11 @@
               '-lm',
             ],
             'conditions': [
+              ['component=="shared_library"', {
+                'libraries': [
+                  '-lgnustl_shared',

[digit1, 2012/11/20 20:33:19]

sorry, didn't catch that earlier, but it's probably a bad idea to link to both
stlport and gnu libstdc++ at the same time. Is this really necesssary? Since
you're not adding the include paths to the gnustl headers to the gyp files, I
assume this is not.

[Yaron, 2012/11/21 02:58:30]

Probably true but since this is just in bring-up phase, I was going to see if it
caused issues down the line. For now, it appears to be needed for shared library
build at link time because of some missing symbols in icui18n:

third_party/android_tools/ndk/toolchains/arm-linux-androideabi-4.6/prebuilt/linux-x86_64/bin/../lib/gcc/arm-linux-androideabi/4.6.x-google/../../../../arm-linux-androideabi/bin/ld.gold:
obj/third_party/icu/source/i18n/icui18n.ucal.o: in function
ucal_setGregorianChange_46:../../third_party/icu/source/i18n/ucal.cpp:254:
error: undefined reference to '__cxa_bad_typeid'
third_party/android_tools/ndk/toolchains/arm-linux-androideabi-4.6/prebuilt/linux-x86_64/bin/../lib/gcc/arm-linux-androideabi/4.6.x-google/../../../../arm-linux-androideabi/bin/ld.gold:
obj/third_party/icu/source/i18n/icui18n.ucal.o: in function
ucal_getGregorianChange_46:../../third_party/icu/source/i18n/ucal.cpp:270:
error: undefined reference to '__cxa_bad_typeid'

I suspect that since libchromeview.so doesn't have issues normally, that the
linker is optimizing out the code that refers to this. So I'll probably have to
spend more time spelunking in icu.

+                ],
+              }],
               ['android_upstream_bringup==1', {
                 'defines': ['ANDROID_UPSTREAM_BRINGUP=1',],
               }],
@@ -3000,16 +3011,19 @@
                   ['target_arch=="arm" and armv7==1', {
                     'ldflags': [
                       '-L<(android_ndk_root)/sources/cxx-stl/stlport/libs/armeabi-v7a',
+                      '-L<(android_ndk_root)/sources/cxx-stl/gnu-libstdc++/4.6/libs/armeabi-v7a',
                     ],
                   }],
                   ['target_arch=="arm" and armv7==0', {
                     'ldflags': [
                       '-L<(android_ndk_root)/sources/cxx-stl/stlport/libs/armeabi',
+                      '-L<(android_ndk_root)/sources/cxx-stl/gnu-libstdc++/4.6/libs/armeabi',
                     ],
                   }],
                   ['target_arch=="ia32"', {
                     'ldflags': [
                       '-L<(android_ndk_root)/sources/cxx-stl/stlport/libs/x86',
+                      '-L<(android_ndk_root)/sources/cxx-stl/gnu-libstdc++/4.6/libs/x86',
                     ],
                   }],
                 ],