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

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

Issue 1950123003: [Findit] Fetch DEPS from buildspec/ instead of trunk for chrome official builds. (Closed) Base URL: https://chromium.googlesource.com/infra/infra.git@master
Patch Set: Created 4 years, 7 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
Index: appengine/findit/crash/fracas_crash_pipeline.py
diff --git a/appengine/findit/crash/fracas_crash_pipeline.py b/appengine/findit/crash/fracas_crash_pipeline.py
index 381b3757fe101e54e2906b3caf42150df91eb929..aac69a9a03d16b815e96af5f8bddcc50999ac1fd 100644
--- a/appengine/findit/crash/fracas_crash_pipeline.py
+++ b/appengine/findit/crash/fracas_crash_pipeline.py
@@ -55,9 +55,8 @@ class FracasAnalysisPipeline(FracasBasePipeline):
# Run the analysis.
result, tags = fracas.FindCulpritForChromeCrash(
- analysis.channel, analysis.platform, analysis.signature,
- analysis.stack_trace, analysis.crashed_version,
- analysis.historic_metadata)
+ analysis.signature, analysis.platform, analysis.stack_trace,
+ analysis.crashed_version, analysis.historic_metadata)
# Update analysis status and save the analysis result.
analysis.completed_time = datetime.datetime.utcnow()
@@ -108,7 +107,6 @@ def _NeedsNewAnalysis(
# A new analysis is not needed if last one didn't complete or succeeded.
# TODO(http://crbug.com/600535): re-analyze if stack trace or regression
# range changed.
- print 'lala'
return False
if not analysis:

Powered by Google App Engine
This is Rietveld 408576698