Do not actually remove trees when test bisect in dry-run mode

tools/auto_bisect/bisect_perf_regression.py

Index: tools/auto_bisect/bisect_perf_regression.py
diff --git a/tools/auto_bisect/bisect_perf_regression.py b/tools/auto_bisect/bisect_perf_regression.py
index ec3e118df267f5206f97389b10d752d1ddf47621..9d90d71f2ebe252e2d9632cdea071c5d443c6548 100755
--- a/tools/auto_bisect/bisect_perf_regression.py
+++ b/tools/auto_bisect/bisect_perf_regression.py
@@ -56,7 +56,7 @@ import math_utils
 import request_build
 import source_control as source_control_module
 import ttest
-from telemetry.util import cloud_storage
+from telemetry.util import cloud_storage # pylint: disable=F0401

[Sergiy Byelozyorov, 2014/09/19 12:35:59]

This was necessary for PRESUBMIT to pass

[ojan, 2014/09/19 17:42:52]

I don't think this is the best fix for this. It looks like
telemetry/telemetry/util is what this is importing. So the other script that
runs this file must have telemetry on it's python path or be run from the
telemetry directory. Ideally we'd fix all the things to do imports from the
root, but that's a big change. So, instead, I think you should change pylint to
put the root telemetry directory in sys.path.

I *think* you need to modify
https://code.google.com/p/chromium/codesearch#chromium/src/PRESUBMIT.py to make
that happen. I don't know for sure though. maruel, do you know this stuff?

[M-A Ruel, 2014/09/20 00:47:44]

pylint is not run from src/PRESUBMIT.py because there's no project level
consistent sys.path setup. It's the one in tools/auto_bisect. There's a way to
pass additional paths to the check.

[Sergiy Byelozyorov, 2014/09/22 20:04:15]

Done.

 
 # Below is the map of "depot" names to information about each depot. Each depot
 # is a repository, and in the process of bisecting, revision ranges in these