test_isolation_outdir must be considered an opaque string, it could be an url.

build/isolate.gypi

Index: build/isolate.gypi
diff --git a/build/isolate.gypi b/build/isolate.gypi
index d7523fdfb389028747bca14cc61ceb2d82ca407b..2b80c42ac60ebf9e7bba84158b9c14c1c90d2362 100644
--- a/build/isolate.gypi
+++ b/build/isolate.gypi
@@ -55,21 +55,30 @@
       'outputs': [
         '<(PRODUCT_DIR)/<(RULE_INPUT_ROOT).isolated',
       ],
-      'action': [
-        'python',
-        '<(DEPTH)/tools/swarm_client/isolate.py',
-        '<(test_isolation_mode)',
-        '--outdir', '<(test_isolation_outdir)',
-        '--variable', 'PRODUCT_DIR', '<(PRODUCT_DIR)',
-        '--variable', 'OS', '<(OS)',
-        '--variable', 'chromeos', '<(chromeos)',
-        '--result', '<@(_outputs)',
-        '--isolate', '<(RULE_INPUT_PATH)',
-      ],
       'conditions': [
-        ["test_isolation_outdir!=''", {
+        ["test_isolation_outdir==''", {

[M-A Ruel, 2013/03/13 19:20:01]

changed to be more explicit.

+          'action': [
+            'python',
+            '<(DEPTH)/tools/swarm_client/isolate.py',
+            '<(test_isolation_mode)',
+            '--outdir', '<(PRODUCT_DIR)',
+            '--variable', 'PRODUCT_DIR', '<(PRODUCT_DIR)',
+            '--variable', 'OS', '<(OS)',
+            '--variable', 'chromeos', '<(chromeos)',
+            '--result', '<@(_outputs)',
+            '--isolate', '<(RULE_INPUT_PATH)',
+          ],
+        }, {
           'action': [
-            '--outdir', '<(PRODUCT_DIR)/<(test_isolation_outdir)',
+            'python',
+            '<(DEPTH)/tools/swarm_client/isolate.py',
+            '<(test_isolation_mode)',
+            '--outdir', '<(test_isolation_outdir)',
+            '--variable', 'PRODUCT_DIR', '<(PRODUCT_DIR)',
+            '--variable', 'OS', '<(OS)',
+            '--variable', 'chromeos', '<(chromeos)',
+            '--result', '<@(_outputs)',
+            '--isolate', '<(RULE_INPUT_PATH)',
           ],
         }],
         ['test_isolation_fail_on_missing == 0', {