.isolated version 1.2 uses 'path_variables' instead of 'variables'.

.isolated version 1.2 uses 'path_variables' instead of 'variables'.

Fix runisolatedtest.py hack to support both. Add note about the life expectancy
of this script.

R=stip@chromium.org
BUG=311622
Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=240727

[M-A Ruel, 2013-12-13 19:47:27]

The .isolated format upgraded broke the expectations in runisolatedtest.py. It
is annoying because this script is only used on the Win Tests builders on the
CI.
https://codereview.chromium.org/101143003/ was the revert caused by this bug.

[ghost stip (do not use), 2013-12-13 19:59:53]

lgtm

[Nico, 2013-12-13 20:00:06]

lgtm

https://codereview.chromium.org/101303005/diff/1/scripts/slave/runisolatedtes...
File scripts/slave/runisolatedtest.py (right):

https://codereview.chromium.org/101303005/diff/1/scripts/slave/runisolatedtes...
scripts/slave/runisolatedtest.py:115: if isolated_data['version'].split('.',
1)[0] != '1':
Should this be a whitelist of versions then after all? Looks like 1.x aren't
backwards-compatible

[M-A Ruel, 2013-12-13 20:02:25]

https://codereview.chromium.org/101303005/diff/1/scripts/slave/runisolatedtes...
File scripts/slave/runisolatedtest.py (right):

https://codereview.chromium.org/101303005/diff/1/scripts/slave/runisolatedtes...
scripts/slave/runisolatedtest.py:115: if isolated_data['version'].split('.',
1)[0] != '1':
On 2013/12/13 20:00:06, Nico wrote:
> Should this be a whitelist of versions then after all? Looks like 1.x aren't
> backwards-compatible

.isolated files *are* backward compatibles. It's because this script is hooking
into .isolated.state, which are explicitly an implementation details.

I don't think this buys us much in the 1.x versions, I think the next change
will be to 2.0 anyhow with the changes I plan to do.

[commit-bot: I haz the power, 2013-12-13 20:13:24]

CQ is trying da patch. Follow status at
https://chromium-status.appspot.com/cq/maruel@chromium.org/101303005/1

[commit-bot: I haz the power, 2013-12-13 20:14:28]

Change committed as 240727