summaryrefslogtreecommitdiffstats
path: root/tools/auto_bisect
diff options
context:
space:
mode:
authorprasadv <prasadv@chromium.org>2014-12-18 14:02:36 -0800
committerCommit bot <commit-bot@chromium.org>2014-12-18 22:03:19 +0000
commit0374e30b2b498a91dad757e854c43590fc20ac52 (patch)
tree222365cd5153fa3957de797c2905f14ebae5de4c /tools/auto_bisect
parentdfa9bb8ffd481f47c52d9d495d59b07f68cfb8c3 (diff)
downloadchromium_src-0374e30b2b498a91dad757e854c43590fc20ac52.zip
chromium_src-0374e30b2b498a91dad757e854c43590fc20ac52.tar.gz
chromium_src-0374e30b2b498a91dad757e854c43590fc20ac52.tar.bz2
Avoid skipping DEPS change revisions for chromium repo.
BUG=442809 NOTRY=true Review URL: https://codereview.chromium.org/818503002 Cr-Commit-Position: refs/heads/master@{#309082}
Diffstat (limited to 'tools/auto_bisect')
-rwxr-xr-xtools/auto_bisect/bisect_perf_regression.py8
1 files changed, 5 insertions, 3 deletions
diff --git a/tools/auto_bisect/bisect_perf_regression.py b/tools/auto_bisect/bisect_perf_regression.py
index 9ad9189..15bef30 100755
--- a/tools/auto_bisect/bisect_perf_regression.py
+++ b/tools/auto_bisect/bisect_perf_regression.py
@@ -1400,8 +1400,9 @@ class BisectPerformanceMetrics(object):
def ShouldSkipRevision(depot, revision):
"""Checks whether a particular revision can be safely skipped.
- Some commits can be safely skipped (such as a DEPS roll), since the tool
- is git based those changes would have no effect.
+ Some commits can be safely skipped (such as a DEPS roll for the repos
+ still using .DEPS.git), since the tool is git based those changes
+ would have no effect.
Args:
depot: The depot being bisected.
@@ -1410,7 +1411,8 @@ class BisectPerformanceMetrics(object):
Returns:
True if we should skip building/testing this revision.
"""
- if depot == 'chromium':
+ # Skips revisions with DEPS on android-chrome.
+ if depot == 'android-chrome':
cmd = ['diff-tree', '--no-commit-id', '--name-only', '-r', revision]
output = bisect_utils.CheckRunGit(cmd)