Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(39)

Unified Diff: tools/bisect-perf-regression.py

Issue 608643002: Revert of Add commit position to bisect result output. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 6 years, 3 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/bisect-perf-regression.py
diff --git a/tools/bisect-perf-regression.py b/tools/bisect-perf-regression.py
old mode 100644
new mode 100755
index f812bd93caddb963ce86ee476ac8f3f7d9a5e82e..6166f402b3fc0b3eb050632982cd21281f2ca51b
--- a/tools/bisect-perf-regression.py
+++ b/tools/bisect-perf-regression.py
@@ -2636,10 +2636,9 @@
warning = ''
return confidence_status % {'level': level, 'warning': warning}
- def _GetViewVCLinkFromDepotAndHash(self, revision_id, depot):
- """Gets link to the repository browser."""
- info = self.source_control.QueryRevisionInfo(
- revision_id, self._GetDepotDirectory(depot))
+ def _GetViewVCLinkFromDepotAndHash(self, cl, depot):
+ info = self.source_control.QueryRevisionInfo(cl,
+ self._GetDepotDirectory(depot))
if depot and DEPOT_DEPS_NAME[depot].has_key('viewvc'):
try:
# Format is "git-svn-id: svn://....@123456 <other data>"
@@ -2673,41 +2672,37 @@
def _PrintTestedCommitsHeader(self):
if self.opts.bisect_mode == BISECT_MODE_MEAN:
_PrintTableRow(
- [20, 12, 70, 14, 12, 13],
- ['Depot', 'Position', 'SHA', 'Mean', 'Std. Error', 'State'])
+ [20, 70, 14, 12, 13],
+ ['Depot', 'Commit SHA', 'Mean', 'Std. Error', 'State'])
elif self.opts.bisect_mode == BISECT_MODE_STD_DEV:
_PrintTableRow(
- [20, 12, 70, 14, 12, 13],
- ['Depot', 'Position', 'SHA', 'Std. Error', 'Mean', 'State'])
+ [20, 70, 14, 12, 13],
+ ['Depot', 'Commit SHA', 'Std. Error', 'Mean', 'State'])
elif self.opts.bisect_mode == BISECT_MODE_RETURN_CODE:
_PrintTableRow(
- [20, 12, 70, 14, 13],
- ['Depot', 'Position', 'SHA', 'Return Code', 'State'])
+ [20, 70, 14, 13],
+ ['Depot', 'Commit SHA', 'Return Code', 'State'])
else:
assert False, 'Invalid bisect_mode specified.'
- def _PrintTestedCommitsEntry(self, current_data, commit_position, cl_link,
- state_str):
+ def _PrintTestedCommitsEntry(self, current_data, cl_link, state_str):
if self.opts.bisect_mode == BISECT_MODE_MEAN:
std_error = '+-%.02f' % current_data['value']['std_err']
mean = '%.02f' % current_data['value']['mean']
_PrintTableRow(
- [20, 12, 70, 12, 14, 13],
- [current_data['depot'], commit_position, cl_link, mean, std_error,
- state_str])
+ [20, 70, 12, 14, 13],
+ [current_data['depot'], cl_link, mean, std_error, state_str])
elif self.opts.bisect_mode == BISECT_MODE_STD_DEV:
std_error = '+-%.02f' % current_data['value']['std_err']
mean = '%.02f' % current_data['value']['mean']
_PrintTableRow(
- [20, 12, 70, 12, 14, 13],
- [current_data['depot'], commit_position, cl_link, std_error, mean,
- state_str])
+ [20, 70, 12, 14, 13],
+ [current_data['depot'], cl_link, std_error, mean, state_str])
elif self.opts.bisect_mode == BISECT_MODE_RETURN_CODE:
mean = '%d' % current_data['value']['mean']
_PrintTableRow(
- [20, 12, 70, 14, 13],
- [current_data['depot'], commit_position, cl_link, mean,
- state_str])
+ [20, 70, 14, 13],
+ [current_data['depot'], cl_link, mean, state_str])
def _PrintTestedCommitsTable(
self, revision_data_sorted, first_working_revision, last_broken_revision,
@@ -2747,9 +2742,7 @@
current_data['depot'])
if not cl_link:
cl_link = current_id
- commit_position = self.source_control.GetCommitPosition(current_id)
- self._PrintTestedCommitsEntry(current_data, commit_position, cl_link,
- state_str)
+ self._PrintTestedCommitsEntry(current_data, cl_link, state_str)
def _PrintReproSteps(self):
"""Prints out a section of the results explaining how to run the test.
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698