[Android] Only set recommended flags if they are unset

This allows developers to force the values of individual flags in their
chrome/supplement.gypi. Add documentation for how a user can do that.

Also add a comment about the fastbuild option, but do not ever set it
by default.

NOTRY=true
BUG=158821

Review URL: https://chromiumcodereview.appspot.com/14366023

git-svn-id: svn://svn.chromium.org/chrome/trunk/src@195334 0039d316-1c4b-4281-b951-d872f2087c98

1 files changed, 21 insertions, 6 deletions

diff --git a/build/android/developer_recommended_flags.gypi b/build/android/developer_recommended_flags.gypi
index 1bd6bf8..2edf9ab 100644
--- a/build/android/developer_recommended_flags.gypi
+++ b/build/android/developer_recommended_flags.gypi
@@ -4,23 +4,33 @@
 
 # This is the set of recommended gyp variable settings for Chrome for Android development.
 #
-# These can be used by copying them into ~/.gyp/include.gypi or $CHROME_SRC/chrome/supplement.gypi
-# Even better, use the below in chrome/supplement.gypi:
+# These can be used by copying this file to $CHROME_SRC/chrome/supplement.gypi.
+#
+# Even better, create chrome/supplement.gypi containing the following:
 #   {
 #     'includes': [ '../build/android/developer_recommended_flags.gypi' ]
 #   }
 # and you'll get new settings automatically.
+# When using this method, you can override individual settings by setting them unconditionally (with
+# no %) in chrome/supplement.gypi.
+# I.e. to disable optimize_jni_generation but use everything else:
+#   {
+#     'variables': {
+#       'optimize_jni_generation': 0,
+#     },
+#     'includes': [ '../build/android/developer_recommended_flags.gypi' ]
+#   }
 
 {
   'variables': {
     # When set to 1, only write jni generated files if they've changed. This can prevent unnecessary
     # compiling/linking of native libraries when editing java files.
-    'optimize_jni_generation': 1,
+    'optimize_jni_generation%': 1,
 
     # Set component to 'shared_library' to enable the component build. This builds native code as
     # many small shared libraries instead of one monolithic library. This slightly reduces the time
     # required for incremental builds.
-    'component': 'shared_library',
+    'component%': 'shared_library',
 
     # When gyp_managed_install is set to 1, building an APK will install that APK on the connected
     # device(/emulator). To install on multiple devices (or onto a new device), build the APK once
@@ -30,12 +40,17 @@
     #   Only works with a single device connected.
     #   Some actions are always run (i.e. ninja will never say "no work to do").
     #   Native libraries are not packaged in the APK (you can not manually install the APK)
-    'gyp_managed_install': 1,
+    'gyp_managed_install%': 1,
 
     # Set clang to 1 to use the clang compiler. Clang has much (much, much) better warning/error
     # messages than gcc.
     # TODO(cjhopman): Enable this when http://crbug.com/156420 is addressed. Until then, users can
     # set clang to 1, but Android stack traces will sometimes be incomplete.
-    #'clang': 1,
+    #'clang%': 1,
+
+    # Set fastbuild to 1 to build with less debugging information. This can greatly decrease linking
+    # time. The downside is that stack traces will be missing useful information (like line
+    # numbers).
+    #'fastbuild%': 1,
   },
 }