[Cronet] Don't forget EmbeddedTestServer apk when testing with cr_cronet.py

components/cronet/tools/cr_cronet.py

Index: components/cronet/tools/cr_cronet.py
diff --git a/components/cronet/tools/cr_cronet.py b/components/cronet/tools/cr_cronet.py
index bb19362d647bd6f88ecd2da5c732969684e55ec0..f447e94ca96ea192df7f637daeb027f14a1bb625 100755
--- a/components/cronet/tools/cr_cronet.py
+++ b/components/cronet/tools/cr_cronet.py
@@ -25,7 +25,9 @@ def build(out_dir, extra_options=''):
 
 def install(release_arg):
   return run('build/android/adb_install_apk.py ' + release_arg + \
-             ' --apk=CronetTest.apk')
+             ' --apk=CronetTest.apk') or \

[xunjieli, 2016/04/06 13:10:04]

I don't really know Python. Should this be "and" instead of "or' ? Does Python
evaluate both even though first run returns true?

[pauljensen, 2016/04/06 13:12:53]

run() returns the return code, so 0 is success, so we want to stop when either
returns non-zero.  "or" in Python is short-circuting, so if the first one
returns non-zero it will not run the second one, which is the desire behavior.

http://stackoverflow.com/questions/2580136/does-python-support-short-circuiting

[xunjieli, 2016/04/06 13:14:37]

Ah, that makes sense. Thanks.

+             run('build/android/adb_install_apk.py ' + release_arg + \
+             ' --apk=ChromiumNetTestSupport.apk')
 
 
 def test(release_arg, extra_options):