Introduce skia_pic gyp variable.

gyp/common_conditions.gypi

 # conditions used in both common.gypi and skia.gyp in chromium
 #
 {
   'defines': [
     'SK_ALLOW_STATIC_GLOBAL_INITIALIZERS=<(skia_static_initializers)',
     'SK_SUPPORT_GPU=<(skia_gpu)',
     'SK_SUPPORT_OPENCL=<(skia_opencl)',
     'SK_FORCE_DISTANCEFIELD_FONTS=<(skia_force_distancefield_fonts)',
   ],
   'conditions' : [
+    ['skia_pic', {
+     'cflags': [
+       '-fPIC',
+     ],
+    }],
     [ 'skia_arch_type == "arm64"', {
       'cflags': [
         '-ffp-contract=off',
       ],
     }],
 
     # As of M35, Chrome requires SSE2 on x86 (and SSSE3 on Mac).
     [ 'skia_arch_type == "x86"', {
       'cflags': [
         '-msse2',
@@ skipping 375 matching lines @@
           },
           'Release': {
             'cflags': [
               '-O<(skia_release_optimization_level)',
             ],
             'defines': [ 'NDEBUG' ],
           },
         },
         'conditions' : [
           [ 'skia_shared_lib', {
-            'cflags': [
-              '-fPIC',
-            ],
             'defines': [
               'SKIA_DLL',
               'SKIA_IMPLEMENTATION=1',
             ],
           }],
           [ 'skia_os == "nacl"', {
             'defines': [
               'SK_BUILD_FOR_NACL',
             ],
             'variables': {
@@ skipping 23 matching lines @@
           [ 'skia_sanitizer', {
             'cflags': [
               '-fsanitize=<(skia_sanitizer)',
             ],
             'ldflags': [
               '-fsanitize=<(skia_sanitizer)',
             ],
             'conditions' : [
               [ 'skia_sanitizer == "thread"', {
                 'defines': [ 'SK_DYNAMIC_ANNOTATIONS_ENABLED=1' ],
-                'cflags': [ '-fPIC' ],
                 'target_conditions': [
                   [ '_type == "executable"', {

[mtklein, 2014/07/19 14:49:19]

Can we make skia_pic imply PIE executables too, or is that somehow now required
for Android?

I guess it's moot if we're always deploying as shared_libraries on our Android
bots?

[djsollen, 2014/07/22 14:20:17]

We need PIE for android executables as well.  The only android executable is
currently skia_launcher and it explicitly sets that in the gyp.

[scroggo, 2014/07/22 14:53:36]

(Derek and I discussed in person, but for posterity:)
Moving this into skia_pic breaks Android. Our Android build currently does
something sneaky: our tools (e.g. SampleApp) use the type 'executable', but then
we change its type to 'shared_library' after the fact (see
platform_tools/android/gyp/dependencies.gyp). Then skia_launcher includes that
shared library. When I run android_ninja, SampleApp is considered an executable
(I'm guessing gyp reaches here before reaching the gyp file to change the type),
so it complains that executable is lacking a function main.

I can think of a few solutions:
- Don't define skia_pic for Android (downside here is that if someone *does*
  define it locally, they won't build)
- Don't include this as part of skia_pic (reduces code sharing)
- Change the way the executables get turned into shared libraries on Android
  (the current approach has a comment that it is designed to increase code
  sharing, but perhaps we can come up with another solution that does not
  reduce it).
- Skip the skia_launcher, and actually build the executables as executables
  (I'm sure there is a good reason for using skia_launcher though)
- Skip this part on Android (Derek actually proposed this, and it's probably
  the simplest approach.)

Thoughts?

                     'cflags': [ '-fPIE' ],
                     'ldflags': [ '-pie' ],
                   }],
                 ],
               }],
               [ 'skia_sanitizer == "undefined"', {
-                'cflags': [ '-fPIC' ],
                 'cflags_cc!': ['-fno-rtti'],
                 'target_conditions': [
                   [ '_type == "executable"', {
                     'cflags': [ '-fPIE' ],
                     'ldflags': [ '-pie' ],
                   }],
                 ],
               }],
             ],
           }],
@@ skipping 185 matching lines @@
           [ 'skia_android_framework', {
             'libraries!': [
               '-lstdc++',
               '-lm',
             ],
             'cflags!': [
               '-fuse-ld=gold',
             ],
           }],
           [ 'skia_shared_lib', {
-            'cflags': [
-              '-fPIC',
-            ],
             'defines': [
               'SKIA_DLL',
               'SKIA_IMPLEMENTATION=1',
               # Needed until we fix skbug.com/2440.
               'SK_SUPPORT_LEGACY_CLIPTOLAYERFLAG',
             ],
           }],
           [ 'skia_profile_enabled == 1', {
             'cflags': ['-g', '-fno-omit-frame-pointer', '-marm', '-mapcs'],
           }],
@@ skipping 18 matching lines @@
     [ 'skia_crash_handler', {
       'defines': [ 'SK_CRASH_HANDLER' ],
     }],
 
   ], # end 'conditions'
   # The Xcode SYMROOT must be at the root. See build/common.gypi in chromium for more details
   'xcode_settings': {
     'SYMROOT': '<(DEPTH)/xcodebuild',
   },
 }