Title: [92360] trunk/Tools
Revision
92360
Author
[email protected]
Date
2011-08-04 00:56:34 -0700 (Thu, 04 Aug 2011)

Log Message

Another subtle bug due to lack of testing in main.js.  :(

* BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.js:

Modified Paths

Diff

Modified: trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.js (92359 => 92360)


--- trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.js	2011-08-04 07:52:12 UTC (rev 92359)
+++ trunk/Tools/BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.js	2011-08-04 07:56:34 UTC (rev 92360)
@@ -164,7 +164,7 @@
 
 function failureInfoFromResultsDetail()
 {
-    var failureDetails = $('.failure-details', $(this).parents('.results-detail'));
+    var failureDetails = $('.results-detail .failure-details');
     return {
         'builderName': failureDetails.attr(config.kBuilderNameAttr),
         'testName': failureDetails.attr(config.kTestNameAttr),

Modified: trunk/Tools/ChangeLog (92359 => 92360)


--- trunk/Tools/ChangeLog	2011-08-04 07:52:12 UTC (rev 92359)
+++ trunk/Tools/ChangeLog	2011-08-04 07:56:34 UTC (rev 92360)
@@ -1,5 +1,11 @@
 2011-08-04  Adam Barth  <[email protected]>
 
+        Another subtle bug due to lack of testing in main.js.  :(
+
+        * BuildSlaveSupport/build.webkit.org-config/public_html/TestFailures/main.js:
+
+2011-08-04  Adam Barth  <[email protected]>
+
         Fix types.  Sadly, main.js has no test coverage.  (I need to think
         about how to test this part of the code.)
 
_______________________________________________
webkit-changes mailing list
[email protected]
http://lists.webkit.org/mailman/listinfo.cgi/webkit-changes

Reply via email to