https://bugs.webkit.org/show_bug.cgi?id=155549
Reviewed by Chris Dumez.
Since a progression is desirable, we should close an analysis task once its cause is identified.
Also fix some typos.
* init-database.sql: Fixed a typo.
* public/api/analysis-tasks.php:
* public/v3/models/analysis-task.js:
(AnalysisTask.prototype.dissociateBug): Renamed from dissociateBug.
* public/v3/pages/analysis-task-page.js:
(AnalysisTaskPage.prototype.render):
(AnalysisTaskPage.prototype._dissociateBug): Renamed from _dissociateBug.
(AnalysisTaskPage.prototype._dissociateCommit): Fixed the typo in the alert.
git-svn-id: http://svn.webkit.org/repository/webkit/trunk@198351
268f45cc-cd09-0410-ab3c-
d52691b4dbfc
+2016-03-17 Ryosuke Niwa <rniwa@webkit.org>
+
+ An analysis task should be closed if a progression cause is identified
+ https://bugs.webkit.org/show_bug.cgi?id=155549
+
+ Reviewed by Chris Dumez.
+
+ Since a progression is desirable, we should close an analysis task once its cause is identified.
+
+ Also fix some typos.
+
+ * init-database.sql: Fixed a typo.
+ * public/api/analysis-tasks.php:
+ * public/v3/models/analysis-task.js:
+ (AnalysisTask.prototype.dissociateBug): Renamed from dissociateBug.
+ * public/v3/pages/analysis-task-page.js:
+ (AnalysisTaskPage.prototype.render):
+ (AnalysisTaskPage.prototype._dissociateBug): Renamed from _dissociateBug.
+ (AnalysisTaskPage.prototype._dissociateCommit): Fixed the typo in the alert.
+
2016-03-16 Ryosuke Niwa <rniwa@webkit.org>
Analysis task page should allow specifying commits that caused or fixed a regression or a progression
CREATE TABLE task_commits (
taskcommit_task integer NOT NULL REFERENCES analysis_tasks ON DELETE CASCADE,
taskcommit_commit integer NOT NULL REFERENCES commits ON DELETE CASCADE,
- taskcommit_is_fix boolean NOT NULL
+ taskcommit_is_fix boolean NOT NULL,
CONSTRAINT task_commit_must_be_unique UNIQUE(taskcommit_task, taskcommit_commit));
CREATE TABLE bugs (
$category = 'unconfirmed';
$result = $task['result'];
- if ($result == 'unchanged' || $result == 'inconclusive' || $task['fixes'])
+ if ($result == 'unchanged' || $result == 'inconclusive' || $task['fixes'] || ($result == 'progression' && $task['causes']))
$category = 'closed';
else if ($task['causes'])
$category = 'identified';
});
}
- disassociateBug(bug)
+ dissociateBug(bug)
{
console.assert(bug instanceof Bug);
console.assert(this.bugs().includes(bug));
var self = this;
this._bugList.setList(this._task.bugs().map(function (bug) {
return new MutableListItem(bug.bugTracker(), bug.label(), bug.title(), bug.url(),
- 'Disassociate this bug', self._disassociateBug.bind(self, bug));
+ 'Disassociate this bug', self._dissociateBug.bind(self, bug));
}));
this._causeList.setList(this._task.causes().map(this._makeCommitListItem.bind(this)));
});
}
- _disassociateBug(bug)
+ _dissociateBug(bug)
{
var render = this.render.bind(this);
- return this._task.disassociateBug(bug).then(render, function (error) {
+ return this._task.dissociateBug(bug).then(render, function (error) {
render();
- alert('Failed to disassociate the bug: ' + error);
+ alert('Failed to dissociate the bug: ' + error);
});
}
var render = this.render.bind(this);
return this._task.dissociateCommit(commit).then(render, function (error) {
render();
- alert('Failed to disassociate the commit: ' + error);
+ alert('Failed to dissociate the commit: ' + error);
});
}