[WebView] Update CTS logic to load test name from json.

[WebView] Update CTS logic to load test name from json.

We will want to be able to specify multiple CTS test zips for
different android platforms. Adding logic to grab the CTS test
name from a formatted json file isntead of just a text file.

BUG=633739
Committed: https://chromium.googlesource.com/chromium/tools/build/+/9698e1fca3cdca11d25bdef82986b13ab37c24ca

[mikecase (-- gone --), 2016-08-03 01:43:44]

mikecase@chromium.org changed reviewers:
+ yolandyan@chromium.org

[mikecase (-- gone --), 2016-08-03 01:43:45]

Would commit this CL alongside https://codereview.chromium.org/2202853003

Tell me what you think. Your review is powerful enough for both of these CLs to
get committed :)

[the real yoland, 2016-08-03 23:25:02]

https://codereview.chromium.org/2206113002/diff/40001/scripts/slave/recipe_mo...
File scripts/slave/recipe_modules/chromium_android/api.py (right):

https://codereview.chromium.org/2206113002/diff/40001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium_android/api.py:1153: android_platform='L',
arch='arm_64'):
wouldn't the n bot still run this without android_platform specified?

[mikecase (-- gone --), 2016-08-03 23:34:27]

https://codereview.chromium.org/2206113002/diff/40001/scripts/slave/recipe_mo...
File scripts/slave/recipe_modules/chromium_android/api.py (right):

https://codereview.chromium.org/2206113002/diff/40001/scripts/slave/recipe_mo...
scripts/slave/recipe_modules/chromium_android/api.py:1153: android_platform='L',
arch='arm_64'):
On 2016/08/03 at 23:25:01, the real yoland wrote:
> wouldn't the n bot still run this without android_platform specified?

Yes, yeah...............................
So about that...............

2 things.
- Will need to download an N CTS zip and upload it to the bucket.
- It will still not work since the N CTS tests need java8.

Ill update the N CTS tests in a follow up CL if that sounds good.

Also, idk if you have any preference on platform names. I dont see platform
names written out anywhere else in the code. Ill probably have to add whether or
not it is AOSP to the name as well.

'L'
'L_AOSP'

or
'Lollipop'
'Lollipop_AOSP'

or something else?

[the real yoland, 2016-08-03 23:44:39]

On 2016/08/03 at 23:34:27, mikecase wrote:
>
https://codereview.chromium.org/2206113002/diff/40001/scripts/slave/recipe_mo...
> File scripts/slave/recipe_modules/chromium_android/api.py (right):
> 
>
https://codereview.chromium.org/2206113002/diff/40001/scripts/slave/recipe_mo...
> scripts/slave/recipe_modules/chromium_android/api.py:1153:
android_platform='L', arch='arm_64'):
> On 2016/08/03 at 23:25:01, the real yoland wrote:
> > wouldn't the n bot still run this without android_platform specified?
> 
> Yes, yeah...............................
> So about that...............
> 
> 2 things.
> - Will need to download an N CTS zip and upload it to the bucket.
> - It will still not work since the N CTS tests need java8.
> 
> Ill update the N CTS tests in a follow up CL if that sounds good.
> 
> Also, idk if you have any preference on platform names. I dont see platform
names written out anywhere else in the code. Ill probably have to add whether or
not it is AOSP to the name as well.
> 
> 'L'
> 'L_AOSP'
> 
> or
> 'Lollipop'
> 'Lollipop_AOSP'
> 
> or something else?

Got it, so as discussed, we can specify to download different version of cts
test for different bot once java8 issue is solved.
Also, I don't think AOSP matters, CTS is always a part of AOSP :)

[the real yoland, 2016-08-03 23:44:43]

lgtm

[mikecase (-- gone --), 2016-08-05 18:24:51]

The CQ bit was checked by mikecase@chromium.org

[commit-bot: I haz the power, 2016-08-05 18:25:03]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2016-08-05 18:34:24]

Description was changed from

==========
[WebView] Update CTS logic to load test name from json.

We will want to be able to specify multiple CTS test zips for
different android platforms. Adding logic to grab the CTS test
name from a formatted json file isntead of just a text file.

BUG=633739
==========

to

==========
[WebView] Update CTS logic to load test name from json.

We will want to be able to specify multiple CTS test zips for
different android platforms. Adding logic to grab the CTS test
name from a formatted json file isntead of just a text file.

BUG=633739

Committed:
https://chromium.googlesource.com/chromium/tools/build/+/9698e1fca3cdca11d25b...
==========

[commit-bot: I haz the power, 2016-08-05 18:34:25]

Committed patchset #3 (id:40001) as
https://chromium.googlesource.com/chromium/tools/build/+/9698e1fca3cdca11d25b...