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

Unified Diff: appengine/findit/crash/findit_for_chromecrash.py

Issue 2373133002: [Findit] Update found_project and found_components. (Closed)
Patch Set: Created 4 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 | appengine/findit/crash/test/findit_for_chromecrash_test.py » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: appengine/findit/crash/findit_for_chromecrash.py
diff --git a/appengine/findit/crash/findit_for_chromecrash.py b/appengine/findit/crash/findit_for_chromecrash.py
index 407f9f16420c609683915494f079ef9658abe2cc..bd5c3913a5639fa2f29ec82734a97146d0926677 100644
--- a/appengine/findit/crash/findit_for_chromecrash.py
+++ b/appengine/findit/crash/findit_for_chromecrash.py
@@ -130,7 +130,7 @@ def FindCulpritForChromeCrash(signature, platform,
return (
{
'found': (bool(suspected_project) or bool(suspected_components) or
- bool(culprit_results_list)),
+ bool(culprit_results_list) or bool(regression_versions)),
'regression_range': regression_versions,
'suspected_project': suspected_project,
'suspected_components': suspected_components,
@@ -138,6 +138,8 @@ def FindCulpritForChromeCrash(signature, platform,
},
{
'found_suspects': bool(culprit_results_list),
+ 'found_project': bool(suspected_project),
+ 'found_components': bool(suspected_components),
'has_regression_range': bool(regression_versions),
'solution': 'core_algorithm',
}
« no previous file with comments | « no previous file | appengine/findit/crash/test/findit_for_chromecrash_test.py » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698