Cleanup gms update/preprocess scripts, roll android_tools

build/android/play_services/config.json

Index: build/android/play_services/config.json
diff --git a/build/android/play_services/config.json b/build/android/play_services/config.json
index f4f9c6e8e206382cb08b1117221aa3cfd906f4cb..4a9a9d06aef177b83e8fe68235aedae4d0910c97 100644
--- a/build/android/play_services/config.json
+++ b/build/android/play_services/config.json
@@ -1,3 +1,4 @@
 {
-  "version_number": 8298000
+  "version_number": 8298000,
+  "version_xml_path": "res/values/version.xml"

[jbudorick, 2015/12/02 19:09:40]

Should this file also have sdk_version, clients, and locale_whitelist? If not,
how do those get added?

[dgn, 2015/12/03 16:36:57]

This is a bit weird because it's read/write file.
Scripts can write it to update the version number when pushing a new gmscore
update, which affects the download script
People can write it to change the configuration of the update or preprocess
script

Upstream we have no use of the other arguments, since we don't do the
preprocessing. That's why they are not included in this file.

[jbudorick, 2015/12/10 14:16:31]

So what happens if somewhat attempts to use one of these other fields via
ConfigParser?

[dgn, 2015/12/10 14:35:19]

Currently it would raise a KeyError when requesting one of the missing fields,
but since they are looked at lazily, when trying to read the field, it sounds
okay to me as way to communicate the issue. Is it better to have placeholder
values?

 }