Support multiple proguard flags from java_apk.gypi. Proguard supports multiple proguard configurati…

build/android/tests/multiple_proguards/multiple_proguards.gyp

Index: build/android/tests/multiple_proguards/multiple_proguards.gyp
diff --git a/build/android/tests/multiple_proguards/multiple_proguards.gyp b/build/android/tests/multiple_proguards/multiple_proguards.gyp
new file mode 100644
index 0000000000000000000000000000000000000000..d996442f66959dba2ccb99b79beea1f3741eed40
--- /dev/null
+++ b/build/android/tests/multiple_proguards/multiple_proguards.gyp
@@ -0,0 +1,34 @@
+# Copyright (c) 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.
+{
+  'variables': {
+    'chromium_code': 1,
+    'package_name': 'multiple_proguard',
+  },
+  'targets': [
+    {
+      'target_name': 'multiple_proguards_test_apk',
+      'type': 'none',
+      'variables': {
+        'app_manifest_version_name%': '<(android_app_version_name)',
+        'app_manifest_version_code%': '<(android_app_version_code)',

[cjhopman, 2013/03/29 14:43:46]

You shouldn't need the app_manifest_version* variables here, the defaults should
be fine.

[jayjang, 2013/03/29 15:05:19]

Done.

+        'java_in_dir': '../multiple_proguards',

[cjhopman, 2013/03/29 14:43:46]

change this to:
'java_in_dir': 'src',

[jayjang, 2013/03/29 15:05:19]

it couldn't be 'src'. In fact it searches manifest file from java_in_dir. it
should be '.'

+        'proguard_enabled': 'true',
+        'proguard_flags_paths': [
+          '<(DEPTH)/build/android/tests/multiple_proguards/proguard1.flags',

[cjhopman, 2013/03/29 14:43:46]

These paths can be relative to this directory, i.e:
'proguard1.flags',
'proguard2.flags',

[jayjang, 2013/03/29 15:05:19]

Done.

+          '<(DEPTH)/build/android/tests/multiple_proguards/proguard2.flags',
+        ],
+        'R_package': 'dummy',
+        'R_package_relpath': 'dummy',

[cjhopman, 2013/03/29 14:43:46]

I don't think you need these R_package* variables defined here (it should work
without them, but I'm not certain).

[jayjang, 2013/03/29 15:05:19]

Unfortunately it's required because some variables like additional_R_text_files
in java_apk.gypi created when R_package name specified

+        'apk_name': 'MultipleProguards',
+      },
+      'dependencies': [
+        # guava has references to objects using reflection which
+        # should be ignored in proguard step.
+        '../../../../third_party/guava/guava.gyp:guava_javalib',
+      ],
+      'includes': [ '../../../../build/java_apk.gypi' ],
+    },
+  ],
+}