Title: [90639] trunk/Tools
- Revision
- 90639
- Author
- aro...@apple.com
- Date
- 2011-07-08 09:09:24 -0700 (Fri, 08 Jul 2011)
Log Message
Teach TestFailures how to find test names in commit-log-editor-style commit messages
TestFailures was relying on Trac turning the list of modified files in our commit messages
into an HTML list. But Trac only does this when the list of modified files is indented.
commit-log-editor doesn't indent the file list, so the list wasn't being turned into an HTML
list, which was confusing TestFailures.
TestFailures now does much simpler parsing of the commit message (i.e., just a substring
search) without relying at all on its structure.
Fixes <http://webkit.org/b/64173> TestFailures page fails to blame r90608 for breaking
fast/dom/HTMLProgressElement/progress-element-markup.html on Windows
Reviewed by David Kilzer.
* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Trac.js:
(Trac.prototype.getCommitDataForRevisionRange): Instead of trying to parse the commit
message, just return its text.
* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:
(ViewController.prototype._domForRegressionRange): Instead of searching for test names in
each commit's list of modified files, just search for test names anywhere in the commit's
message.
Modified Paths
Diff
Modified: trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Trac.js (90638 => 90639)
--- trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Trac.js 2011-07-08 16:04:28 UTC (rev 90638)
+++ trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Trac.js 2011-07-08 16:09:24 UTC (rev 90639)
@@ -70,19 +70,13 @@
var container = document.createElement('div');
container.innerHTML = item.getElementsByTagName('description')[0].textContent;
- var listItems = container.querySelectorAll('li');
- var files = [];
- for (var i = 0; i < listItems.length; ++i) {
- var match = /^([^:]+)/.exec(listItems[i].textContent);
- if (!match)
- continue;
- files.push(match[1]);
- }
return {
revision: revision,
title: title,
- modifiedFiles: files,
+ // FIXME: This isn't a very high-fidelity reproduction of the commit message,
+ // but it's good enough for our purposes.
+ message: container.innerText,
};
});
Modified: trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js (90638 => 90639)
--- trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js 2011-07-08 16:04:28 UTC (rev 90638)
+++ trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js 2011-07-08 16:09:24 UTC (rev 90639)
@@ -204,10 +204,8 @@
self._trac.getCommitDataForRevisionRange('trunk', firstSuspectRevision, lastSuspectRevision, function(commits) {
var failingTestNamesWithoutExtensions = failingTestNames.map(removePathExtension);
var suspectCommits = commits.filter(function(commit) {
- return commit.modifiedFiles.some(function(file) {
- return failingTestNamesWithoutExtensions.some(function(testName) {
- return file.indexOf(testName) >= 0;
- });
+ return failingTestNamesWithoutExtensions.some(function(testName) {
+ return commit.message.contains(testName);
});
});
Modified: trunk/Tools/ChangeLog (90638 => 90639)
--- trunk/Tools/ChangeLog 2011-07-08 16:04:28 UTC (rev 90638)
+++ trunk/Tools/ChangeLog 2011-07-08 16:09:24 UTC (rev 90639)
@@ -1,3 +1,29 @@
+2011-07-08 Adam Roben <aro...@apple.com>
+
+ Teach TestFailures how to find test names in commit-log-editor-style commit messages
+
+ TestFailures was relying on Trac turning the list of modified files in our commit messages
+ into an HTML list. But Trac only does this when the list of modified files is indented.
+ commit-log-editor doesn't indent the file list, so the list wasn't being turned into an HTML
+ list, which was confusing TestFailures.
+
+ TestFailures now does much simpler parsing of the commit message (i.e., just a substring
+ search) without relying at all on its structure.
+
+ Fixes <http://webkit.org/b/64173> TestFailures page fails to blame r90608 for breaking
+ fast/dom/HTMLProgressElement/progress-element-markup.html on Windows
+
+ Reviewed by David Kilzer.
+
+ * BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/Trac.js:
+ (Trac.prototype.getCommitDataForRevisionRange): Instead of trying to parse the commit
+ message, just return its text.
+
+ * BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/ViewController.js:
+ (ViewController.prototype._domForRegressionRange): Instead of searching for test names in
+ each commit's list of modified files, just search for test names anywhere in the commit's
+ message.
+
2011-07-08 Adam Barth <aba...@webkit.org>
sheriffbot is too spammy in IRC
_______________________________________________
webkit-changes mailing list
webkit-changes@lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes
