Android: makes "apk_package" optional in adb_install_apk.py

build/android/pylib/test_options_parser.py

Index: build/android/pylib/test_options_parser.py
diff --git a/build/android/pylib/test_options_parser.py b/build/android/pylib/test_options_parser.py
index f8644fb8482712fa4211caaa4d2558cd00fe6d09..68eaff99c7e87e8c856651f4d9d910537f29411f 100644
--- a/build/android/pylib/test_options_parser.py
+++ b/build/android/pylib/test_options_parser.py
@@ -35,6 +35,14 @@ def AddInstallAPKOption(option_parser):
                            help=('The package name used by the apk containing '
                                  'the application.'))
 
+
+def ValidateInstallAPKOption(options):
+  if not os.path.exists(options.apk):

[frankf, 2012/10/31 18:51:10]

Can you verify that options.apk and options.build_type are not None and throw an
optionparse error.

[bulach, 2012/10/31 19:00:08]

good point. since there's only one user and it'd be required anyways, I called
AddBuildTypeOption() on AddInstallAPKOption() so that build_type is guaranteed
to be set..

added a test for the apk being None.

+    options.apk = os.path.join(os.environ['CHROME_SRC'],
+                               'out', options.build_type,
+                               'apks', options.apk)
+
+
 def AddTestRunnerOptions(option_parser, default_timeout=60):
   """Decorates OptionParser with options applicable to all tests."""