Use JSON instead of YAML for the play services scripts config

Use JSON instead of YAML for the play services scripts config

Bots don't have the yaml python module installed. JSON is
supported. Added a parser object that will also hold the
description of the different possible fields since JSON
doesn't support comments

BUG=541727
Committed: https://crrev.com/24f457db32c79f72e5a445bcdf5d6bc0647f4edb
Cr-Commit-Position: refs/heads/master@{#357656}

[dgn, 2015-10-28 18:27:20]

dgn@chromium.org changed reviewers:
+ jbudorick@chromium.org

[dgn, 2015-10-28 18:27:21]

PTAL

[dgn, 2015-11-02 14:06:47]

On 2015/10/28 18:27:21, dgn wrote:
> PTAL

It seems that most bots have the yaml python module installed, looking at the
results of this batch try jobs:
https://codereview.chromium.org/1418573010/#ps100001

[jbudorick, 2015-11-03 02:54:41]

lgtm w/ nit

https://codereview.chromium.org/1411183004/diff/20001/build/android/play_serv...
File build/android/play_services/utils.py (right):

https://codereview.chromium.org/1411183004/diff/20001/build/android/play_serv...
build/android/play_services/utils.py:65: # Reload the configuration in case the
cached version isn't up to date
nit: This seems unnecessary. How often will the configuration change while a
script using the class is running?

[dgn, 2015-11-03 10:36:31]

Thanks!

https://codereview.chromium.org/1411183004/diff/20001/build/android/play_serv...
File build/android/play_services/utils.py (right):

https://codereview.chromium.org/1411183004/diff/20001/build/android/play_serv...
build/android/play_services/utils.py:65: # Reload the configuration in case the
cached version isn't up to date
On 2015/11/03 02:54:41, jbudorick wrote:
> nit: This seems unnecessary. How often will the configuration change while a
> script using the class is running?

Should be extremely unlikely, yes. Removed

[dgn, 2015-11-03 10:36:34]

The CQ bit was checked by dgn@chromium.org

[dgn, 2015-11-03 10:36:35]

The patchset sent to the CQ was uploaded after l-g-t-m from
jbudorick@chromium.org
Link to the patchset: https://codereview.chromium.org/1411183004/#ps40001
(title: "Skip reloading config before save")

[commit-bot: I haz the power, 2015-11-03 10:36:55]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1411183004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1411183004/40001

[commit-bot: I haz the power, 2015-11-03 12:55:28]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2015-11-03 12:55:29]

Try jobs failed on following builders:
  mac_chromium_rel_ng on tryserver.chromium.mac (JOB_FAILED,
http://build.chromium.org/p/tryserver.chromium.mac/builders/mac_chromium_rel_...)

[dgn, 2015-11-03 21:28:09]

The CQ bit was checked by dgn@chromium.org

[commit-bot: I haz the power, 2015-11-03 21:30:36]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1411183004/40001
View timeline at
 https://chromium-cq-status.appspot.com/patch-timeline/1411183004/40001

[commit-bot: I haz the power, 2015-11-03 22:39:28]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2015-11-03 22:41:17]

Patchset 3 (id:??) landed as
https://crrev.com/24f457db32c79f72e5a445bcdf5d6bc0647f4edb
Cr-Commit-Position: refs/heads/master@{#357656}